https://bugs.webkit.org/show_bug.cgi?id=32524
authorggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Dec 2009 18:40:59 +0000 (18:40 +0000)
committerggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Dec 2009 18:40:59 +0000 (18:40 +0000)
REGRESSION(52084): fast/dom/prototypes.html failing two CSS tests

Reviewed by Simon Fraser.

* wtf/StdLibExtras.h:
(WTF::bitCount): The original patch put the parentheses in the wrong
place, completely changing the calculation and making it almost always
wrong. Moved the parentheses around the '+' operation, like the original
compiler warning suggested.

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

JavaScriptCore/ChangeLog
JavaScriptCore/wtf/StdLibExtras.h

index 58f0f9a..5332875 100644 (file)
@@ -1,3 +1,16 @@
+2009-12-14  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Simon Fraser.
+
+        https://bugs.webkit.org/show_bug.cgi?id=32524
+        REGRESSION(52084): fast/dom/prototypes.html failing two CSS tests
+
+        * wtf/StdLibExtras.h:
+        (WTF::bitCount): The original patch put the parentheses in the wrong
+        place, completely changing the calculation and making it almost always
+        wrong. Moved the parentheses around the '+' operation, like the original
+        compiler warning suggested.
+
 2009-12-14  Gabor Loki  <loki@inf.u-szeged.hu>
 
         Unreviewed trivial buildfix.
index 9d7b36b..9dfb969 100644 (file)
@@ -74,7 +74,7 @@ namespace WTF {
     {
         bits = bits - ((bits >> 1) & 0x55555555);
         bits = (bits & 0x33333333) + ((bits >> 2) & 0x33333333);
-        return ((bits + ((bits >> 4) & 0xF0F0F0F)) * 0x1010101) >> 24;
+        return (((bits + (bits >> 4)) & 0xF0F0F0F) * 0x1010101) >> 24;
     }
 
 } // namespace WTF