svn commit: r1571017 - in /jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging: FmtLog.java LogCmd.java

[an...@apache.org]

Author: andy
Date: Sun Feb 23 14:07:47 2014
New Revision: 1571017

URL: http://svn.apache.org/r1571017
Log:
Tidy up.

Removed:
    jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging/LogCmd.java
Modified:
    jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging/FmtLog.java

Modified: jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging/FmtLog.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging/FmtLog.java?rev=1571017&r1=1571016&r2=1571017&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging/FmtLog.java (original)
+++ jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/logging/FmtLog.java Sun Feb 23 14:07:47 2014
@@ -27,9 +27,11 @@ import org.slf4j.Logger ;
  *  but they do delay forming strings for output
  *  until it is know that a log message is actually
  *  required by level setting. 
+ *  
+ *  An odd effect is order of the arguments - vararg arguments must be last
+ *  so the order is Logger/Format/Thorwable?/args
  */
 public class FmtLog {
-    // -- Delayed argument formatting.
     /* Log at 'trace' level. */
     public static void trace(Logger log, String fmt, Object...args) {
         if ( log.isTraceEnabled() )
@@ -42,7 +44,6 @@ public class FmtLog {
             log.trace(format(fmt, args), th) ;
     }
 
-
     /* Log at 'debug' level */
     public static void debug(Logger log, String fmt, Object...args) {
         if ( log.isDebugEnabled() )
@@ -100,7 +101,5 @@ public class FmtLog {
             return fmt+" "+args ;
         }
     }
-    // ----
-
 }