You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2012/10/11 16:27:56 UTC
svn commit: r1397074 -
/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java
Author: ggregory
Date: Thu Oct 11 14:27:56 2012
New Revision: 1397074
URL: http://svn.apache.org/viewvc?rev=1397074&view=rev
Log:
Replace magic char with constant COMMA.
Modified:
commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java
Modified: commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java?rev=1397074&r1=1397073&r2=1397074&view=diff
==============================================================================
--- commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java (original)
+++ commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java Thu Oct 11 14:27:56 2012
@@ -29,6 +29,8 @@ import java.io.StringWriter;
*/
public class CSVFormat implements Serializable {
+ private static final char COMMA = ',';
+
private static final long serialVersionUID = 1L;
/** According to RFC 4180, line breaks are delimited by CRLF */
@@ -70,7 +72,7 @@ public class CSVFormat implements Serial
*/
public static final CSVFormat DEFAULT =
PRISTINE.
- withDelimiter(',')
+ withDelimiter(COMMA)
.withEncapsulator('"')
.withEmptyLinesIgnored(true)
.withLineSeparator(CRLF);
@@ -86,7 +88,7 @@ public class CSVFormat implements Serial
*/
public static final CSVFormat RFC4180 =
PRISTINE.
- withDelimiter(',')
+ withDelimiter(COMMA)
.withEncapsulator('"')
.withLineSeparator(CRLF);
@@ -102,7 +104,7 @@ public class CSVFormat implements Serial
*/
public static final CSVFormat EXCEL =
PRISTINE
- .withDelimiter(',')
+ .withDelimiter(COMMA)
.withEncapsulator('"')
.withLineSeparator(CRLF);