Adding support for missing scenario

MikeStall / DataTable

Class library for working with tabular data, especially CSV files

Apache License 2.0

150 stars 63 forks source link

Closed tpwalke2 closed 12 years ago

tpwalke2 commented 12 years ago

Readers.cs did not correctly handle parsing abc,"d=""efg""" as CSV Added tests to ReaderTests.cs for various CSV scenarios

MikeStall commented 12 years ago

Thanks for fixing this scenario and adding tests. And it looks like you made the whitespace trimming more efficient too.

tpwalke2 commented 12 years ago

No problem. I changed the tests to focus more on the Reader itself, but I can pull back in the original tests.

MikeStall commented 12 years ago

I already took the pull request and then re-added the original tests. There was a bug in the tests I had to fix anyways.