pvdung / jcsv

Automatically exported from code.google.com/p/jcsv
0 stars 0 forks source link

suggestion - add newLine to CSVStrategy #6

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. There is webapp running on unix that is relying on system property 
"line.separator", so cannot change that. And requirement is that jcsv is 
generating CRLF rows.

What is the expected output? What do you see instead?
What i'm using is:
Changed in com.googlecode.jcsv.CSVStrategy

  private final String newLine;
  public String getNewLine() {...}
  public CSVStrategy-constructor modified to accept String newLine parameter
  ...DEFAULT and UK_DEFAULT strategies updated to read System.getProperty("line.separator")

Changed in com.googlecode.jcsv.writer.internal.CSVWriterImpl
  // method
  public void write(E e) throws IOException {
  ...
  sb.append(strategy.getNewLine());
  // replaced the old line: sb.append(System.getProperty("line.separator"));
  ...
  }

What version of the product are you using? On what operating system?
jcsv-1.4.0, on Linux app1 2.6.16.60-0.21-xen, java version "1.6.0_34" 64-Bit 
Server

Please provide any additional information below.
No rush, it's working fine with my modifications just trying to share idea.

Original issue reported on code.google.com by marian.s...@gmail.com on 4 Jun 2013 at 12:59