Author: markt
Date: Fri Dec 5 14:55:38 2014
New Revision: 1643309
URL: http://svn.apache.org/r1643309
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57308
Avoid unnecessary use of System.getProperty("file.separator")
Modified:
tomcat/trunk/java/javax/el/ExpressionFactory.java
tomcat/trunk/test/org/apache/juli/TestClassLoaderLogManager.java
Modified: tomcat/trunk/java/javax/el/ExpressionFactory.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/javax/el/ExpressionFactory.java?rev=1643309&r1=1643308&r2=1643309&view=diff
==============================================================================
--- tomcat/trunk/java/javax/el/ExpressionFactory.java (original)
+++ tomcat/trunk/java/javax/el/ExpressionFactory.java Fri Dec 5 14:55:38 2014
@@ -53,7 +53,6 @@ public abstract class ExpressionFactory
private static final String PROPERTY_NAME = "javax.el.ExpressionFactory";
- private static final String SEP;
private static final String PROPERTY_FILE;
private static final CacheValue nullTcclFactory = new CacheValue();
@@ -62,29 +61,19 @@ public abstract class ExpressionFactory
static {
if (IS_SECURITY_ENABLED) {
- SEP = AccessController.doPrivileged(
- new PrivilegedAction<String>(){
- @Override
- public String run() {
- return System.getProperty("file.separator");
- }
-
- }
- );
PROPERTY_FILE = AccessController.doPrivileged(
new PrivilegedAction<String>(){
@Override
public String run() {
- return System.getProperty("java.home") + SEP +
- "lib" + SEP + "el.properties";
+ return System.getProperty("java.home") +
File.separator +
+ "lib" + File.separator + "el.properties";
}
}
);
} else {
- SEP = System.getProperty("file.separator");
- PROPERTY_FILE = System.getProperty("java.home") + SEP + "lib" +
- SEP + "el.properties";
+ PROPERTY_FILE = System.getProperty("java.home") + File.separator +
"lib" +
+ File.separator + "el.properties";
}
}
Modified: tomcat/trunk/test/org/apache/juli/TestClassLoaderLogManager.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/juli/TestClassLoaderLogManager.java?rev=1643309&r1=1643308&r2=1643309&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/juli/TestClassLoaderLogManager.java (original)
+++ tomcat/trunk/test/org/apache/juli/TestClassLoaderLogManager.java Fri Dec 5
14:55:38 2014
@@ -16,6 +16,7 @@
*/
package org.apache.juli;
+import java.io.File;
import java.util.Collections;
import java.util.Random;
import java.util.logging.LogManager;
@@ -39,22 +40,22 @@ public class TestClassLoaderLogManager {
Assert.assertEquals(
System.getProperty("line.separator")
+ System.getProperty("path.separator")
- + System.getProperty("file.separator"),
+ + File.separator,
logManager
.replace("${line.separator}${path.separator}${file.separator}"));
Assert.assertEquals(
- "foo" + System.getProperty("file.separator") + "bar"
+ "foo" + File.separator + "bar"
+ System.getProperty("line.separator")
+ System.getProperty("path.separator") + "baz",
logManager
.replace("foo${file.separator}bar${line.separator}${path.separator}baz"));
// BZ 51249
Assert.assertEquals(
- "%{file.separator}" + System.getProperty("file.separator"),
+ "%{file.separator}" + File.separator,
logManager.replace("%{file.separator}${file.separator}"));
Assert.assertEquals(
- System.getProperty("file.separator") + "${undefinedproperty}"
- + System.getProperty("file.separator"),
+ File.separator + "${undefinedproperty}"
+ + File.separator,
logManager
.replace("${file.separator}${undefinedproperty}${file.separator}"));
Assert.assertEquals("${}" + System.getProperty("path.separator"),
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
