Skip to content
This repository has been archived by the owner on Feb 19, 2021. It is now read-only.

Missed matchAll-related features #25

Closed
zloirock opened this issue Sep 27, 2019 · 1 comment
Closed

Missed matchAll-related features #25

zloirock opened this issue Sep 27, 2019 · 1 comment

Comments

@zloirock
Copy link
Contributor

#24 adds changes to String#matchAll, but not to related methods. Maybe, in this case, makes sense remove handling non-global regexes from RegExp.prototype[@@matchAll], CreateRegExpStringIterator, %RegExpStringIteratorPrototype%.next() and simplify those methods?

@ljharb
Copy link
Member

ljharb commented Sep 27, 2019

I don’t think that’s necessary or ideal. The string matchAll method, alone, is where the confusion lies with non-global regexes.

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