Changeset 28491 in webkit for trunk/LayoutTests/fast/js/resources/regexp-overflow.js

Timestamp:

Dec 6, 2007, 12:24:52 PM (18 years ago)

Author:

Darin Adler

Message:

JavaScriptCore:

Reviewed by Eric Seidel.

• fix ​http://bugs.webkit.org/show_bug.cgi?id=16321 new RegExp("[\u0097]{4,6}", "gmy") crashes in DEBUG builds <rdar://problem/5632992>

Test: fast/js/regexp-oveflow.html

• pcre/pcre_compile.cpp: (calculateCompiledPatternLengthAndFlags): In the case where a single character character class is optimized to not use a character class at all, the preflight code was not setting the lastitemlength variable.

LayoutTests:

Reviewed by Eric Seidel.

• fix ​http://bugs.webkit.org/show_bug.cgi?id=16321 new RegExp("[\u0097]{4,6}", "gmy") crashes in DEBUG builds <rdar://problem/5632992>

• fast/js/regexp-overflow-expected.txt: Updated.
• fast/js/resources/regexp-overflow.js: Added test.

File:

• trunk/LayoutTests/fast/js/resources/regexp-overflow.js (modified) (1 diff)

trunk/LayoutTests/fast/js/resources/regexp-overflow.js

@@ -15 +15 @@
 shouldBe('/(x){0,2}/.exec("").toString()', '","');
 
+shouldBe('/[\u00A1]{4,6}/.exec("\u00A1\u00A1\u00A1\u00A1").toString()', '"\u00A1\u00A1\u00A1\u00A1"');
+shouldBe('/[\u00A1]{1,100}[\u00A1]{1,100}[\u00A1]{1,100}[\u00A1]{1,100}[\u00A1]{1,100}[\u00A1]{1,100}[\u00A1]{1,100}[\u00A1]{1,100}/.exec("\u00A1\u00A1\u00A1\u00A1\u00A1\u00A1\u00A1\u00A1").toString()', '"\u00A1\u00A1\u00A1\u00A1\u00A1\u00A1\u00A1\u00A1"');
+
 debug('');