From 30c48c66b8fdc523b4fa3bf9649a4fb45dde0e67 Mon Sep 17 00:00:00 2001 From: Joel Klinghed Date: Tue, 5 Nov 2024 22:49:58 +0100 Subject: Style cleanup Remove unused parameter names, extra colons, code that is never executed and an opinionated detekt check. --- libs/samba/src/main/cpp/samba.cpp | 42 +++++++++++++++++++-------------------- 1 file changed, 21 insertions(+), 21 deletions(-) (limited to 'libs/samba/src/main/cpp/samba.cpp') diff --git a/libs/samba/src/main/cpp/samba.cpp b/libs/samba/src/main/cpp/samba.cpp index 38ad5da..1632a05 100644 --- a/libs/samba/src/main/cpp/samba.cpp +++ b/libs/samba/src/main/cpp/samba.cpp @@ -219,19 +219,19 @@ class Context { std::shared_ptr context_; }; -jlong nativeContextNew(JNIEnv* env, jclass clazz, jint timeout) { +jlong nativeContextNew(JNIEnv*, jclass, jint timeout) { return reinterpret_cast(new Context(static_cast(timeout))); } -void nativeContextDestroy(JNIEnv* env, jclass clazz, jlong ptr) { +void nativeContextDestroy(JNIEnv*, jclass, jlong ptr) { delete reinterpret_cast(ptr); } -jlong nativeContextParseUrl(JNIEnv* env, jclass clazz, jlong ptr, jstring url) { +jlong nativeContextParseUrl(JNIEnv* env, jclass, jlong ptr, jstring url) { return reinterpret_cast(reinterpret_cast(ptr)->ParseUrl(jni::StringToUTF8(env, jni::ParamRef(env, url))).release()); } -jboolean nativeContextConnect(JNIEnv* env, jclass clazz, jlong context_ptr, jlong url_ptr, jstring j_username, jstring j_password) { +jboolean nativeContextConnect(JNIEnv* env, jclass, jlong context_ptr, jlong url_ptr, jstring j_username, jstring j_password) { auto* url = reinterpret_cast(url_ptr); if (!url) return JNI_FALSE; std::optional username; @@ -241,58 +241,58 @@ jboolean nativeContextConnect(JNIEnv* env, jclass clazz, jlong context_ptr, jlon return reinterpret_cast(context_ptr)->Connect(*url, username, password) ? JNI_TRUE : JNI_FALSE; } -jstring nativeContextGetError(JNIEnv* env, jclass clazz, jlong ptr) { +jstring nativeContextGetError(JNIEnv* env, jclass, jlong ptr) { return jni::UTF8ToString(env, std::string(reinterpret_cast(ptr)->GetError())).release(); } -jlong nativeContextOpenDir(JNIEnv* env, jclass clazz, jlong ptr, jstring path) { +jlong nativeContextOpenDir(JNIEnv* env, jclass, jlong ptr, jstring path) { return reinterpret_cast(reinterpret_cast(ptr)->OpenDir(jni::StringToUTF8(env, jni::ParamRef(env, path))).release()); } -jobject nativeContextEntry(JNIEnv* env, jclass clazz, jlong ptr, jstring path) { +jobject nativeContextEntry(JNIEnv* env, jclass, jlong ptr, jstring path) { return reinterpret_cast(ptr)->Entry(env, jni::StringToUTF8(env, jni::ParamRef(env, path))).release(); } -jboolean nativeContextMakeDir(JNIEnv* env, jclass clazz, jlong ptr, jstring path) { +jboolean nativeContextMakeDir(JNIEnv* env, jclass, jlong ptr, jstring path) { return reinterpret_cast(ptr)->MakeDir(jni::StringToUTF8(env, jni::ParamRef(env, path))) ? JNI_TRUE : JNI_FALSE; } -jboolean nativeContextRemoveDir(JNIEnv* env, jclass clazz, jlong ptr, jstring path) { +jboolean nativeContextRemoveDir(JNIEnv* env, jclass, jlong ptr, jstring path) { return reinterpret_cast(ptr)->RemoveDir(jni::StringToUTF8(env, jni::ParamRef(env, path))) ? JNI_TRUE : JNI_FALSE; } -jboolean nativeContextUnlink(JNIEnv* env, jclass clazz, jlong ptr, jstring path) { +jboolean nativeContextUnlink(JNIEnv* env, jclass, jlong ptr, jstring path) { return reinterpret_cast(ptr)->Unlink(jni::StringToUTF8(env, jni::ParamRef(env, path))) ? JNI_TRUE : JNI_FALSE; } -jlong nativeContextOpenFile(JNIEnv* env, jclass clazz, jlong ptr, jstring path, jint mode) { +jlong nativeContextOpenFile(JNIEnv* env, jclass, jlong ptr, jstring path, jint mode) { return reinterpret_cast(reinterpret_cast(ptr)->OpenFile(jni::StringToUTF8(env, jni::ParamRef(env, path)), mode).release()); } -void nativeUrlDestroy(JNIEnv* env, jclass clazz, jlong ptr) { +void nativeUrlDestroy(JNIEnv*, jclass, jlong ptr) { delete reinterpret_cast(ptr); } -jstring nativeUrlPath(JNIEnv* env, jclass clazz, jlong ptr) { +jstring nativeUrlPath(JNIEnv* env, jclass, jlong ptr) { auto* path = reinterpret_cast(ptr)->path(); return jni::UTF8ToString(env, std::string(path ? path : "")).release(); } -void nativeDirDestroy(JNIEnv* env, jclass clazz, jlong ptr) { +void nativeDirDestroy(JNIEnv*, jclass, jlong ptr) { delete reinterpret_cast(ptr); } jni::GlobalRef g_DirEntryClass(nullptr, nullptr); -jobjectArray nativeDirList(JNIEnv* env, jclass clazz, jlong ptr) { +jobjectArray nativeDirList(JNIEnv* env, jclass, jlong ptr) { return reinterpret_cast(ptr)->List(env, g_DirEntryClass).release(); } -void nativeFileDestroy(JNIEnv* env, jclass clazz, jlong ptr) { +void nativeFileDestroy(JNIEnv*, jclass, jlong ptr) { delete reinterpret_cast(ptr); } -jint nativeFileRead(JNIEnv* env, jclass clazz, jlong ptr, jbyteArray array, jint offset, jint length) { +jint nativeFileRead(JNIEnv* env, jclass, jlong ptr, jbyteArray array, jint offset, jint length) { jboolean is_copy = JNI_FALSE; bool critical = true; auto* data = reinterpret_cast(env->GetPrimitiveArrayCritical(array, &is_copy)); @@ -310,7 +310,7 @@ jint nativeFileRead(JNIEnv* env, jclass clazz, jlong ptr, jbyteArray array, jint return ret; } -jint nativeFileWrite(JNIEnv* env, jclass clazz, jlong ptr, jbyteArray array, jint offset, jint length) { +jint nativeFileWrite(JNIEnv* env, jclass, jlong ptr, jbyteArray array, jint offset, jint length) { jboolean is_copy = JNI_FALSE; bool critical = true; auto* data = reinterpret_cast(env->GetPrimitiveArrayCritical(array, &is_copy)); @@ -328,7 +328,7 @@ jint nativeFileWrite(JNIEnv* env, jclass clazz, jlong ptr, jbyteArray array, jin return ret; } -jlong nativeFileSeek(JNIEnv* env, jclass clazz, jlong ptr, jlong offset, jint whence) { +jlong nativeFileSeek(JNIEnv*, jclass, jlong ptr, jlong offset, jint whence) { return reinterpret_cast(ptr)->seek(offset, whence); } @@ -401,7 +401,7 @@ jni::LocalRef CreateDirEntry(JNIEnv* env, const std::string& name, cons } // namespace -jint JNI_OnLoad(JavaVM *vm, void *reserved) { +jint JNI_OnLoad(JavaVM *vm, void *) { auto* env = jni::OnLoad(vm); RegisterSamba(env); @@ -409,7 +409,7 @@ jint JNI_OnLoad(JavaVM *vm, void *reserved) { return jni::JNI_VERSION; } -void JNI_OnUnload(JavaVM *vm, void *reserved) { +void JNI_OnUnload(JavaVM *, void *) { // Not called on Android (or in general), but if it where, this would be the place to unregister. UnregisterSamba(); } -- cgit v1.2.3-70-g09d2