Author: markt
Date: Fri Dec 5 15:30:18 2014
New Revision: 1643332
URL: http://svn.apache.org/r1643332
Log:
Follow up to r1643329
Replace System.getProperty("line.separator") with System.lineSeparator()
Modified:
tomcat/tc8.0.x/trunk/ (props changed)
tomcat/tc8.0.x/trunk/java/org/apache/catalina/connector/CoyoteWriter.java
tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/Constants.java
tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/SecurityListener.java
tomcat/tc8.0.x/trunk/java/org/apache/el/parser/ParseException.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/Constants.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/AntCompiler.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/ErrorDispatcher.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/JavacErrorDetail.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/BodyContentImpl.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java
tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java
tomcat/tc8.0.x/trunk/java/org/apache/juli/JdkLoggerFormatter.java
tomcat/tc8.0.x/trunk/java/org/apache/juli/OneLineFormatter.java
tomcat/tc8.0.x/trunk/java/org/apache/juli/VerbatimFormatter.java
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/buildutil/CheckEol.java
tomcat/tc8.0.x/trunk/test/org/apache/juli/TestClassLoaderLogManager.java
Propchange: tomcat/tc8.0.x/trunk/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Dec 5 15:30:18 2014
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1638720-1638725,1639653,1640010,1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642668,1642679,1642697,1642699,1643002,1643066,1643121,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1638720-1638725,1639653,1640010,1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642668,1642679,1642697,1642699,1643002,1643066,1643121,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/catalina/connector/CoyoteWriter.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/catalina/connector/CoyoteWriter.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/catalina/connector/CoyoteWriter.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/catalina/connector/CoyoteWriter.java
Fri Dec 5 15:30:18 2014
@@ -24,16 +24,12 @@ import java.io.PrintWriter;
*
* @author Remy Maucherat
*/
-public class CoyoteWriter
- extends PrintWriter {
+public class CoyoteWriter extends PrintWriter {
// -------------------------------------------------------------- Constants
- // No need for a do privileged block - every web app has permission to read
- // this by default
- private static final char[] LINE_SEP =
- System.getProperty("line.separator").toCharArray();
+ private static final char[] LINE_SEP =
System.lineSeparator().toCharArray();
// ----------------------------------------------------- Instance Variables
Modified: tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/Constants.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/Constants.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/Constants.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/Constants.java Fri
Dec 5 15:30:18 2014
@@ -20,6 +20,5 @@ public class Constants {
public static final String PACKAGE = "org.apache.catalina.security";
- public static final String LINE_SEP = System.getProperty("line.separator");
public static final String CRLF = "\r\n";
}
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/SecurityListener.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/SecurityListener.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
---
tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/SecurityListener.java
(original)
+++
tomcat/tc8.0.x/trunk/java/org/apache/catalina/security/SecurityListener.java
Fri Dec 5 15:30:18 2014
@@ -170,7 +170,7 @@ public class SecurityListener implements
}
}
if (umask == null) {
- if (Constants.CRLF.equals(Constants.LINE_SEP)) {
+ if (Constants.CRLF.equals(System.lineSeparator())) {
// Probably running on Windows so no umask
if (log.isDebugEnabled()) {
log.debug(sm.getString("SecurityListener.checkUmaskSkip"));
Modified: tomcat/tc8.0.x/trunk/java/org/apache/el/parser/ParseException.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/el/parser/ParseException.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/el/parser/ParseException.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/el/parser/ParseException.java Fri Dec
5 15:30:18 2014
@@ -89,7 +89,6 @@ public class ParseException extends Exce
private static String initialise(Token currentToken,
int[][] expectedTokenSequences,
String[] tokenImage) {
- String eol = System.getProperty("line.separator", "\n");
StringBuffer expected = new StringBuffer();
int maxSize = 0;
for (int i = 0; i < expectedTokenSequences.length; i++) {
@@ -102,7 +101,7 @@ public class ParseException extends Exce
if (expectedTokenSequences[i][expectedTokenSequences[i].length - 1] !=
0) {
expected.append("...");
}
- expected.append(eol).append(" ");
+ expected.append(System.lineSeparator()).append(" ");
}
String retval = "Encountered \"";
Token tok = currentToken.next;
@@ -119,22 +118,17 @@ public class ParseException extends Exce
tok = tok.next;
}
retval += "\" at line " + currentToken.next.beginLine + ", column " +
currentToken.next.beginColumn;
- retval += "." + eol;
+ retval += "." + System.lineSeparator();
if (expectedTokenSequences.length == 1) {
- retval += "Was expecting:" + eol + " ";
+ retval += "Was expecting:" + System.lineSeparator() + " ";
} else {
- retval += "Was expecting one of:" + eol + " ";
+ retval += "Was expecting one of:" + System.lineSeparator() + " ";
}
retval += expected.toString();
return retval;
}
/**
- * The end of line string for this machine.
- */
- protected String eol = System.getProperty("line.separator", "\n");
-
- /**
* Used to convert raw characters to their escaped version
* when these raw version cannot be used as part of an ASCII
* string literal.
Modified: tomcat/tc8.0.x/trunk/java/org/apache/jasper/Constants.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/Constants.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/Constants.java (original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/Constants.java Fri Dec 5
15:30:18 2014
@@ -93,11 +93,6 @@ public class Constants {
public static final int MAX_POOL_SIZE = 5;
/**
- * Platform specific new line sequence.
- */
- public static final String NEWLINE = System.getProperty("line.separator");
-
- /**
* The query parameter that causes the JSP engine to just
* pregenerated the servlet but not invoke it.
*/
Modified: tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/AntCompiler.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/AntCompiler.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/AntCompiler.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/AntCompiler.java Fri
Dec 5 15:30:18 2014
@@ -101,7 +101,7 @@ public class AntCompiler extends Compile
@Override
protected void log(String message) {
reportBuf.append(message);
- reportBuf.append(Constants.NEWLINE);
+ reportBuf.append(System.lineSeparator());
}
protected String getReport() {
@@ -244,7 +244,7 @@ public class AntCompiler extends Compile
// Stop capturing the System.err output for this thread
String errorCapture = SystemLogHandler.unsetThread();
if (errorCapture != null) {
- errorReport.append(Constants.NEWLINE);
+ errorReport.append(System.lineSeparator());
errorReport.append(errorCapture);
}
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
---
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java
(original)
+++
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java
Fri Dec 5 15:30:18 2014
@@ -17,7 +17,6 @@
package org.apache.jasper.compiler;
-import org.apache.jasper.Constants;
import org.apache.jasper.JasperException;
/**
@@ -77,28 +76,28 @@ class DefaultErrorHandler implements Err
args = new Object[] {
Integer.valueOf(details[i].getJspBeginLineNumber()),
details[i].getJspFileName() };
- buf.append(Constants.NEWLINE);
- buf.append(Constants.NEWLINE);
+ buf.append(System.lineSeparator());
+ buf.append(System.lineSeparator());
buf.append(Localizer.getMessage("jsp.error.single.line.number",
args));
- buf.append(Constants.NEWLINE);
+ buf.append(System.lineSeparator());
buf.append(details[i].getErrorMessage());
- buf.append(Constants.NEWLINE);
+ buf.append(System.lineSeparator());
buf.append(details[i].getJspExtract());
} else {
args = new Object[] {
Integer.valueOf(details[i].getJavaLineNumber()),
details[i].getJavaFileName() };
- buf.append(Constants.NEWLINE);
- buf.append(Constants.NEWLINE);
+ buf.append(System.lineSeparator());
+ buf.append(System.lineSeparator());
buf.append(Localizer.getMessage("jsp.error.java.line.number",
args));
- buf.append(Constants.NEWLINE);
+ buf.append(System.lineSeparator());
buf.append(details[i].getErrorMessage());
}
}
- buf.append(Constants.NEWLINE);
- buf.append(Constants.NEWLINE);
+ buf.append(System.lineSeparator());
+ buf.append(System.lineSeparator());
buf.append("Stacktrace:");
throw new JasperException(
Localizer.getMessage("jsp.error.unable.compile") + ": " + buf);
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/ErrorDispatcher.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/ErrorDispatcher.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/ErrorDispatcher.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/ErrorDispatcher.java
Fri Dec 5 15:30:18 2014
@@ -22,7 +22,6 @@ import java.io.StringReader;
import java.net.MalformedURLException;
import java.util.ArrayList;
-import org.apache.jasper.Constants;
import org.apache.jasper.JasperException;
import org.apache.jasper.JspCompilationContext;
import org.xml.sax.SAXException;
@@ -342,7 +341,7 @@ public class ErrorDispatcher {
// Ignore messages preceding first error
if (errMsgBuf != null) {
errMsgBuf.append(line);
- errMsgBuf.append(Constants.NEWLINE);
+ errMsgBuf.append(System.lineSeparator());
}
}
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/JavacErrorDetail.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/JavacErrorDetail.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/JavacErrorDetail.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/compiler/JavacErrorDetail.java
Fri Dec 5 15:30:18 2014
@@ -25,7 +25,6 @@ import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;
-import org.apache.jasper.Constants;
import org.apache.jasper.JspCompilationContext;
/**
@@ -126,7 +125,7 @@ public class JavacErrorDetail {
fragment.append(i+1);
fragment.append(": ");
fragment.append(jspLines[i]);
- fragment.append(Constants.NEWLINE);
+ fragment.append(System.lineSeparator());
}
jspExtract = fragment.toString();
}
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/BodyContentImpl.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/BodyContentImpl.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/BodyContentImpl.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/BodyContentImpl.java
Fri Dec 5 15:30:18 2014
@@ -39,8 +39,6 @@ import org.apache.jasper.Constants;
*/
public class BodyContentImpl extends BodyContent {
- private static final String LINE_SEPARATOR =
- System.getProperty("line.separator");
private static final boolean LIMIT_BUFFER =
Boolean.valueOf(System.getProperty("org.apache.jasper.runtime.BodyContentImpl.LIMIT_BUFFER",
"false")).booleanValue();
@@ -172,9 +170,9 @@ public class BodyContentImpl extends Bod
@Override
public void newLine() throws IOException {
if (writer != null) {
- writer.write(LINE_SEPARATOR);
+ writer.write(System.lineSeparator());
} else {
- write(LINE_SEPARATOR);
+ write(System.lineSeparator());
}
}
Modified: tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java Fri
Dec 5 15:30:18 2014
@@ -330,9 +330,6 @@ public class JspWriterImpl extends JspWr
}
- private static final String lineSeparator =
- System.getProperty("line.separator");
-
/**
* Write a line separator. The line separator string is defined by the
* system property <tt>line.separator</tt>, and is not necessarily a single
@@ -343,7 +340,7 @@ public class JspWriterImpl extends JspWr
@Override
public void newLine() throws IOException {
- write(lineSeparator);
+ write(System.lineSeparator());
}
Modified:
tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java
Fri Dec 5 15:30:18 2014
@@ -33,7 +33,6 @@ import javax.servlet.http.HttpServletReq
import javax.servlet.http.HttpServletResponse;
import javax.servlet.jsp.tagext.TagInfo;
-import org.apache.jasper.Constants;
import org.apache.jasper.JasperException;
import org.apache.jasper.JspCompilationContext;
import org.apache.jasper.Options;
@@ -567,9 +566,9 @@ public class JspServletWrapper {
if (options.getDisplaySourceFragment()) {
return new JasperException(Localizer.getMessage
("jsp.exception", detail.getJspFileName(),
- "" + jspLineNumber) + Constants.NEWLINE +
- Constants.NEWLINE + detail.getJspExtract() +
- Constants.NEWLINE + Constants.NEWLINE +
+ "" + jspLineNumber) + System.lineSeparator() +
+ System.lineSeparator() +
detail.getJspExtract() +
+ System.lineSeparator() +
System.lineSeparator() +
"Stacktrace:", ex);
}
Modified: tomcat/tc8.0.x/trunk/java/org/apache/juli/JdkLoggerFormatter.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/juli/JdkLoggerFormatter.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/juli/JdkLoggerFormatter.java (original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/juli/JdkLoggerFormatter.java Fri Dec
5 15:30:18 2014
@@ -14,14 +14,11 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
-
package org.apache.juli;
import java.util.logging.Formatter;
import java.util.logging.LogRecord;
-
-
/**
* A more compact formatter.
*
@@ -42,8 +39,6 @@ import java.util.logging.LogRecord;
*/
public class JdkLoggerFormatter extends Formatter {
- private static final String LINE_SEP =
System.getProperty("line.separator");
-
// values from JDK Level
public static final int LOG_LEVEL_TRACE = 400;
public static final int LOG_LEVEL_DEBUG = 500;
@@ -96,7 +91,7 @@ public class JdkLoggerFormatter extends
// Append stack trace if not null
if(t != null) {
- buf.append(LINE_SEP);
+ buf.append(System.lineSeparator());
java.io.StringWriter sw= new java.io.StringWriter(1024);
java.io.PrintWriter pw= new java.io.PrintWriter(sw);
@@ -105,9 +100,8 @@ public class JdkLoggerFormatter extends
buf.append(sw.toString());
}
- buf.append(LINE_SEP);
+ buf.append(System.lineSeparator());
// Print to the appropriate destination
return buf.toString();
}
-
}
Modified: tomcat/tc8.0.x/trunk/java/org/apache/juli/OneLineFormatter.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/juli/OneLineFormatter.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/juli/OneLineFormatter.java (original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/juli/OneLineFormatter.java Fri Dec 5
15:30:18 2014
@@ -38,8 +38,7 @@ import java.util.logging.LogRecord;
*/
public class OneLineFormatter extends Formatter {
- private static final String LINE_SEP =
System.getProperty("line.separator");
- private static final String ST_SEP = LINE_SEP + " ";
+ private static final String ST_SEP = System.lineSeparator() + " ";
private static final String UNKONWN_THREAD_NAME = "Unknown thread with ID
";
private static final Object threadMxBeanLock = new Object();
private static volatile ThreadMXBean threadMxBean = null;
@@ -136,7 +135,7 @@ public class OneLineFormatter extends Fo
}
// New line for next record
- sb.append(LINE_SEP);
+ sb.append(System.lineSeparator());
return sb.toString();
}
Modified: tomcat/tc8.0.x/trunk/java/org/apache/juli/VerbatimFormatter.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/juli/VerbatimFormatter.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/juli/VerbatimFormatter.java (original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/juli/VerbatimFormatter.java Fri Dec 5
15:30:18 2014
@@ -23,21 +23,19 @@ import java.util.logging.LogRecord;
/**
* Outputs the just the log message with no additional elements. Stack traces
* are not logged. Log messages are separated by
- * <code>System.getProperty("line.separator")</code>. This is intended for use
+ * <code>System.lineSeparator()</code>. This is intended for use
* by access logs and the like that need complete control over the output
* format.
*/
public class VerbatimFormatter extends Formatter {
- private static final String LINE_SEP =
System.getProperty("line.separator");
-
@Override
public String format(LogRecord record) {
// Timestamp
StringBuilder sb = new StringBuilder(record.getMessage());
// New line for next record
- sb.append(LINE_SEP);
+ sb.append(System.lineSeparator());
return sb.toString();
}
Modified: tomcat/tc8.0.x/trunk/java/org/apache/tomcat/buildutil/CheckEol.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/tomcat/buildutil/CheckEol.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/tomcat/buildutil/CheckEol.java
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/tomcat/buildutil/CheckEol.java Fri Dec
5 15:30:18 2014
@@ -41,8 +41,6 @@ import org.apache.tools.ant.types.FileSe
*/
public class CheckEol extends Task {
- private static final String eoln = System.getProperty("line.separator");
-
/** The files to be checked */
private final List<FileSet> filesets = new LinkedList<>();
@@ -65,9 +63,9 @@ public class CheckEol extends Task {
public void execute() throws BuildException {
Mode mode = null;
- if ("\n".equals(eoln)) {
+ if ("\n".equals(System.lineSeparator())) {
mode = Mode.LF;
- } else if ("\r\n".equals(eoln)) {
+ } else if ("\r\n".equals(System.lineSeparator())) {
mode = Mode.CRLF;
} else {
log("Line ends check skipped, because OS line ends setting is
neither LF nor CRLF.",
@@ -132,7 +130,7 @@ public class CheckEol extends Task {
@Override
public String toString() {
- return eoln + file + ": uses " + value + " on line " + line;
+ return System.lineSeparator() + file + ": uses " + value + " on
line " + line;
}
}
Modified:
tomcat/tc8.0.x/trunk/test/org/apache/juli/TestClassLoaderLogManager.java
URL:
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/test/org/apache/juli/TestClassLoaderLogManager.java?rev=1643332&r1=1643331&r2=1643332&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/test/org/apache/juli/TestClassLoaderLogManager.java
(original)
+++ tomcat/tc8.0.x/trunk/test/org/apache/juli/TestClassLoaderLogManager.java
Fri Dec 5 15:30:18 2014
@@ -37,11 +37,10 @@ public class TestClassLoaderLogManager {
Assert.assertEquals("${", logManager.replace("${"));
Assert.assertEquals("${undefinedproperty}",
logManager.replace("${undefinedproperty}"));
Assert.assertEquals(
- System.getProperty("line.separator") + File.pathSeparator +
File.separator,
+ System.lineSeparator() + File.pathSeparator + File.separator,
logManager.replace("${line.separator}${path.separator}${file.separator}"));
Assert.assertEquals(
- "foo" + File.separator + "bar" +
System.getProperty("line.separator")
- + File.pathSeparator + "baz",
+ "foo" + File.separator + "bar" + System.lineSeparator() +
File.pathSeparator + "baz",
logManager.replace("foo${file.separator}bar${line.separator}${path.separator}baz"));
// BZ 51249
Assert.assertEquals(
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
