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