Author: ceki
Date: Fri Oct  3 19:09:11 2008
New Revision: 1175

Modified:
   slf4j/trunk/integration/build.xml
   slf4j/trunk/slf4j-api/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
   slf4j/trunk/slf4j-jcl/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
   slf4j/trunk/slf4j-jdk14/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
   slf4j/trunk/slf4j-nop/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
   slf4j/trunk/slf4j-simple/src/main/java/org/slf4j/impl/StaticLoggerBinder.java

Log:
- minor clean up

Modified: slf4j/trunk/integration/build.xml
==============================================================================
--- slf4j/trunk/integration/build.xml   (original)
+++ slf4j/trunk/integration/build.xml   Fri Oct  3 19:09:11 2008
@@ -4,7 +4,7 @@
        <echo message="runtime classpath: ${runtime_classpath}" />
        <echo message="test classpath:    ${test_classpath}" />
        <echo message="plugin classpath:  ${plugin_classpath}" />
-       
+
        <path id="path150">
                <pathelement location="xtarget/classes/" />
                <pathelement location="target/test-classes/" />
@@ -27,14 +27,8 @@
 
                <junit printsummary="yes" fork="no" haltonfailure="yes">
                        <classpath refid="path150" />
-
                        <formatter type="plain" />
-                       <batchtest fork="yes" todir="target/unit-reports">
-                               <fileset dir="src/test/java">
-                                       <include name="**/VersionTest.java" />
-                                       <exclude name="**/XTests.java" />
-                               </fileset>
-                       </batchtest>
+                       <test fork="yes" todir="target/unit-reports" 
name="org.slf4j.VersionTest" />
                </junit>
 
        </target>

Modified: 
slf4j/trunk/slf4j-api/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
==============================================================================
--- slf4j/trunk/slf4j-api/src/main/java/org/slf4j/impl/StaticLoggerBinder.java  
(original)
+++ slf4j/trunk/slf4j-api/src/main/java/org/slf4j/impl/StaticLoggerBinder.java  
Fri Oct  3 19:09:11 2008
@@ -45,7 +45,7 @@
   
   /**
    * Version tag used to check compatibility. The value of this field is
-   * modified in each release. 
+   * modified with each release. 
    */
   // to avoid constant folding by the compiler, VERSION field should *not* be 
final
   public static String VERSION = "1.5.4-SNAPSHOT";  // !final

Modified: 
slf4j/trunk/slf4j-jcl/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
==============================================================================
--- slf4j/trunk/slf4j-jcl/src/main/java/org/slf4j/impl/StaticLoggerBinder.java  
(original)
+++ slf4j/trunk/slf4j-jcl/src/main/java/org/slf4j/impl/StaticLoggerBinder.java  
Fri Oct  3 19:09:11 2008
@@ -52,9 +52,10 @@
 
   /**
    * Version tag used to check compatibility. The value of this field is
-   * modified in each release. 
-   */
-  public static final String VERSION = "1.5.4-SNAPSHOT";
+   * modified with each release. 
+   */ 
+  //to avoid constant folding by the compiler, VERSION field should *not* be 
final
+  public static String VERSION = "1.5.4-SNAPSHOT";  
   
   // Binding specific code:
   private static final String loggerFactoryClassStr = JCLLoggerFactory.class

Modified: 
slf4j/trunk/slf4j-jdk14/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
==============================================================================
--- 
slf4j/trunk/slf4j-jdk14/src/main/java/org/slf4j/impl/StaticLoggerBinder.java    
    (original)
+++ 
slf4j/trunk/slf4j-jdk14/src/main/java/org/slf4j/impl/StaticLoggerBinder.java    
    Fri Oct  3 19:09:11 2008
@@ -49,11 +49,13 @@
    * The unique instance of this class.
    */
   public static final StaticLoggerBinder SINGLETON = new StaticLoggerBinder();
+
   /**
    * Version tag used to check compatibility. The value of this field is
-   * modified in each release. 
-   */
-  public static final String VERSION = "1.5.4-SNAPSHOT";
+   * modified with each release. 
+   */ 
+  //to avoid constant folding by the compiler, VERSION field should *not* be 
final
+  public static String VERSION = "1.5.4-SNAPSHOT";  
   
   private static final String loggerFactoryClassStr = 
org.slf4j.impl.JDK14LoggerFactory.class.getName();
 

Modified: 
slf4j/trunk/slf4j-nop/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
==============================================================================
--- slf4j/trunk/slf4j-nop/src/main/java/org/slf4j/impl/StaticLoggerBinder.java  
(original)
+++ slf4j/trunk/slf4j-nop/src/main/java/org/slf4j/impl/StaticLoggerBinder.java  
Fri Oct  3 19:09:11 2008
@@ -52,9 +52,10 @@
   
   /**
    * Version tag used to check compatibility. The value of this field is
-   * modified in each release. 
-   */
-  public static final String VERSION = "1.5.4-SNAPSHOT";
+   * modified with each release. 
+   */ 
+  //to avoid constant folding by the compiler, VERSION field should *not* be 
final
+  public static String VERSION = "1.5.4-SNAPSHOT";  
   
   private static final String loggerFactoryClassStr = 
NOPLoggerFactory.class.getName();
 

Modified: 
slf4j/trunk/slf4j-simple/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
==============================================================================
--- 
slf4j/trunk/slf4j-simple/src/main/java/org/slf4j/impl/StaticLoggerBinder.java   
    (original)
+++ 
slf4j/trunk/slf4j-simple/src/main/java/org/slf4j/impl/StaticLoggerBinder.java   
    Fri Oct  3 19:09:11 2008
@@ -41,11 +41,13 @@
    * The unique instance of this class.
    */
   public static final StaticLoggerBinder SINGLETON = new StaticLoggerBinder();
+  
   /**
    * Version tag used to check compatibility. The value of this field is
-   * modified in each release. 
-   */
-  public static final String VERSION = "1.5.4-SNAPSHOT";
+   * modified with each release. 
+   */ 
+  //to avoid constant folding by the compiler, VERSION field should *not* be 
final
+  public static String VERSION = "1.5.4-SNAPSHOT";  
   
   private static final String loggerFactoryClassStr = 
SimpleLoggerFactory.class.getName();
 
_______________________________________________
dev mailing list
dev@slf4j.org
http://www.slf4j.org/mailman/listinfo/dev

Reply via email to