commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From brit...@apache.org
Subject svn commit: r1617070 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVPrinter.java
Date Sun, 10 Aug 2014 08:54:34 GMT
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) {



Mime
View raw message