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

Sebb commented on CSV-126:
--------------------------

The patch makes unnecessary changes to the import statements. Whilst these 
don't affect the functional behaviour, it makes the patch harder to review, 
both now, and later when other developers review the commit message (assuming 
that the patch is applied as is).

Given that the intention is to create a class instance from a file, a different 
approach might be to use something like XStream.

However I'm not sure that the use case is sufficiently compelling to include 
the function in CSV.

> Initialize a CSVFormat from Properties
> --------------------------------------
>
>                 Key: CSV-126
>                 URL: https://issues.apache.org/jira/browse/CSV-126
>             Project: Commons CSV
>          Issue Type: Wish
>            Reporter: Xavier Courangon
>            Priority: Minor
>             Fix For: Patch Needed
>
>         Attachments: commons-csv_CSV-126.patch
>
>
> Hi,
> It would be nice to be able to initialize a CSVFormat from a Properties 
> object or a properties file.
> Some CSVParser may have to be initialized form a configuration file.
> Thanks. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to