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]

Reply via email to