Skip to content
This repository has been archived by the owner on Jan 26, 2022. It is now read-only.

One parser potentially would not handle escaped lone surrogates #5

Closed
apaprocki opened this issue May 22, 2018 · 3 comments
Closed

Comments

@apaprocki
Copy link

Looks like this would return a parse error?

https://github.com/nlohmann/json/blob/develop/single_include/nlohmann/json.hpp#L2321-L2359

@gibson042
Copy link
Collaborator

Yes, as it is allowed to do. That parser already rejects unescaped lone surrogates, too: https://github.com/nlohmann/json/blob/develop/single_include/nlohmann/json.hpp#L2666-L2671

@apaprocki
Copy link
Author

Ok, just wanted to check it. Thanks

@mathiasbynens
Copy link
Member

Thanks for bringing it up! Always better to double-check.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants