Merge pull request #474 from agateau/fix-duplicated-log-messages

fix duplicated log messages
agateau · Dec 11, 2024 · bb43832 · bb43832
2 parents 27dd0fe + 6957b4d
commit bb43832
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/android/src/main/com/agateau/pixelwheels/android/AndroidLauncher.java b/android/src/main/com/agateau/pixelwheels/android/AndroidLauncher.java
@@ -71,10 +71,15 @@ public void onWindowFocusChanged(boolean hasFocus) {
     }
 
     private void setupLogging(PwGame game) {
+        // Some log files end up with duplicated messages: looks like onCreate() can be called more
+        // than once. To avoid this, remove any existing printers.
+        NLog.clearPrinters();
+
         AndroidLogFileOpener opener = new AndroidLogFileOpener(this);
         LogFilePrinter printer =
                 new LogFilePrinter(Constants.LOG_FILENAME, Constants.LOG_MAX_SIZE, opener);
         NLog.addPrinter(printer);
+
         NLog.addPrinter(new AndroidNLogPrinter());
 
         AndroidLogExporter exporter = new AndroidLogExporter(this, printer);

diff --git a/core/src/com/agateau/utils/log/NLog.java b/core/src/com/agateau/utils/log/NLog.java
@@ -64,6 +64,10 @@ public static void backtrace() {
         }
     }
 
+    public static void clearPrinters() {
+        sPrinters.clear();
+    }
+
     public static void addPrinter(Printer printer) {
         sPrinters.add(printer);
     }