search

fubar-coder / beanio

Automatically exported from code.google.com/p/beanio
Apache License 2.0
0 stars 0 forks source link

recordTerminator needs to be defined as a String to support CR LF #77

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In org.beanio.builder.DelimitedParserBuilder I need to set recordTerminator to 
( CR LF ) to support dos format.

Original issue reported on code.google.com by stephen....@gmail.com on 25 May 2013 at 11:00

GoogleCodeExporter commented 9 years ago 
Are you running your code on a Windows machine?  If so, that should be the 
default.

Original comment by kevin.s...@gmail.com on 29 May 2013 at 3:02

GoogleCodeExporter commented 9 years ago 
No, I am running on linux. I have to build a file for windows.

Original comment by stephen....@gmail.com on 29 May 2013 at 11:33

GoogleCodeExporter commented 9 years ago 
Looks like I had just interchanged the CRLF (i.e. LFCR) in the code.  Try the 
attached 2.0.7 snapshot JAR.

Thanks,
Kevin

Original comment by kevin.s...@gmail.com on 30 May 2013 at 2:27

Attachments:

GoogleCodeExporter commented 9 years ago 
I do not see any changes commited on http://code.google.com/p/beanio/source/list

I am currently using 2.1.0.M1 for annotation support.

In org.beanio.builder.DelimitedParserBuilder: 

     public DelimitedParserBuilder recordTerminator(char terminator) {
        parser.setRecordTerminator(Character.toString(terminator));
        return this;
    }

should change to:

     public DelimitedParserBuilder recordTerminator(String terminator) {
        parser.setRecordTerminator(terminator);
        return this;
    }

Original comment by stephen....@gmail.com on 31 May 2013 at 3:58

GoogleCodeExporter commented 9 years ago 
Sorry about that, 2.1.0 snapshot JAR attached as well.

Original comment by kevin.s...@gmail.com on 2 Jun 2013 at 9:32

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by kevin.s...@gmail.com on 6 Jun 2013 at 2:02

GoogleCodeExporter commented 9 years ago

Original comment by kevin.s...@gmail.com on 16 Nov 2013 at 5:28