Changeset 50675 in webkit for trunk/JavaScriptCore/interpreter

Timestamp:

Nov 9, 2009, 12:04:49 PM (16 years ago)

Author:

[email protected]

Message:

2009-11-09 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

Use explicit parentheses to silence gcc 4.4 -Wparentheses warnings
​https://bugs.webkit.org/show_bug.cgi?id=31040

• interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):

2009-11-09 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

Use explicit parentheses to silence gcc 4.4 -Wparentheses warnings
​https://bugs.webkit.org/show_bug.cgi?id=31040

No new tests as there is no functional change.

• dom/Document.cpp: (WebCore::Document::recalcStyleSelector):
• editing/TextIterator.cpp: (WebCore::pushFullyClippedState):
• editing/VisibleSelection.cpp: (WebCore::VisibleSelection::appendTrailingWhitespace):
• html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::process):
• loader/RedirectScheduler.cpp: (WebCore::RedirectScheduler::mustLockBackForwardList):
• loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didFinishLoadingManifest):
• platform/graphics/qt/FontCacheQt.cpp: (WebCore::FontPlatformDataCacheKey::computeHash):
• rendering/RenderBlock.cpp: (WebCore::RenderBlock::handleRunInChild):
• rendering/RenderBox.cpp: (WebCore::RenderBox::calcHeight):
• rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::nodeAtPoint):
• rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::diff):
• svg/SVGAnimateElement.cpp: (WebCore::parseNumberValueAndUnit):
• svg/SVGAnimationElement.cpp: (WebCore::SVGAnimationElement::startedActiveInterval):
• svg/SVGPreserveAspectRatio.cpp: (WebCore::SVGPreserveAspectRatio::getCTM):

2009-11-09 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

Use explicit parentheses to silence gcc 4.4 -Wparentheses warnings
​https://bugs.webkit.org/show_bug.cgi?id=31040

• Api/qwebpage.cpp: (QWebPagePrivate::handleScrolling):

File:

• trunk/JavaScriptCore/interpreter/Interpreter.cpp (modified) (2 diffs)

trunk/JavaScriptCore/interpreter/Interpreter.cpp

@@ -1496 +1496 @@
         JSValue src1 = callFrame->r(vPC[2].u.operand).jsValue();
         JSValue src2 = callFrame->r(vPC[3].u.operand).jsValue();
-        if (src1.isInt32() && src2.isInt32() && !(src1.asInt32() | src2.asInt32() & 0xc0000000)) // no overflow
+        if (src1.isInt32() && src2.isInt32() && !(src1.asInt32() | (src2.asInt32() & 0xc0000000))) // no overflow
             callFrame->r(dst) = jsNumber(callFrame, src1.asInt32() + src2.asInt32());
         else {
@@ -1583 +1583 @@
         JSValue src1 = callFrame->r(vPC[2].u.operand).jsValue();
         JSValue src2 = callFrame->r(vPC[3].u.operand).jsValue();
-        if (src1.isInt32() && src2.isInt32() && !(src1.asInt32() | src2.asInt32() & 0xc0000000)) // no overflow
+        if (src1.isInt32() && src2.isInt32() && !(src1.asInt32() | (src2.asInt32() & 0xc0000000))) // no overflow
             callFrame->r(dst) = jsNumber(callFrame, src1.asInt32() - src2.asInt32());
         else {