#38 AbstractCsvWriter.escapeString() does not increment the line count correctly for embedded newlines

[James Bassett]

AbstractCsvWriter.escapeString() does not increment the line count correctly for embedded newlines. Currently, it only increments the line count for \n. This needs to be updated to cater for \r and \r\n as well (just like LineNumberReader).

It also replaces line terminators with the end of line symbols in the preferences (someone might disagree, but I think that's not a bad idea), so it should do the same for the other line terminators as well.

[James Bassett]

Fixed in [r274].

Now I'm having second thoughts about how it replaces embedded line terminators (any of the 3) with the preference 'end of line symbols'. That is actually modifying the data (what if you wanted your data to be independent of the line terminators used for the CSV file??).

It might be a candidate for a preference, though I'm not sure what the default should be (replace or preserve?)!