Do not escape / (Solidus, Forwardslash)#197
Conversation
|
This is, unfortunately, a breaking change, so I'm moving it to target the v2 branch. |
|
The secondary solution (allow it to be escaped in parsing, but don't require it to be escaped in printing) appears to already be expected by the testsuite; see (I think superfluous escapes are fine - all it means is we can't give special meaning to an escaped forward slash |
|
oh lol I should look at commit histories; that was changed by the OP several days ago. 👍 |
|
@tabatkins oh, sorry, i made a bunch of alternative fixes, one of which was to do exactly what you've suggested, which got merged 😝 |
|
If an alternate has been merged, can we close this? |
|
The alternate that was merged was just a change to the expected serialization for the tests; the larger (breaking change) question of whether to remove the |
I don't believe
/need to be escaped in strings. This may be considered a breaking change now that we've reached 1.0, so another solution might be to allow it to be escaped during parsing, but not to escape it when stringifying.Closes #194