WIP: New CSV parser

[marty1885]

See #1280.

[SylvainCorlay]

Thanks for this work @marty1885! We will be reviewing this tomorrow!

[JohanMabille]

I think we should keep the same method name and dispatch according to a TAG or enum, something like:

enum class csv_mode
{
    fast,
    strict
};
template <class T, class A = std::allocator<T>>
xcsv_tensor<T, A> load_csv(std::istream& stream, csv_mode mode = csv_mode::fast)
{
    // Calls the implementation method accoriding to the value of csv_mode
}

[JohanMabille]

Hi @marty1885 ,

Sorry for the late review. The implementation of the parser itself looks good to me (maybe we could simplify it with std::regex but I'm not sure and this is something I don't master yet), my only remark is about the API.

[marty1885]

@JohanMabille Thanks. I'll change that. My main concern is in my latest comment #1280 . May you check it out?