fix: place preferred char in space token #295
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes #122 that claims that
\r\n
takes just the\r
into the space token value.The case is a little different. The space token takes only a single char into the value and breaks the rest of the whitespace to
before
andafter
. It used to search for normal space, and fallback to the first character. Now it looks for normal space and falls back to\n
and then\t
and only then picks the first character.This is done presumably to help code formatting, but seems unnecessary to me. I change this in order to close the issue, but would prefer removing he
before
and `after altogether at some point.