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

Quote ASCII format differently #1075

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

LemmingAvalanche
Copy link
Contributor

qsv fmt --ascii uses the ASCII unit separator (U+001F) as the record delimiter and the ASCII record separator (U+001E) as the record terminator. The point is to use metacharacters that don’t appear in regular ASCII text; these characters are neither part of the printable character subset, nor the whitespace control codes.

But it still uses " as the quote character. That means that data that contains quotes will need to be quoted using this printable character.

All metacharacters (delimiter, terminator, and quote) being non-whitespace control codes would enable you to write out any regular text (printable character or whitespace) without “quotations”.

Use the control code ASCII “Substitute” (U+001A) as the quote character.

Fixes: #1074

`qsv fmt --ascii` uses the ASCII unit separator (U+001F) as the
record *delimiter* and the ASCII record separator (U+001E) as the record
*terminator*. The point is to use metacharacters that don’t appear in
regular ASCII text; these characters are neither part of the printable
character subset, nor the whitespace control codes.

But it still uses `"` as the quote character. That means that data that
contains quotes will need to be quoted using this printable character.

All metacharacters (delimiter, terminator, and quote) being non-whitespace
control codes would enable you to write out any regular text (printable
character or whitespace) without “quotations”.

Use the control code ASCII “Substitute” (U+001A) as the quote character.

Fixes: dathere#1074
@jqnatividad jqnatividad merged commit 5be4893 into dathere:master Jun 27, 2023
@LemmingAvalanche LemmingAvalanche deleted the ascii-quote branch June 27, 2023 18:51
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

Successfully merging this pull request may close these issues.

qsv fmt --ascii should not use " as the quote character
2 participants