Fix bug in the setCString function when reading Frame from CSV files.
This PR closes issue #880. This issue was caused by an incorrect position being returned, leading to errors when reading Frames from CSV files.
Problem
In the setCString function:
The pos variable was set to the size of the result string.
Instead of returning the position just before the next column, the function would return pos, causing incorrect parsing of Frames with mixed data types (strings and numbers).
Solution
To correct this:
When the string has no line breaker, the function returns pos + start_pos instead of simply pos.
This change ensures that the returned position aligns correctly with the next column.
Testing
Additional test cases have been added to validate correct position handling when reading a Frame from a CSV with a string column in the middle.
Fix bug in the
setCStringfunction when reading Frame from CSV files.This PR closes issue #880. This issue was caused by an incorrect position being returned, leading to errors when reading Frames from CSV files.
Problem
In the
setCStringfunction:posvariable was set to the size of the result string.pos, causing incorrect parsing of Frames with mixed data types (strings and numbers).Solution
To correct this:
pos + start_posinstead of simplypos.Testing
Additional test cases have been added to validate correct position handling when reading a Frame from a CSV with a string column in the middle.