[ https://issues.apache.org/jira/browse/CSV-75?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239072#comment-13239072 ]
Sebb commented on CSV-75: ------------------------- Yes, perhaps it should be done by the lexer. But the quickest fix would be to patch the reader. I wonder whether ExtendedBufferReader is actually necessary; lookAhead() could easily be provided by the Lexer class. And I'm not sure that readAgain() is really necessary. > ExtendedBufferReader does not handle EOL consistently > ----------------------------------------------------- > > Key: CSV-75 > URL: https://issues.apache.org/jira/browse/CSV-75 > Project: Commons CSV > Issue Type: Bug > Reporter: Sebb > Attachments: CSV-75.patch > > > ExtendedBufferReader checks for '\n' (LF) in the read() methods, incrementing > linecount when found. > However, the readLine() method calls BufferedReader.readLine() which treats > CR, LF and CRLF equally (and drops them). > If the code is to be flexible in what it accepts, the class should also allow > for CR alone as a line terminator. > It should work if the code increments the line counter for CR, and for LF if > the previous character was not CR. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira