Bug 1356843 - Fix -Wcomma warnings in dom/svg/SVGGeomeetryElement.h. r=dholbert draft
authorChris Peterson <cpeterson@mozilla.com>
Mon, 17 Apr 2017 22:22:51 -0700
changeset 563946 5ab03d10f255179f68aeb9559e9a64600c313b42
parent 563802 a374c35469935a874fefe64d3e07003fc5bc8884
child 563947 ed9a0bf53f232a887d40918d2ff7d9ca63a45455
push id54485
push usercpeterson@mozilla.com
push dateTue, 18 Apr 2017 05:57:33 +0000
reviewersdholbert
bugs1356843
milestone55.0a1
Bug 1356843 - Fix -Wcomma warnings in dom/svg/SVGGeomeetryElement.h. r=dholbert clang's -Wcomma warning warns about suspicious use of the comma operator such as between two statements. dom/svg/SVGGeometryElement.h:117:21 [-Wcomma] possible misuse of comma operator here dom/svg/SVGGeometryElement.h:117:41 [-Wcomma] possible misuse of comma operator here dom/svg/SVGGeometryElement.h:128:14 [-Wcomma] possible misuse of comma operator here dom/svg/SVGGeometryElement.h:128:23 [-Wcomma] possible misuse of comma operator here MozReview-Commit-ID: CfNOsfcJTEp
dom/svg/SVGGeometryElement.h
--- a/dom/svg/SVGGeometryElement.h
+++ b/dom/svg/SVGGeometryElement.h
@@ -109,28 +109,34 @@ public:
   public:
     SimplePath()
       : mType(NONE)
     {}
     bool IsPath() const {
       return mType != NONE;
     }
     void SetRect(Float x, Float y, Float width, Float height) {
-      mX = x; mY = y, mWidthOrX2 = width, mHeightOrY2 = height;
+      mX = x;
+      mY = y;
+      mWidthOrX2 = width;
+      mHeightOrY2 = height;
       mType = RECT;
     }
     Rect AsRect() const {
       MOZ_ASSERT(mType == RECT);
       return Rect(mX, mY, mWidthOrX2, mHeightOrY2);
     }
     bool IsRect() const {
       return mType == RECT;
     }
     void SetLine(Float x1, Float y1, Float x2, Float y2) {
-      mX = x1, mY = y1, mWidthOrX2 = x2, mHeightOrY2 = y2;
+      mX = x1;
+      mY = y1;
+      mWidthOrX2 = x2;
+      mHeightOrY2 = y2;
       mType = LINE;
     }
     Point Point1() const {
       MOZ_ASSERT(mType == LINE);
       return Point(mX, mY);
     }
     Point Point2() const {
       MOZ_ASSERT(mType == LINE);