Changeset 35661 in osm for applications/editors

Timestamp:

2020-12-06T02:26:39+01:00 (4 years ago)

Author:

Don-vip

Message:

see #20205 - fix stupid bug + add Windows event logging

Location:

applications/editors/josm/plugins/jna/src/org/openstreetmap/josm/plugins/jna

Files:

• JNAPlugin.java (modified) (3 diffs)
• JosmNativeLogHandler.java (modified) (1 diff)
• mac/ID.java (modified) (1 diff)
• win (added)
• win/JosmWinNativeLogHandler.java (added)

applications/editors/josm/plugins/jna/src/org/openstreetmap/josm/plugins/jna/JNAPlugin.java

@@ -5 +5 @@
 import org.openstreetmap.josm.plugins.PluginInformation;
 import org.openstreetmap.josm.plugins.jna.mac.JosmMacNativeLogHandler;
+import org.openstreetmap.josm.plugins.jna.win.JosmWinNativeLogHandler;
 import org.openstreetmap.josm.tools.Logging;
 import org.openstreetmap.josm.tools.PlatformManager;
@@ -12 +13 @@
  */
 public class JNAPlugin extends Plugin {
+
     /**
      * Constructs a new {@code JNAPlugin}
@@ -20 +22 @@
         if (PlatformManager.isPlatformOsx()) {
             Logging.getLogger().addHandler(new JosmMacNativeLogHandler());
+        } else if (PlatformManager.isPlatformWindows()) {
+            Logging.getLogger().addHandler(new JosmWinNativeLogHandler());
         }
     }

applications/editors/josm/plugins/jna/src/org/openstreetmap/josm/plugins/jna/JosmNativeLogHandler.java

@@ -38 +38 @@
 
     private static boolean isLoggable(LogRecord record, Level level, BooleanProperty prop) {
-        return level.equals(record.getLevel()) && prop.isSet();
+        return level.equals(record.getLevel()) && prop.get();
     }
 

applications/editors/josm/plugins/jna/src/org/openstreetmap/josm/plugins/jna/mac/ID.java

@@ -3 +3 @@
 
 import com.sun.jna.Native; // NOSONAR
-import com.sun.jna.NativeLibrary; // NOSONAR
-import com.sun.jna.NativeLong; // NOSONAR
+import com.sun.jna.NativeLibrary;
+import com.sun.jna.NativeLong;
 
 /**