tc39 / proposal-json-parse-with-source

Proposal for extending JSON.parse to expose input source text.
https://tc39.github.io/proposal-json-parse-with-source
MIT License
213 stars 9 forks source link

Editorial: Narrow ShallowestContained to be JSON-specific #30

Closed gibson042 closed 2 years ago

gibson042 commented 2 years ago

Responding to https://github.com/tc39/proposal-json-parse-with-source/issues/29#issuecomment-1162465280

The ShallowestContained algorithm doesn't work in general. It may happen to work for the JSON subset, but it forms an attractive nuisance that hides land mines for others who might call it in the future.

cc @waldemarhorwat