search

jviereck / regjsparser

Parsing the JavaScript's RegExp in JavaScript.
http://www.julianviereck.de/regjsparser/
BSD 2-Clause "Simplified" License
77 stars 20 forks source link

Fixing #105 /{(?<x>)}/ #106

Closed jviereck closed 3 years ago

jviereck commented 3 years ago

This is work in progress. The current change improves parsing of empty brackets like /{}/ but does not yet parse cases like /{(?<x>)}/ from #105 .

jviereck commented 3 years ago

Turns out, when turning on the named group feature, the regexp /{(?<x>)}/ as well as the originally reported regex in #105 parse nicely.

@nicolo-ribaudo , can I ask you to do a review on this PR?

jviereck commented 3 years ago

Thanks for the review @nicolo-ribaudo !