[jira] [Commented] (CSV-61) CSVFormat combines input and output settings in a single class; might be clearer as separate classes

2012-03-14 Thread Emmanuel Bourg (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/CSV-61?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13229506#comment-13229506
 ] 

Emmanuel Bourg commented on CSV-61:
---

I'm -1 on this, this will make the API more complex for no real gain.

 CSVFormat combines input and output settings in a single class; might be 
 clearer as separate classes
 

 Key: CSV-61
 URL: https://issues.apache.org/jira/browse/CSV-61
 Project: Commons CSV
  Issue Type: Improvement
Reporter: Sebb

 CSVFormat currently includes both input (parsing) and output (formatting) 
 settings in a single class.
 This is a bit confusing; for example lineSeparator could be either.
 (The Javadoc has now been corrected)
 It might be clearer to have separate classes for input and output.
 This would also reduce the number of parameters in each ctor.

--
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




[jira] [Commented] (CSV-61) CSVFormat combines input and output settings in a single class; might be clearer as separate classes

2012-03-14 Thread Sebb (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/CSV-61?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13229522#comment-13229522
 ] 

Sebb commented on CSV-61:
-

Applications that read CSV files don't necessarily want to write them and 
vice-versa.

Separating the configurations would allow them to be used independently.

It might be a bit different if the same format definitions were used for input 
and output (as for DateFormat), but they are not, and I suspect that would not 
be possible.

 CSVFormat combines input and output settings in a single class; might be 
 clearer as separate classes
 

 Key: CSV-61
 URL: https://issues.apache.org/jira/browse/CSV-61
 Project: Commons CSV
  Issue Type: Improvement
Reporter: Sebb

 CSVFormat currently includes both input (parsing) and output (formatting) 
 settings in a single class.
 This is a bit confusing; for example lineSeparator could be either.
 (The Javadoc has now been corrected)
 It might be clearer to have separate classes for input and output.
 This would also reduce the number of parameters in each ctor.

--
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




[jira] [Commented] (CSV-61) CSVFormat combines input and output settings in a single class; might be clearer as separate classes

2012-03-14 Thread Emmanuel Bourg (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/CSV-61?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13229690#comment-13229690
 ] 

Emmanuel Bourg commented on CSV-61:
---

I want to be able to read and write files with the same format. Splitting the 
format will make this task more complicated.

 CSVFormat combines input and output settings in a single class; might be 
 clearer as separate classes
 

 Key: CSV-61
 URL: https://issues.apache.org/jira/browse/CSV-61
 Project: Commons CSV
  Issue Type: Improvement
Reporter: Sebb

 CSVFormat currently includes both input (parsing) and output (formatting) 
 settings in a single class.
 This is a bit confusing; for example lineSeparator could be either.
 (The Javadoc has now been corrected)
 It might be clearer to have separate classes for input and output.
 This would also reduce the number of parameters in each ctor.

--
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