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. --- .../io/documents/DelayedCreationDocumentFile.kt | 2 +- .../cleversync/io/documents/DocumentDirectory.kt | 4 +-- libs/samba/src/main/cpp/jni.hpp | 8 ++--- libs/samba/src/main/cpp/samba.cpp | 42 +++++++++++----------- .../org/the_jk/cleversync/io/samba/NativeSamba.kt | 2 +- 5 files changed, 29 insertions(+), 29 deletions(-) (limited to 'libs') diff --git a/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DelayedCreationDocumentFile.kt b/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DelayedCreationDocumentFile.kt index 9fdfb91..50d5918 100644 --- a/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DelayedCreationDocumentFile.kt +++ b/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DelayedCreationDocumentFile.kt @@ -6,7 +6,7 @@ import java.io.InputStream import java.io.OutputStream import java.time.Instant -internal class DelayedCreationDocumentFile constructor( +internal class DelayedCreationDocumentFile( private val metadata: DocumentMetadata, private val create: () -> DocumentFile ) : ModifiableFile { diff --git a/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DocumentDirectory.kt b/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DocumentDirectory.kt index e8b94aa..23ffad9 100644 --- a/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DocumentDirectory.kt +++ b/libs/documents/src/main/java/org/the_jk/cleversync/io/documents/DocumentDirectory.kt @@ -82,7 +82,7 @@ internal open class DocumentDirectory( override fun modifiableList(): ModifiableDirectory.Content { val list = contentResolver.getAllMetadata( DocumentsContract.buildChildDocumentsUriUsingTree(treeUri, metadata.documentId), - ) ?: throw IOException("Unable to list dir") + ) val dirs = mutableListOf() val files = mutableListOf() val links = mutableListOf() @@ -281,7 +281,7 @@ internal open class DocumentDirectory( override fun equals(other: Any?) = other is DocumentDirectory && getRealTreeDocumentId(other.treeUri) == getRealTreeDocumentId(treeUri) override fun hashCode() = getRealTreeDocumentId(treeUri).hashCode() - override fun toString() = getRealTreeDocumentId(treeUri) ?: "null" + override fun toString() = getRealTreeDocumentId(treeUri) private fun findChild(displayName: String): DocumentMetadata? { val queryArgs = Bundle() diff --git a/libs/samba/src/main/cpp/jni.hpp b/libs/samba/src/main/cpp/jni.hpp index aa79bbe..55b7752 100644 --- a/libs/samba/src/main/cpp/jni.hpp +++ b/libs/samba/src/main/cpp/jni.hpp @@ -24,7 +24,7 @@ template class Ref { public: constexpr Ref() : ptr_(nullptr) {} - constexpr Ref(std::nullptr_t) : ptr_(nullptr) {} + constexpr Ref(std::nullptr_t) : ptr_(nullptr) {} // NOLINT Ref(const Ref&) = delete; [[nodiscard]] T get() const { return ptr_; } @@ -62,7 +62,7 @@ template class LocalRef : public Ref { public: constexpr LocalRef(): Ref() {} - constexpr LocalRef(std::nullptr_t): Ref() {} + constexpr LocalRef(std::nullptr_t): Ref() {} // NOLINT LocalRef(JNIEnv* env, T ptr): Ref(env, ptr) {} ~LocalRef() override { free(); } @@ -109,7 +109,7 @@ template class GlobalRef : public Ref { public: constexpr GlobalRef() : Ref() {} - constexpr GlobalRef(std::nullptr_t) : Ref() {} + constexpr GlobalRef(std::nullptr_t) : Ref() {} // NOLINT GlobalRef(JNIEnv* env, T ptr) : Ref() { if (ptr) { if (!env) @@ -117,7 +117,7 @@ class GlobalRef : public Ref { this->ptr_ = static_cast(env->NewGlobalRef(static_cast(ptr))); } } - GlobalRef(const Ref& other) : Ref() { + GlobalRef(const Ref& other) : Ref() { // NOLINT if (other) { auto* env = AttachCurrentThread(); this->ptr_ = static_cast(env->NewGlobalRef(static_cast(other.get()))); 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(); } diff --git a/libs/samba/src/main/java/org/the_jk/cleversync/io/samba/NativeSamba.kt b/libs/samba/src/main/java/org/the_jk/cleversync/io/samba/NativeSamba.kt index 590fab8..c0f18a4 100644 --- a/libs/samba/src/main/java/org/the_jk/cleversync/io/samba/NativeSamba.kt +++ b/libs/samba/src/main/java/org/the_jk/cleversync/io/samba/NativeSamba.kt @@ -112,7 +112,7 @@ internal object NativeSamba { return nativeContextUnlink(ptr, path) } - override fun openFile(path: String, mode: NativeSamba.OpenMode): File? { + override fun openFile(path: String, mode: OpenMode): File? { val file = nativeContextOpenFile(ptr, path, mode.value) return if (file != 0L) NativeFile(path, file) else null } -- cgit v1.2.3-70-g09d2