Use Jvm.{level}() handlers instead of calling loggers directly, and unit tests cleanup esp use of @after, OpenHFT/Chronicle-Core#233

peter-lawrey · peter-lawrey · commit 5f30051f6345 · 2021-06-28T12:49:39.000+01:00
diff --git a/affinity/src/main/java/net/openhft/affinity/Affinity.java b/affinity/src/main/java/net/openhft/affinity/Affinity.java
@@ -33,8 +33,7 @@
  *
  * @author peter.lawrey
  */
-public enum Affinity {
-    ;
+public enum Affinity {;
     static final Logger LOGGER = LoggerFactory.getLogger(Affinity.class);
     @NotNull
     private static final IAffinity AFFINITY_IMPL;
diff --git a/affinity/src/main/java/net/openhft/affinity/LockCheck.java b/affinity/src/main/java/net/openhft/affinity/LockCheck.java
@@ -28,8 +28,7 @@
 /**
  * @author Rob Austin.
  */
-enum LockCheck {
-    ;
+enum LockCheck {;
 
     private static final Logger LOGGER = LoggerFactory.getLogger(LockCheck.class);
     private static final String OS = System.getProperty("os.name").toLowerCase();
diff --git a/affinity/src/main/java/net/openhft/affinity/impl/LinuxHelper.java b/affinity/src/main/java/net/openhft/affinity/impl/LinuxHelper.java
@@ -41,7 +41,7 @@ public class LinuxHelper {
                 ver = new VersionHelper(uname.getRealeaseVersion());
             }
         } catch (Throwable e) {
-            //logger.warn("Failed to determine Linux version: " + e);
+            //Jvm.warn().on(getClass(), "Failed to determine Linux version: " + e);
         }
 
         version = ver;

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public class LinuxHelper {`
`41`	`41`	`ver = new VersionHelper(uname.getRealeaseVersion());`
`42`	`42`	`}`
`43`	`43`	`} catch (Throwable e) {`
`44`		`- //logger.warn("Failed to determine Linux version: " + e);`
	`44`	`+ //Jvm.warn().on(getClass(), "Failed to determine Linux version: " + e);`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`version = ver;`