Hi James, James Carman wrote:
> It might need a rename, since it looks like its scope has changed. It > doesn't really just deal with CSV files anymore. It allows custom > delimiter characters. MS always happily interpreted CSV in various ways. Excel used to export CSV with semicolon as delimiter running with a German Windows locale :-/ Wikipedia also has an article about "Delimiter-separated_values", but I doubt that our users will associate with the abbreviation "DSV" something useful regarding the functionality of our component. IMHO is "CSV" as name still appropriate as long as its default settings will act as expected. > I have a few other suggestions as well: > > 1. Change the char properties that can be "disabled" to Character > objects. Thus, a null value indicates disabling that feature? > 2. Change references to CSV to Csv in the class names. +1 > 3. Use a "builder" pattern to define the formats. +1 - Jörg --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
