Changeset 37876 in webkit for trunk/JavaScriptCore/pcre

Timestamp:

Oct 24, 2008, 5:09:56 PM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin Adler.

Get rid of a bonus ASSERT when using a null string as a regexp.
Specifically calling: RegularExpression::match() with String::empty()
will hit this ASSERT.
Chromium hits this, but I don't know of any way to make a layout test.

• pcre/pcre_exec.cpp: (jsRegExpExecute):

File:

• trunk/JavaScriptCore/pcre/pcre_exec.cpp (modified) (1 diff)

trunk/JavaScriptCore/pcre/pcre_exec.cpp

@@ -1956 +1956 @@
 {
     ASSERT(re);
-    ASSERT(subject);
+    ASSERT(subject || !length);
     ASSERT(offsetCount >= 0);
     ASSERT(offsets || offsetCount == 0);