Changed the default line separator to \r\n to comply with RFC 4180 (SANDBOX-163)

git-svn-id: https://svn.apache.org/repos/asf/commons/sandbox/csv/trunk@1201115 13f79535-47bb-0310-9956-ffa450edef68

Author: ebourg

src/main/java/org/apache/commons/csv/CSVFormat.java

@@ -34,7 +34,7 @@ public class CSVFormat implements Cloneable, Serializable {
     private boolean trailingSpacesIgnored = true;
     private boolean unicodeEscapesInterpreted = false;
     private boolean emptyLinesIgnored = true;
-    private String lineSeparator = "\n";
+    private String lineSeparator = "\r\n";
 
 
     /**
@@ -45,7 +45,7 @@ public class CSVFormat implements Cloneable, Serializable {
      */
     public static final char DISABLED = '\ufffe';
 
-    /** Standard comma separated format. */
+    /** Standard comma separated format as defined by RFC 4180. */
     public static final CSVFormat DEFAULT = new CSVFormat(',', '"', DISABLED, DISABLED, true, true, false, true);
 
     /** Excel file format (using a comma as the value delimiter). */

src/test/java/org/apache/commons/csv/CSVFormatTest.java

@@ -37,8 +37,8 @@ public void testImmutalibity() {
         assertEquals('!', format.getDelimiter());
         assertEquals('!', format.getEncapsulator());
         assertEquals('!', format.getCommentStart());
-        assertEquals("\n", format.getLineSeparator());
         assertEquals('!', format.getEscape());
+        assertEquals("\r\n", format.getLineSeparator());
 
         assertEquals(true, format.isLeadingSpacesIgnored());
         assertEquals(true, format.isTrailingSpacesIgnored());

src/test/java/org/apache/commons/csv/CSVPrinterTest.java

@@ -28,7 +28,7 @@
  */
 public class CSVPrinterTest extends TestCase {
 
-    String lineSeparator = "\n";
+    String lineSeparator = "\r\n";
 
     public void testPrinter1() throws IOException {
         StringWriter sw = new StringWriter();