AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[regexp] fix subtle bug in RegExpTest.

Browse Source 
R=bmeurer@chromium.org

Review-Url: https://codereview.chromium.org/2032713003
Cr-Commit-Position: refs/heads/master@{#36670}
This commit is contained in:
yangguo 2016-06-02 01:03:40 -07:00 committed by Commit bot
parent 864b07e9eb
commit 7ca611d1a5
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/js/regexp.js
View File

@ -325,10 +325,10 @@ function RegExpTest(string) {
// not a '?'. But see https://code.google.com/p/v8/issues/detail?id=3560 // not a '?'. But see https://code.google.com/p/v8/issues/detail?id=3560
var regexp = this; var regexp = this;
var source = REGEXP_SOURCE(regexp); var source = REGEXP_SOURCE(regexp);
if (regexp.length >= 3 && if (source.length >= 3 &&
%_StringCharCodeAt(regexp, 0) == 46 && // '.' %_StringCharCodeAt(source, 0) == 46 && // '.'
%_StringCharCodeAt(regexp, 1) == 42 && // '*' %_StringCharCodeAt(source, 1) == 42 && // '*'
%_StringCharCodeAt(regexp, 2) != 63) { // '?' %_StringCharCodeAt(source, 2) != 63) { // '?'
regexp = TrimRegExp(regexp); regexp = TrimRegExp(regexp);
} }
// matchIndices is either null or the RegExpLastMatchInfo array. // matchIndices is either null or the RegExpLastMatchInfo array.