Author: rjung
Date: Wed Dec  5 10:59:10 2012
New Revision: 1417353

URL: http://svn.apache.org/viewvc?rev=1417353&view=rev
Log:
- Move default setting to the right place.

- Don't overwrite fileDateFormat=""
  configuration by hard code default.

- remove obsolete comment warning.

Modified:
    tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java

Modified: tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java?rev=1417353&r1=1417352&r2=1417353&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java Wed Dec  5 
10:59:10 2012
@@ -519,9 +519,9 @@ public class AccessLogValve extends Valv
     protected String conditionIf = null;
 
     /**
-     * Date format to place in log file name. Use at your own risk!
+     * Date format to place in log file name.
      */
-    protected String fileDateFormat = null;
+    protected String fileDateFormat = ".yyyy-MM-dd";
 
 
     /**
@@ -1232,8 +1232,8 @@ public class AccessLogValve extends Valv
 
         // Initialize the Date formatters
         String format = getFileDateFormat();
-        if (format == null || format.length() == 0) {
-            format = ".yyyy-MM-dd";
+        if (format == null) {
+            format = "";
             setFileDateFormat(format);
         }
         fileDateFormatter = new SimpleDateFormat(format, Locale.US);



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to