Reviewed by Darin Adler.

        Removed bogus ASSERT.

        ASSERT should only be used when we know that a code path will not be
        taken. This code path is taken often during the jsFunFuzz test.

        * pcre/pcre_exec.cpp:
        (jsRegExpExecute):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@28627 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/JavaScriptCore/ChangeLog b/JavaScriptCore/ChangeLog
index 12c578237a25726569866aba2002ac906d324057..e855d370bba8b142aaa610a6230a881a9daff5cd 100644 (file)
--- a/JavaScriptCore/ChangeLog
+++ b/JavaScriptCore/ChangeLog
@@ -1,3 +1,15 @@
+2007-12-11  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Darin Adler.
+        
+        Removed bogus ASSERT.
+        
+        ASSERT should only be used when we know that a code path will not be
+        taken. This code path is taken often during the jsFunFuzz test.
+
+        * pcre/pcre_exec.cpp:
+        (jsRegExpExecute):
+
 2007-12-11  Darin Adler  <darin@apple.com>
 
         * wtf/unicode/qt4/UnicodeQt4.h: Try to fix Qt build by adding U16_IS_SINGLE.

diff --git a/JavaScriptCore/pcre/pcre_exec.cpp b/JavaScriptCore/pcre/pcre_exec.cpp
index f9b9329a3ed392e4a1572ab88417a6f4d70f5fc6..1b4cca36f2d3b2305a330171b8787eb2866df1d2 100644 (file)
--- a/JavaScriptCore/pcre/pcre_exec.cpp
+++ b/JavaScriptCore/pcre/pcre_exec.cpp
@@ -1992,7 +1992,6 @@ int jsRegExpExecute(const JSRegExp* re,
         if (!match_block.offset_vector)
             return JSRegExpErrorNoMemory;
         using_temporary_offsets = true;
-        ASSERT_NOT_REACHED(); // Fail debug builds -- No one should be hitting this vestigal (slow!) code, see comment above.
     } else
         match_block.offset_vector = offsets;