ReadCSV should take a Reader as input rather than a string about gota

[kniren]

Hi @tobgu this makes complete sense. Reading from a Reader should be the preferred way of dealing with this.

In addition to this, it would also make sense that the Save methods would write to a Writer or WriteCloser for consistency and better composability.

The exception to this would be the ReadRecords/SaveRecords methods, but for other formats (CSV, JSON, XML...) it should work.

[tobgu]

Sounds good!

[kniren]

I've finished with these modifications on the dev branch as of commit 48ef419

As a summary:

• ReadCSV now accepts an io.Reader instead of a string
• SaveCSV/SaveJSON now write directly to the given io.Writer instead of returning a []byte
• SaveRecords/SaveMaps has been renamed to Records/Maps respectively to be consistent with the Dense method
• ReadCSV/ReadMaps has been renamed to LoadCSV/LoadMaps since they don't accept readers.
• SaveCSV/SaveMaps has been renamed to WriteCSV/WriteMaps for consistency with the Read* methods.
• SaveMaps was not working properly and thus it has been fixed.