quick-lint-js icon indicating copy to clipboard operation
quick-lint-js copied to clipboard

Report error for invalid regexp literal flags

Open strager opened this issue 4 years ago • 7 comments

If a regular expression literal contains invalid flags (/hello/q4), quick-lint-js should report an error.

See the ECMAScript standard RegExpInitialize for an explanation of what flags are valid.

See lex.cpp, error.h, and test-lex.cpp for implementation.

strager avatar Oct 01 '20 03:10 strager

how i do dis

strager avatar Oct 01 '20 05:10 strager

@collidedscope Do you want to make a Pull Request for your patch?

strager avatar Feb 06 '21 10:02 strager

Hey strager, workin on this one

KenAKAFrosty avatar Oct 27 '21 00:10 KenAKAFrosty

Just a heads-up I had some personal stuff come up this past week, so I've not made any progress but it's not due to my disinterest or abandonment

KenAKAFrosty avatar Nov 05 '21 00:11 KenAKAFrosty

@KenAKAFrosty Do you still plan to work on this task?

strager avatar Jan 08 '22 03:01 strager

Oh my god - I'm so sorry. A lot went on and this fell off my radar. As of right now I can't commit to it, so feel free to remove me. Congrats on 1.0 release btw!

KenAKAFrosty avatar Jan 11 '22 00:01 KenAKAFrosty

Can I work on this?

wilson1375 avatar Jun 01 '22 18:06 wilson1375

@wilson1375 Are you still working on this task? Do you need any help?

strager avatar Sep 25 '22 05:09 strager