Remove max value assertion in GetStringIndex

tc39 / proposal-regexp-match-indices

ECMAScript RegExp Match Indices

https://arai-a.github.io/ecma262-compare/?pr=1713

BSD 3-Clause "New" or "Revised" License

64 stars 13 forks source link

Remove max value assertion in GetStringIndex #43

Closed rbuckton closed 3 years ago

rbuckton commented 4 years ago

Per @targos:

In GetStringIndex, there's an assertion that e < Input.length, though the next step has a special case for when e is greater than or equal to Input.length.

Fixes #40

rbuckton commented 4 years ago

Preview is at https://tc39.es/proposal-regexp-match-indices/pr/43/index.html#sec-getstringindex (step 3).

rbuckton commented 3 years ago

@ljharb, I've added the empty string check step that you mentioned. Do you have any further comments for this PR?