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.

Sign up for GitHub

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

Jump to bottom

"no newline found" - BUFSIZE of 1024 too small for long headers #40

Closed
simonpercivall opened this issue Apr 6, 2017 · 3 comments
Closed

"no newline found" - BUFSIZE of 1024 too small for long headers #40

simonpercivall opened this issue Apr 6, 2017 · 3 comments
Labels
bug important

Comments

@simonpercivall
Copy link

For CSVs with long headers, reading 1024 characters for the header row may be too conservative. In fact, the header row of the first CSV I tried opening with Tad has a length of just under 4000 characters.
@antonycourtney
Copy link
Owner

Well spotted - thank you!

@antonycourtney
Copy link
Owner

@simonpercivall - Going to bump this to 8K and close the issue for now. Hope to issue a new point release in the next few days. Please feel free to re-open the issue if this proves insufficient in your use cases. Thanks.

@antonycourtney
Copy link
Owner

Fixed (for now) by 5b6b25d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug important
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@simonpercivall @antonycourtney