Use SLF4J to allow easily routing logs to multiple logging frameworks (#33)

Use SLF4J to allow easily routing logs to multiple logging frameworks
Co-authored-by: Andre Brait <[email protected]>

Author: cowtowncoder

pom.xml

@@ -43,7 +43,7 @@ JUG supports all 3 official UUID generation methods.
   </prerequisites>
   <properties>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-    <log4j.version>1.2.17</log4j.version>
+    <slf4j.version>1.7.29</slf4j.version>
   </properties>
   <licenses>
     <license>
@@ -60,13 +60,19 @@ JUG supports all 3 official UUID generation methods.
  <!-- Dependency information -->
 
   <dependencies>
-    <dependency> <!-- log4j is optional, but needed for compilation -->
-      <groupId>log4j</groupId>
-      <artifactId>log4j</artifactId>
-      <version>${log4j.version}</version>
-      <scope>provided</scope>
+    <dependency>
+      <groupId>org.slf4j</groupId>
+      <artifactId>slf4j-api</artifactId>
+      <version>${slf4j.version}</version>
+    </dependency>
+    <dependency>
+        <groupId>org.slf4j</groupId>
+        <artifactId>slf4j-simple</artifactId>
+        <version>${slf4j.version}</version>
+        <scope>runtime</scope>
+        <optional>true</optional>
     </dependency>
-    <!-- and for testing, JUnit is needed -->
+    <!-- For testing, JUnit is needed -->
     <dependency>
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
@@ -90,6 +96,7 @@ JUG supports all 3 official UUID generation methods.
         <plugin><!-- plug-in to attach source bundle in repo -->
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-source-plugin</artifactId>
+          <version>${version.plugin.source}</version>
            <executions>
             <execution>
               <id>attach-sources</id>
@@ -135,12 +142,10 @@ JUG supports all 3 official UUID generation methods.
               <Import-Package>
                 com.fasterxml.uuid;version="[${project.version},${project.version}]",
                 com.fasterxml.uuid.ext;version="[${project.version},${project.version}]",
-                com.fasterxml.uuid.impl;version="[${project.version},${project.version}]"
+                com.fasterxml.uuid.impl;version="[${project.version},${project.version}]",
+                org.slf4j;version="[${slf4j.version},)"
               </Import-Package>
               <Private-Package />
-              <DynamicImport-Package>
-                org.apache.log4j;version="[${log4j.version},1.3)"
-              </DynamicImport-Package>
             </instructions>
           </configuration>
        </plugin>

src/main/java/com/fasterxml/uuid/Logger.java

@@ -20,6 +20,9 @@
 
 import com.fasterxml.uuid.impl.UUIDUtil;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * UUIDTimer produces the time stamps required for time-based UUIDs.
  * It works as outlined in the UUID specification, with following
@@ -72,6 +75,9 @@
  */
 public class UUIDTimer
 {
+
+    private static final Logger logger = LoggerFactory.getLogger(UUIDTimer.class);
+    
     // // // Constants
 
     /**
@@ -241,7 +247,7 @@ public synchronized long getTimestamp()
          * independent of whether we can use it:
          */
         if (systime < _lastSystemTimestamp) {
-            Logger.logWarning("System time going backwards! (got value "+systime+", last "+_lastSystemTimestamp);
+            logger.warn("System time going backwards! (got value {}, last {}", systime, _lastSystemTimestamp);
             // Let's write it down, still
             _lastSystemTimestamp = systime;
         }
@@ -261,7 +267,7 @@ public synchronized long getTimestamp()
                 long origTime = systime;
                 systime = _lastUsedTimestamp + 1L;
 
-                Logger.logWarning("Timestamp over-run: need to reinitialize random sequence");
+                logger.warn("Timestamp over-run: need to reinitialize random sequence");
 
                 /* Clock counter is now at exactly the multiplier; no use
                  * just anding its value. So, we better get some random
@@ -382,7 +388,7 @@ protected static void slowDown(long startTime, long actDiff)
         } else {
             delay = 5L;
         }
-        Logger.logWarning("Need to wait for "+delay+" milliseconds; virtual clock advanced too far in the future");
+        logger.warn("Need to wait for {} milliseconds; virtual clock advanced too far in the future", delay);
         long waitUntil = startTime + delay;
         int counter = 0;
         do {