Changeset 50143 in webkit for trunk/JavaScriptCore

Timestamp:

Oct 27, 2009, 6:51:42 AM (16 years ago)

Author:

Csaba Osztrogonác

Message:

Rubber-stamped by Tor Arne Vestbø.

Patch by Peter Varga <​[email protected]> on 2009-10-27
Fix typo in RegexInterpreter.cpp and RegexJIT.cpp alterantive to
alternative.

• yarr/RegexInterpreter.cpp:

(JSC::Yarr::ByteCompiler::alternativeBodyDisjunction):
(JSC::Yarr::ByteCompiler::alternativeDisjunction):
(JSC::Yarr::ByteCompiler::emitDisjunction):

• yarr/RegexJIT.cpp:

(JSC::Yarr::RegexGenerator::generateDisjunction):

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• yarr/RegexInterpreter.cpp (modified) (3 diffs)
• yarr/RegexJIT.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2009-10-27  Peter Varga  <[email protected]>
+
+        Rubber-stamped by Tor Arne Vestbø.
+
+        Fix typo in RegexInterpreter.cpp and RegexJIT.cpp alterantive to
+        alternative.
+
+        * yarr/RegexInterpreter.cpp:
+        (JSC::Yarr::ByteCompiler::alternativeBodyDisjunction):
+        (JSC::Yarr::ByteCompiler::alternativeDisjunction):
+        (JSC::Yarr::ByteCompiler::emitDisjunction):
+        * yarr/RegexJIT.cpp:
+        (JSC::Yarr::RegexGenerator::generateDisjunction):
+
 2009-10-26  Laszlo Gombos  <[email protected]>
 

trunk/JavaScriptCore/yarr/RegexInterpreter.cpp

@@ -1491 +1491 @@
     }
 
-    void alterantiveBodyDisjunction()
+    void alternativeBodyDisjunction()
     {
         int newAlternativeIndex = m_bodyDisjunction->terms.size();
@@ -1500 +1500 @@
     }
 
-    void alterantiveDisjunction()
+    void alternativeDisjunction()
     {
         int newAlternativeIndex = m_bodyDisjunction->terms.size();
@@ -1516 +1516 @@
             if (alt) {
                 if (disjunction == m_pattern.m_body)
-                    alterantiveBodyDisjunction();
+                    alternativeBodyDisjunction();
                 else
-                    alterantiveDisjunction();
+                    alternativeDisjunction();
             }
 

trunk/JavaScriptCore/yarr/RegexJIT.cpp

@@ -1265 +1265 @@
         //
         // FIXME: a nice improvement here may be to stop trying to match sooner, based on the least
-        // of the minimum-alterantive-lengths.  E.g. if I have two alternatives of length 200 and 150,
+        // of the minimum-alternative-lengths.  E.g. if I have two alternatives of length 200 and 150,
         // and a string of length 100, we'll end up looping index from 0 to 100, checking whether there
         // is sufficient input to run either alternative (constantly failing).  If there had been only