Title: [102522] trunk/Source/_javascript_Core
Revision
102522
Author
[email protected]
Date
2011-12-10 09:13:07 -0800 (Sat, 10 Dec 2011)

Log Message

Unused variable in YarrJIT.cpp.
https://bugs.webkit.org/show_bug.cgi?id=74237

Reviewed by Andreas Kling.

Variable is set but not used so we can remove it.

* yarr/YarrJIT.cpp:
(JSC::Yarr::YarrGenerator::generatePatternCharacterOnce):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (102521 => 102522)


--- trunk/Source/_javascript_Core/ChangeLog	2011-12-10 14:50:55 UTC (rev 102521)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-12-10 17:13:07 UTC (rev 102522)
@@ -1,3 +1,15 @@
+2011-12-10  Alexis Menard  <[email protected]>
+
+        Unused variable in YarrJIT.cpp.
+        https://bugs.webkit.org/show_bug.cgi?id=74237
+
+        Reviewed by Andreas Kling.
+
+        Variable is set but not used so we can remove it.
+
+        * yarr/YarrJIT.cpp:
+        (JSC::Yarr::YarrGenerator::generatePatternCharacterOnce):
+
 2011-12-09  Filip Pizlo  <[email protected]>
 
         DFG ArithMul power-of-two case does not check for overflow

Modified: trunk/Source/_javascript_Core/yarr/YarrJIT.cpp (102521 => 102522)


--- trunk/Source/_javascript_Core/yarr/YarrJIT.cpp	2011-12-10 14:50:55 UTC (rev 102521)
+++ trunk/Source/_javascript_Core/yarr/YarrJIT.cpp	2011-12-10 17:13:07 UTC (rev 102522)
@@ -675,7 +675,6 @@
         const RegisterID character = regT0;
         int maxCharactersAtOnce = m_charSize == Char8 ? 4 : 2;
         unsigned ignoreCaseMask = 0;
-        unsigned currentCharacterMask = m_charSize == Char8 ? 0xff : 0xffff;
         int allCharacters = ch;
         int numberCharacters;
         int startTermPosition = term->inputPosition;
@@ -699,7 +698,6 @@
             nextOp->m_isDeadCode = true;
 
             int shiftAmount = (m_charSize == Char8 ? 8 : 16) * numberCharacters;
-            currentCharacterMask = (m_charSize == Char8 ? 0xff : 0xffff) << shiftAmount;  
 
             UChar currentCharacter = nextTerm->patternCharacter;
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to