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

Records containing carriage returns should be auto-quoted #56

Closed
mvaneerde opened this issue Nov 13, 2020 · 1 comment
Closed

Records containing carriage returns should be auto-quoted #56

mvaneerde opened this issue Nov 13, 2020 · 1 comment

Comments

@mvaneerde
Copy link
Contributor

If auto-quote is in effect, records containing line feeds (\n) are correctly auto-quoted.
Also, records containing carriage return/line feed pairs (\r\n) are correctly auto-quoted.
But records containing only carriage returns (\r) are not auto-quoted.

@ryu1kn
Copy link
Owner

ryu1kn commented Nov 14, 2020

Hi @mvaneerde , thanks for raising this. You're right, looking at the section 2 of point 7, CR needs to be escaped.

escaped = DQUOTE *(TEXTDATA / COMMA / CR / LF / 2DQUOTE) DQUOTE

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

No branches or pull requests

2 participants