[jira] [Commented] (CSV-61) CSVFormat combines input and output settings in a single class; might be clearer as separate classes
[ 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
[ 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
[ 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