You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by br...@apache.org on 2014/08/10 10:54:34 UTC
svn commit: r1617070 -
/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java
Author: britter
Date: Sun Aug 10 08:54:34 2014
New Revision: 1617070
URL: http://svn.apache.org/r1617070
Log:
Remove commented out code and add TODO for obscure code
Modified:
commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java
Modified: commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java?rev=1617070&r1=1617069&r2=1617070&view=diff
==============================================================================
--- commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java (original)
+++ commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java Sun Aug 10 08:54:34 2014
@@ -209,10 +209,9 @@ public final class CSVPrinter implements
} else {
char c = value.charAt(pos);
- // Hmmm, where did this rule come from?
+ // TODO where did this rule come from?
if (newRecord && (c < '0' || (c > '9' && c < 'A') || (c > 'Z' && c < 'a') || (c > 'z'))) {
quote = true;
- // } else if (c == ' ' || c == '\f' || c == '\t') {
} else if (c <= COMMENT) {
// Some other chars at the start of a value caused the parser to fail, so for now
// encapsulate if we start in anything less than '#'. We are being conservative
@@ -231,7 +230,6 @@ public final class CSVPrinter implements
if (!quote) {
pos = end - 1;
c = value.charAt(pos);
- // if (c == ' ' || c == '\f' || c == '\t') {
// Some other chars at the end caused the parser to fail, so for now
// encapsulate if we end in anything less than ' '
if (c <= SP) {