Fix debug logs (kstenerud#579)

xuezhulian · Oct 11, 2024 · c97aa04 · c97aa04
1 parent 00ca0b0
commit c97aa04
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/Sources/KSCrashRecording/KSCrashReportStore.m b/Sources/KSCrashRecording/KSCrashReportStore.m
@@ -88,7 +88,7 @@ - (void)sendAllReportsWithCompletion:(KSCrashReportFilterCompletion)onCompletion
     __weak __typeof(self) weakSelf = self;
     [self sendReports:reports
          onCompletion:^(NSArray *filteredReports, NSError *error) {
-             KSLOG_DEBUG(@"Process finished with completion: %d", completed);
+             KSLOG_DEBUG(@"Process finished");
              if (error != nil) {
                  KSLOG_ERROR(@"Failed to send reports: %@", error);
              }

diff --git a/Sources/KSCrashRecordingCore/KSCrashMonitor.c b/Sources/KSCrashRecordingCore/KSCrashMonitor.c
@@ -47,6 +47,15 @@ typedef struct {
 
 #define INITIAL_MONITOR_CAPACITY 15
 
+#pragma mark - Helpers
+
+__attribute__((unused))  // Suppress unused function warnings, especially in release builds.
+static inline const char *
+getMonitorNameForLogging(const KSCrashMonitorAPI *api)
+{
+    return kscm_getMonitorId(api) ?: "Unknown";
+}
+
 // ============================================================================
 #pragma mark - Globals -
 // ============================================================================
@@ -96,13 +105,13 @@ static void removeMonitor(MonitorList *list, const KSCrashMonitorAPI *api)
             list->count--;
             list->apis[list->count] = NULL;
 
-            KSLOG_DEBUG("Monitor %s removed from the list.", getMonitorNameForLogging(func));
+            KSLOG_DEBUG("Monitor %s removed from the list.", getMonitorNameForLogging(api));
             break;
         }
     }
 
     if (!found) {
-        KSLOG_DEBUG("Monitor %s not found in the list. No removal performed.", getMonitorNameForLogging(func));
+        KSLOG_DEBUG("Monitor %s not found in the list. No removal performed.", getMonitorNameForLogging(api));
     }
 }
 
@@ -126,15 +135,6 @@ void kscm_resetState(void)
     g_onExceptionEvent = NULL;
 }
 
-#pragma mark - Helpers
-
-__attribute__((unused))  // Suppress unused function warnings, especially in release builds.
-static inline const char *
-getMonitorNameForLogging(const KSCrashMonitorAPI *api)
-{
-    return kscm_getMonitorId(api) ?: "Unknown";
-}
-
 // ============================================================================
 #pragma mark - API -
 // ============================================================================