On 8/14/2013 1:10 PM, Emmanuel Bourg wrote:
Le 14/08/2013 20:09, Benedikt Ritter a écrit :

Every factory method an the constructor now take a CSVFormat. How about
using CSVFormat.DEFAULT is null is passed as CSVFormat?

Keeping null is better, using the default format might hide a bug in the calling code, and it could be difficult to figure it was caused by a null value if no NPE is thrown.

Emmanuel Bourg

I agree. Calling code needs to be explicit. Using the default would be appropriate for a no-arg constructor.

-Adrian


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to