Author: markt Date: Wed Nov 8 19:05:09 2006 New Revision: 472751 URL: http://svn.apache.org/viewvc?view=rev&rev=472751 Log: Fix bug 37509. My reading of the Properties JavaDoc is that property values can't start with white space. Given that values can't start with white space and having white space on the end is valid then I can't see what the purpose of this trim() is.
Modified: tomcat/connectors/trunk/juli/src/java/org/apache/juli/ClassLoaderLogManager.java tomcat/container/tc5.5.x/webapps/docs/changelog.xml Modified: tomcat/connectors/trunk/juli/src/java/org/apache/juli/ClassLoaderLogManager.java URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/juli/src/java/org/apache/juli/ClassLoaderLogManager.java?view=diff&rev=472751&r1=472750&r2=472751 ============================================================================== --- tomcat/connectors/trunk/juli/src/java/org/apache/juli/ClassLoaderLogManager.java (original) +++ tomcat/connectors/trunk/juli/src/java/org/apache/juli/ClassLoaderLogManager.java Wed Nov 8 19:05:09 2006 @@ -452,7 +452,7 @@ * @return the modified string */ protected String replace(String str) { - String result = str.trim(); + String result = str; if (result.startsWith("${")) { int pos = result.indexOf('}'); if (pos != -1) { Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?view=diff&rev=472751&r1=472750&r2=472751 ============================================================================== --- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original) +++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Wed Nov 8 19:05:09 2006 @@ -63,6 +63,10 @@ stop to fail. Based on a fix suggested by Michael Vorburger. (markt) </fix> <fix> + <bug>37509</bug>: Do not remove whitespace from the end of values + defined in logging.properties files. (markt) + </fix> + <fix> <bug>37070</bug>: Update mbean name documentation to include the StandardWrapper. (markt) </fix> --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]