Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Lucretiel
Copy link
Contributor

@Lucretiel Lucretiel commented Sep 1, 2022

Proposed addition of escaped whitespace to KDL strings. The updates to SPEC.md include the specifics, but in general the idea is that, in a string, \ followed by any amount of literal (not escaped) whitespace are discarded. "Hello\n\tWorld" and "Hello\n\ \tWorld" are therefore identical.

Additionally adds a test case.

Written per discussion in #286 (comment).

Resolves #213.

@Lucretiel Lucretiel added enhancement New feature or request breaking This can only be done for the next major version of KDL labels Sep 1, 2022
@Lucretiel Lucretiel requested a review from zkat September 1, 2022 04:05
@Lucretiel Lucretiel changed the base branch from main to kdl-v2 September 1, 2022 04:07
@Lucretiel
Copy link
Contributor Author

Lucretiel commented Sep 1, 2022

Screwed up the merge from v2; don't merge this PR until I rebase

Rebase completed

@zkat zkat merged commit 825ff2c into kdl-org:kdl-v2 Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking This can only be done for the next major version of KDL enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

\ at EOL should escape all whitespace after it and treat it as a single one

2 participants