CSV-203: withNullString value is printed without quotes when QuoteMod…

[kparoth]

…e.ALL is specified

In my opinion this is only a small change. If I already set the option (QuoteMode.ALL), then it should also be used.

JIRA-Reference

[coveralls]

Coverage increased (+0.2%) to 93.266% when pulling e295ac3 on kparoth:CSV-203 into a775784 on apache:master.

[coveralls]

Coverage increased (+0.03%) to 93.09% when pulling e295ac3 on kparoth:CSV-203 into a775784 on apache:master.

[garydgregory]

But then you cannot do a round trip. How would you differentiate a null value from a string with the same representation?

[coveralls]

Coverage increased (+0.2%) to 93.266% when pulling 82b5210 on kparoth:CSV-203 into a775784 on apache:master.

[kparoth]

The programmer should select a value that is unique and does not correspond to one of the possible values. But "N/A" is already quite clear.

But the fact is, if I have set the mode to ALL, then my default for "withNullString" should be formatted as well.

But it is also just a suggestion. ;-)

[garydgregory]

For backward compatibility, maybe we need to add a QUOTE_ALL_NON_NULL

[coveralls]

Coverage increased (+0.04%) to 93.099% when pulling aebe0d0 on kparoth:CSV-203 into a775784 on apache:master.

[coveralls]

Coverage remained the same at 93.373% when pulling 3ddc583 on kparoth:CSV-203 into 74afb17 on apache:master.

[coveralls]

Coverage remained the same at 93.373% when pulling 3ddc583 on kparoth:CSV-203 into 74afb17 on apache:master.

[coveralls]

Coverage remained the same at 93.373% when pulling 3ddc583 on kparoth:CSV-203 into 74afb17 on apache:master.

[garydgregory]

Closing, see git master.