Skip to content
This repository has been archived by the owner on Sep 12, 2018. It is now read-only.

EDN parser swallows and discards \n, \t, \r #813

Open
thomcc opened this issue Aug 22, 2018 · 0 comments
Open

EDN parser swallows and discards \n, \t, \r #813

thomcc opened this issue Aug 22, 2018 · 0 comments
Labels
A-edn EDN parser issues. bug

Comments

@thomcc
Copy link
Contributor

thomcc commented Aug 22, 2018

This doesn't ever unescape string_special_chars, and just assumes they're the same as without the backslash, which is only true for \ and " escapes:

mentat/edn/src/edn.rustpeg

Line 83 in e311378

string_special_char -> &'input str = "\\" $([\\"ntr])

@thomcc thomcc added bug A-edn EDN parser issues. labels Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-edn EDN parser issues. bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thomcc