Unify native logging styles of LOGD and LOGV

We now only include source file name and line number for LOGD.

Moreover, we remove some debugging logs, which contain a typo `creat`.

Author: JingMatrix

daemon/src/main/jni/logging.h

@@ -36,8 +36,12 @@
 #define LOGE(...) 0
 #else
 #ifndef NDEBUG
-#define LOGD(fmt, ...) __android_log_print(ANDROID_LOG_DEBUG, LOG_TAG, "%s:%d#%s" ": " fmt, __FILE_NAME__, __LINE__, __PRETTY_FUNCTION__ __VA_OPT__(,) __VA_ARGS__)
-#define LOGV(fmt, ...) __android_log_print(ANDROID_LOG_VERBOSE, LOG_TAG, "%s:%d#%s" ": " fmt, __FILE_NAME__, __LINE__, __PRETTY_FUNCTION__ __VA_OPT__(,) __VA_ARGS__)
+#define LOGD(fmt, ...)                                                                             \
+    __android_log_print(ANDROID_LOG_DEBUG, LOG_TAG,                                                \
+                        "%s:%d#%s"                                                                 \
+                        ": " fmt,                                                                  \
+                        __FILE_NAME__, __LINE__, __PRETTY_FUNCTION__ __VA_OPT__(, ) __VA_ARGS__)
+#define LOGV(...) __android_log_print(ANDROID_LOG_VERBOSE, LOG_TAG, __VA_ARGS__)
 #else
 #define LOGD(...) 0
 #define LOGV(...) 0

dex2oat/src/main/cpp/include/logging.h

@@ -15,12 +15,12 @@
 #define LOGE(...) 0
 #else
 #ifndef NDEBUG
-#define LOGD(...) __android_log_print(ANDROID_LOG_DEBUG, LOG_TAG, __VA_ARGS__)
-#define LOGV(fmt, ...)                                                                             \
-    __android_log_print(ANDROID_LOG_VERBOSE, LOG_TAG,                                              \
+#define LOGD(fmt, ...)                                                                             \
+    __android_log_print(ANDROID_LOG_DEBUG, LOG_TAG,                                                \
                         "%s:%d#%s"                                                                 \
                         ": " fmt,                                                                  \
                         __FILE_NAME__, __LINE__, __PRETTY_FUNCTION__ __VA_OPT__(, ) __VA_ARGS__)
+#define LOGV(...) __android_log_print(ANDROID_LOG_VERBOSE, LOG_TAG, __VA_ARGS__)
 #else
 #define LOGD(...) 0
 #define LOGV(...) 0

native/include/common/logging.h

@@ -77,14 +77,13 @@ inline void LogToAndroid(int prio, const char *tag, fmt::format_string<T...> fmt
 }  // namespace vector::native::detail
 
 #ifndef NDEBUG
-#define LOGV(fmt, ...)                                                                             \
-    ::vector::native::detail::LogToAndroid(ANDROID_LOG_VERBOSE, LOG_TAG, "{}:{} ({}): " fmt,       \
-                                           __FILE_NAME__, __LINE__,                                \
-                                           __PRETTY_FUNCTION__ __VA_OPT__(, ) __VA_ARGS__)
 #define LOGD(fmt, ...)                                                                             \
     ::vector::native::detail::LogToAndroid(ANDROID_LOG_DEBUG, LOG_TAG, "{}:{} ({}): " fmt,         \
                                            __FILE_NAME__, __LINE__,                                \
                                            __PRETTY_FUNCTION__ __VA_OPT__(, ) __VA_ARGS__)
+#define LOGV(fmt, ...)                                                                             \
+    ::vector::native::detail::LogToAndroid(ANDROID_LOG_VERBOSE, LOG_TAG,                           \
+                                           fmt __VA_OPT__(, ) __VA_ARGS__)
 #else
 #define LOGV(...) ((void)0)
 #define LOGD(...) ((void)0)

zygisk/src/main/kotlin/org/matrix/vector/ParasiticManagerSystemHooker.kt

@@ -64,14 +64,12 @@ class ParasiticManagerSystemHooker : HandleSystemServerProcessHooker.Callback {
 
                 // Hook the resolution method to inject our redirection logic
                 VectorHookBuilder(resolveMethod).intercept { chain ->
-                    Utils.logD("inside resolveMethod, calling proceed")
-                    // 1. Execute the original resolution first
+                    // Execute the original resolution first
                     val result = chain.proceed()
 
                     val intent = chain.args[0] as? Intent ?: return@intercept result
-                    Utils.logD("proceed called, intent ${intent}")
 
-                    // Check if this intent is meant for the LSPosed Manager
+                    // Check if this intent is meant for the Vector Manager
                     if (!intent.hasCategory(BuildConfig.ManagerPackageName + ".LAUNCH_MANAGER"))
                         return@intercept result
 
@@ -88,7 +86,6 @@ class ParasiticManagerSystemHooker : HandleSystemServerProcessHooker.Callback {
                     if (originalActivityInfo.packageName != BuildConfig.InjectedPackageName)
                         return@intercept result
 
-                    Utils.logD("creat redirectedInfo")
                     // --- Redirection Logic ---
                     // We create a copy of the ActivityInfo to avoid polluting the system's cache.
                     val redirectedInfo =
@@ -110,7 +107,6 @@ class ParasiticManagerSystemHooker : HandleSystemServerProcessHooker.Callback {
                     // Notify the bridge service that we are about to start the manager
                     BridgeService.getService()?.preStartManager()
 
-                    Utils.logD("returning redirectedInfo ${redirectedInfo}")
                     redirectedInfo
                 }