Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manage overflow #10

Open
ludoo0d0a opened this issue Jan 18, 2019 · 6 comments
Open

Manage overflow #10

ludoo0d0a opened this issue Jan 18, 2019 · 6 comments

Comments

@ludoo0d0a
Copy link

Overflow of a value in a cell is not managed
When a value length is larger than the header, the row is offset.
then parsing is messy.

NAME          PID    
short_name    1234
long_very_very_long_name  1235
short_name2   1236

Thanks

@doniyor2109
Copy link

Hi. Can I take this issue?

@sindresorhus
Copy link
Owner

@doniyor2109 Go ahead :)

@doniyor2109
Copy link

@sindresorhus I think we need new algorithm for identifying that case. Currently algorithm expects to have same offsets.

@sindresorhus
Copy link
Owner

Yup

@doniyor2109
Copy link

@sindresorhus I wanted to ignore words length however other use cases are broken. Do you have any other thoughts?

@ludoo0d0a
Copy link
Author

From what I saw, you set fixed length from header, then re-use these values.

Just use these value as minimum,
so it should ends with a space/tab in these minimum length ; if not, reach the next space/tab
and then report the cumulative offset (could be null if no overflow) on the right next column.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants