Not reviewed; just undoing a previous commit.
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Dec 2007 05:22:38 +0000 (05:22 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Dec 2007 05:22:38 +0000 (05:22 +0000)
        - remove earlier incorrect fix for http://bugs.webkit.org/show_bug.cgi?id=16220
          <rdar://problem/5625221> Crash opening www.news.com (CNet)

        The real bug was the backwards ?: in the compile function, which Geoff just
        fixed. Rolling out the incorrect earlier fix.

        * pcre/pcre_compile.cpp: (calculateCompiledPatternLengthAndFlags): Take out
        the unneeded preflight change. The regression test proves this is still working
        fine, so the bug remains fixed.

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

JavaScriptCore/ChangeLog
JavaScriptCore/pcre/pcre_compile.cpp

index 310c2e6..f65131d 100644 (file)
@@ -1,3 +1,17 @@
+2007-12-05  Darin Adler  <darin@apple.com>
+
+        Not reviewed; just undoing a previous commit.
+
+        - remove earlier incorrect fix for http://bugs.webkit.org/show_bug.cgi?id=16220
+          <rdar://problem/5625221> Crash opening www.news.com (CNet)
+
+        The real bug was the backwards ?: in the compile function, which Geoff just
+        fixed. Rolling out the incorrect earlier fix.
+
+        * pcre/pcre_compile.cpp: (calculateCompiledPatternLengthAndFlags): Take out
+        the unneeded preflight change. The regression test proves this is still working
+        fine, so the bug remains fixed.
+
 2007-12-01  Mark Rowe  <mrowe@apple.com>
 
         Build fix.  Include headers before trying to use the things that they declare.
index f379581..0bb063e 100644 (file)
@@ -2656,7 +2656,6 @@ static int calculateCompiledPatternLengthAndFlags(const UChar* pattern, int patt
                 if (minRepeats == 0) {
                     length++;
                     if (maxRepeats > 0) length += (maxRepeats - 1) * (duplength + 3 + 2 * LINK_SIZE);
-                    if (maxRepeats > 1) length += 2 + 2 * LINK_SIZE; // BRA/KET created in else case
                 }
                 
                 /* When the minimum is greater than zero, we have to replicate up to