You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Henri Yandell (JIRA)" <ji...@apache.org> on 2008/02/28 06:31:51 UTC

[jira] Commented: (SANDBOX-218) CSV reader doesn't handle older Mac line endings

    [ https://issues.apache.org/jira/browse/SANDBOX-218?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12573155#action_12573155 ] 

Henri Yandell commented on SANDBOX-218:
---------------------------------------

Looks like it's rather coded into CSVParser to be that way [two places to change], and ExtendedBufferedReader in quite a few places.

ExtendedBufferedReader's places are mostly to deal with incrementing the lineCounter, with one part that already seems to allow \r [and also \n\n - which might be a bug]. 

CSVParser's bits are to deal with identifying whether the reader is at the end of the line; which is a method which feels like it is in the wrong class. Also for empty line detection.

> CSV reader doesn't handle older Mac line endings
> ------------------------------------------------
>
>                 Key: SANDBOX-218
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-218
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: CSV
>            Reporter: Trejkaz
>
> Older Mac OS line endings which just have \r don't get treated as line endings by the current CSV parser.
> Most likely the simplest way to get around this is to delegate to a BufferedReader to reuse its line ending code.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.