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:30:04 UTC

svn commit: r1397078 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java

Author: ggregory
Date: Thu Oct 11 14:30:04 2012
New Revision: 1397078

URL: http://svn.apache.org/viewvc?rev=1397078&view=rev
Log:
Replace magic char with constant TAB.

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=1397078&r1=1397077&r2=1397078&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:30:04 2012
@@ -29,6 +29,8 @@ import java.io.StringWriter;
  */
 public class CSVFormat implements Serializable {
 
+    private static final char TAB = '\t';
+
     private static final char DOUBLE_QUOTE = '"';
 
     private static final char COMMA = ',';
@@ -113,7 +115,7 @@ public class CSVFormat implements Serial
     /** Tab-delimited format, with quote; leading and trailing spaces ignored. */
     public static final CSVFormat TDF =
             PRISTINE
-            .withDelimiter('\t')
+            .withDelimiter(TAB)
             .withEncapsulator(DOUBLE_QUOTE)
             .withSurroundingSpacesIgnored(true)
             .withEmptyLinesIgnored(true)
@@ -129,7 +131,7 @@ public class CSVFormat implements Serial
      */
     public static final CSVFormat MYSQL =
             PRISTINE
-            .withDelimiter('\t')
+            .withDelimiter(TAB)
             .withEscape('\\')
             .withLineSeparator("\n");
 



Re: svn commit: r1397078 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java

Posted by Gary Gregory <ga...@gmail.com>.
On Oct 11, 2012, at 21:35, sebb <se...@gmail.com> wrote:

> On 11 October 2012 15:30,  <gg...@apache.org> wrote:
>> Author: ggregory
>> Date: Thu Oct 11 14:30:04 2012
>> New Revision: 1397078
>>
>> URL: http://svn.apache.org/viewvc?rev=1397078&view=rev
>> Log:
>> Replace magic char with constant TAB.
>>
>> 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=1397078&r1=1397077&r2=1397078&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:30:04 2012
>> @@ -29,6 +29,8 @@ import java.io.StringWriter;
>>  */
>> public class CSVFormat implements Serializable {
>>
>> +    private static final char TAB = '\t';
>> +
>>     private static final char DOUBLE_QUOTE = '"';
>>
>>     private static final char COMMA = ',';
>> @@ -113,7 +115,7 @@ public class CSVFormat implements Serial
>>     /** Tab-delimited format, with quote; leading and trailing spaces ignored. */
>>     public static final CSVFormat TDF =
>>             PRISTINE
>> -            .withDelimiter('\t')
>> +            .withDelimiter(TAB)
>>             .withEncapsulator(DOUBLE_QUOTE)
>>             .withSurroundingSpacesIgnored(true)
>>             .withEmptyLinesIgnored(true)
>> @@ -129,7 +131,7 @@ public class CSVFormat implements Serial
>>      */
>>     public static final CSVFormat MYSQL =
>>             PRISTINE
>> -            .withDelimiter('\t')
>> +            .withDelimiter(TAB)
>>             .withEscape('\\')
>>             .withLineSeparator("\n");
>
> What about these magic chars?

I need more hats and rabbits!

Will do later unless someone else gets to it.

G

>
>>
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: svn commit: r1397078 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVFormat.java

Posted by sebb <se...@gmail.com>.
On 11 October 2012 15:30,  <gg...@apache.org> wrote:
> Author: ggregory
> Date: Thu Oct 11 14:30:04 2012
> New Revision: 1397078
>
> URL: http://svn.apache.org/viewvc?rev=1397078&view=rev
> Log:
> Replace magic char with constant TAB.
>
> 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=1397078&r1=1397077&r2=1397078&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:30:04 2012
> @@ -29,6 +29,8 @@ import java.io.StringWriter;
>   */
>  public class CSVFormat implements Serializable {
>
> +    private static final char TAB = '\t';
> +
>      private static final char DOUBLE_QUOTE = '"';
>
>      private static final char COMMA = ',';
> @@ -113,7 +115,7 @@ public class CSVFormat implements Serial
>      /** Tab-delimited format, with quote; leading and trailing spaces ignored. */
>      public static final CSVFormat TDF =
>              PRISTINE
> -            .withDelimiter('\t')
> +            .withDelimiter(TAB)
>              .withEncapsulator(DOUBLE_QUOTE)
>              .withSurroundingSpacesIgnored(true)
>              .withEmptyLinesIgnored(true)
> @@ -129,7 +131,7 @@ public class CSVFormat implements Serial
>       */
>      public static final CSVFormat MYSQL =
>              PRISTINE
> -            .withDelimiter('\t')
> +            .withDelimiter(TAB)
>              .withEscape('\\')
>              .withLineSeparator("\n");

What about these magic chars?

>
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org