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

Timestamp:

Dec 16, 2007, 5:26:54 PM (18 years ago)

Author:

Darin Adler

Message:

JavaScriptCore:

Reviewed by Maciej.

• fix ​http://bugs.webkit.org/show_bug.cgi?id=16459 REGRESSION: assertion failure with regexp with \B in a case-ignoring character range <rdar://problem/5646361>

The problem was that \B was not handled properly in character classes.

Test: fast/js/regexp-overflow.html

• pcre/pcre_compile.cpp: (check_escape): Added handling of ESC_b and ESC_B in character classes here. Allows us to get rid of the handling of \b in character classes from all the call sites that handle it separately and to handle \B properly as well. (compileBranch): Remove the ESC_b handling, since it's not needed any more. (calculateCompiledPatternLengthAndFlags): Ditto.

LayoutTests:

Reviewed by Maciej.

• test for ​http://bugs.webkit.org/show_bug.cgi?id=16459 REGRESSION: assertion failure with regexp with \B in a case-ignoring character range <rdar://problem/5646361>

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

File:

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

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

@@ -18 +18 @@
 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"');
 
+shouldBe('/{([\\D-\\ca]]„£µ+?)}|[[\\B-\\u00d4]√π- ]]]{0,3}/i.exec("B√π- ]]").toString()', '"B√π- ]],"');
+shouldBe('/|[x\\B-\\u00b5]/i.exec("").toString()', '""');
+
 debug('');