[jira] [Commented] (CSV-214) Adding a placeholder in the Lexer and CSV parser to store the Line Ending

2017-08-10 Thread Nitin Mahendru (JIRA)

[ 
https://issues.apache.org/jira/browse/CSV-214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122360#comment-16122360
 ] 

Nitin Mahendru commented on CSV-214:


pull request raised.

> Adding a placeholder in the Lexer and CSV parser to store the Line Ending
> -
>
> Key: CSV-214
> URL: https://issues.apache.org/jira/browse/CSV-214
> Project: Commons CSV
>  Issue Type: Improvement
>  Components: Parser
>Reporter: Nitin Mahendru
>Priority: Trivial
> Fix For: Discussion
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Hi All,
> 
> In my use-case , I have to read a CSV file, Mangle some columns and then 
> write out a new csv file with those mangled columns.
> I have gone through the parser code and I have found no usable way of getting 
> the line ending information from the CSVParser object. The function 
> readEndOfLine just consumes End of line whether it is CRLF or LF.
> Now that problem is that when I am writing my file back using the CSVPrinter, 
> I need to know what line ending my input file was. I could write an external 
> function to do that. A better way would be to store that information in the 
> CSVParser object and just use it.
> To do that I am just saving the state in a variable and exposing that using a 
> getter.
> I have done some basic testing. and Submitting a pull request regarding that.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CSV-214) Adding a placeholder in the Lexer and CSV parser to store the Line Ending

2017-08-10 Thread Nitin Mahendru (JIRA)

[ 
https://issues.apache.org/jira/browse/CSV-214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122354#comment-16122354
 ] 

Nitin Mahendru commented on CSV-214:


That would mean a badly formatted file which needs to be corrected in the first 
place.
Although what would happen is that the line ending in the first line will be 
considered as absolute and final.

> Adding a placeholder in the Lexer and CSV parser to store the Line Ending
> -
>
> Key: CSV-214
> URL: https://issues.apache.org/jira/browse/CSV-214
> Project: Commons CSV
>  Issue Type: Improvement
>  Components: Parser
>Reporter: Nitin Mahendru
>Priority: Trivial
> Fix For: Discussion
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Hi All,
> 
> In my use-case , I have to read a CSV file, Mangle some columns and then 
> write out a new csv file with those mangled columns.
> I have gone through the parser code and I have found no usable way of getting 
> the line ending information from the CSVParser object. The function 
> readEndOfLine just consumes End of line whether it is CRLF or LF.
> Now that problem is that when I am writing my file back using the CSVPrinter, 
> I need to know what line ending my input file was. I could write an external 
> function to do that. A better way would be to store that information in the 
> CSVParser object and just use it.
> To do that I am just saving the state in a variable and exposing that using a 
> getter.
> I have done some basic testing. and Submitting a pull request regarding that.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CSV-214) Adding a placeholder in the Lexer and CSV parser to store the Line Ending

2017-08-10 Thread Gary Gregory (JIRA)

[ 
https://issues.apache.org/jira/browse/CSV-214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122232#comment-16122232
 ] 

Gary Gregory commented on CSV-214:
--

What happens when one line ends with CR and another with CRLF?

> Adding a placeholder in the Lexer and CSV parser to store the Line Ending
> -
>
> Key: CSV-214
> URL: https://issues.apache.org/jira/browse/CSV-214
> Project: Commons CSV
>  Issue Type: Improvement
>  Components: Parser
>Reporter: Nitin Mahendru
>Priority: Trivial
> Fix For: Discussion
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Hi All,
> 
> In my use-case , I have to read a CSV file, Mangle some columns and then 
> write out a new csv file with those mangled columns.
> I have gone through the parser code and I have found no usable way of getting 
> the line ending information from the CSVParser object. The function 
> readEndOfLine just consumes End of line whether it is CRLF or LF.
> Now that problem is that when I am writing my file back using the CSVPrinter, 
> I need to know what line ending my input file was. I could write an external 
> function to do that. A better way would be to store that information in the 
> CSVParser object and just use it.
> To do that I am just saving the state in a variable and exposing that using a 
> getter.
> I have done some basic testing. and Submitting a pull request regarding that.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)