Bug 695324. Fix precedence problem found with clang warning. r=mounir
authorJeff Muizelaar <jmuizelaar@mozilla.com>
Tue, 18 Oct 2011 14:19:07 -0400
changeset 79064 344e0e0cabe451fd8d99f0b5729d8cd3b7121fa9
parent 79063 7eefb2f21c4c60a14c2e89a33024a199ce6eeab5
child 79065 2606bc44cb7cceeafe055da65136e1f9a4d49052
push id247
push usertim.taubert@gmx.de
push dateSat, 22 Oct 2011 19:08:15 +0000
treeherderfx-team@72bb20c484a2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmounir
bugs695324
milestone10.0a1
Bug 695324. Fix precedence problem found with clang warning. r=mounir '*' has higher precedence than '?' so bracket the expression so we get what we want.
content/base/src/nsTextFragment.h
--- a/content/base/src/nsTextFragment.h
+++ b/content/base/src/nsTextFragment.h
@@ -224,17 +224,17 @@ public:
 
   /**
    * Returns the size taken in memory by this text fragment.
    * @return the size taken in memory by this text fragment.
    */
   PRInt64 SizeOf() const
   {
     PRInt64 size = sizeof(*this);
-    size += GetLength() * Is2b() ? sizeof(*m2b) : sizeof(*m1b);
+    size += GetLength() * (Is2b() ? sizeof(*m2b) : sizeof(*m1b));
     return size;
   }
 
 private:
   void ReleaseText();
 
   /**
    * Scan the contents of the fragment and turn on mState.mIsBidi if it