Le 08/08/2013 15:40, Gary Gregory a écrit : > Sans type names: > > parse(File, CSVFormat) > parse(String, Charset, ClassLoader, CSVFormat) > parse(String, Charset, CSVFormat) > [parse(String)] > parse(String, CSVFormat) > parse(URL, Charset, CSVFormat)
That looks better. I would remove the methods for a classpath resource, that's a less common case. That would make: parse(File, CSVFormat) parse(String, CSVFormat) parse(URL, Charset, CSVFormat) And you probably want a charset for the File too. > [I'd probably remove parse(String) so that all APIs take a CSVFormat.] +1. And at this point you realize they could belong to CSVFormat, because they all need one to operate. format.parse(file): instead of: CSVParser.parse(file, format); Emmanuel Bourg --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org