Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(205)

Side by Side Diff: test/mjsunit/regress/regress-6209.js

Issue 2798933003: [regexp] Properly handle HeapNumbers in AdvanceStringIndex (Closed)
Patch Set: Address comments Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/builtins/builtins-regexp-gen.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2017 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 // Flags: --allow-natives-syntax
6
7 function testAdvanceStringIndex(lastIndex, expectedLastIndex) {
8 let exec_count = 0;
9 let last_last_index = -1;
10
11 let fake_re = {
12 exec: () => { return (exec_count++ == 0) ? [""] : null },
13 get lastIndex() { return lastIndex; },
14 set lastIndex(value) { last_last_index = value },
15 get global() { return true; },
16 get flags() { return "g"; }
17 };
18
19 assertEquals([""], RegExp.prototype[Symbol.match].call(fake_re, "abc"));
20 assertEquals(expectedLastIndex, last_last_index);
21 }
22
23 testAdvanceStringIndex(new Number(42), 43); // Value wrapper.
24 testAdvanceStringIndex(%AllocateHeapNumber(), 1); // HeapNumber.
25 testAdvanceStringIndex(4294967295, 4294967296); // HeapNumber.
OLDNEW
« no previous file with comments | « src/builtins/builtins-regexp-gen.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698