Comment #2 on issue 4244 by [email protected]: RegExp lastIndex should
use ToLength
https://code.google.com/p/v8/issues/detail?id=4244#c2
The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/1f61ac50338cc2ef1a54735206351f9798687248
commit 1f61ac50338cc2ef1a54735206351f9798687248
Author: littledan <[email protected]>
Date: Thu Jul 16 21:55:31 2015
In RegExp, lastIndex is read with ToLength, not ToInteger
ES2015 made a change vs ES5, where the "lastIndex" property of a
RegExp (which can be modified by a user to start the next search at
a different location) is cast to an integer with ToLength rather
than ToInteger. The main difference is on negative numbers, and
this is tested by test262. This patch implements that change on
RegExps and enables the test262 test now that it passes.
R=adamk
LOG=Y
BUG=v8:4244
Review URL: https://codereview.chromium.org/1241713004
Cr-Commit-Position: refs/heads/master@{#29715}
[modify]
http://crrev.com/1f61ac50338cc2ef1a54735206351f9798687248/src/regexp.js
[modify]
http://crrev.com/1f61ac50338cc2ef1a54735206351f9798687248/test/mozilla/mozilla.status
[modify]
http://crrev.com/1f61ac50338cc2ef1a54735206351f9798687248/test/test262-es6/test262-es6.status
[modify]
http://crrev.com/1f61ac50338cc2ef1a54735206351f9798687248/test/test262/test262.status
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
