Bug 1356843 - Fix -Wcomma warnings in widget/cocoa/nsDeviceContextSpecX.mm. r?spohl draft
authorChris Peterson <cpeterson@mozilla.com>
Mon, 27 Mar 2017 21:26:25 -0700
changeset 563345 cf3e1f896c8a9709cb6115f3cf17f4598d5d7864
parent 563344 969ec8a3c8f23d77271f88a0038babbf348024e1
child 563346 6285a5d92c1c0a0f8d0249945cd5c81d9c105ec7
push id54258
push usercpeterson@mozilla.com
push dateSun, 16 Apr 2017 05:52:14 +0000
reviewersspohl
bugs1356843
milestone55.0a1
Bug 1356843 - Fix -Wcomma warnings in widget/cocoa/nsDeviceContextSpecX.mm. r?spohl clang's -Wcomma warning warns about suspicious use of the comma operator such as between two statements. widget/cocoa/nsDeviceContextSpecX.mm:246:26 [-Wcomma] possible misuse of comma operator here widget/cocoa/nsDeviceContextSpecX.mm:247:32 [-Wcomma] possible misuse of comma operator here MozReview-Commit-ID: GhZQgNemLAE
widget/cocoa/nsDeviceContextSpecX.mm
--- a/widget/cocoa/nsDeviceContextSpecX.mm
+++ b/widget/cocoa/nsDeviceContextSpecX.mm
@@ -238,18 +238,20 @@ NS_IMETHODIMP nsDeviceContextSpecX::EndD
 
 void nsDeviceContextSpecX::GetPaperRect(double* aTop, double* aLeft, double* aBottom, double* aRight)
 {
     NS_OBJC_BEGIN_TRY_ABORT_BLOCK;
 
     PMRect paperRect;
     ::PMGetAdjustedPaperRect(mPageFormat, &paperRect);
 
-    *aTop = paperRect.top, *aLeft = paperRect.left;
-    *aBottom = paperRect.bottom, *aRight = paperRect.right;
+    *aTop = paperRect.top;
+    *aLeft = paperRect.left;
+    *aBottom = paperRect.bottom;
+    *aRight = paperRect.right;
 
     NS_OBJC_END_TRY_ABORT_BLOCK;
 }
 
 already_AddRefed<PrintTarget> nsDeviceContextSpecX::MakePrintTarget()
 {
     double top, left, bottom, right;
     GetPaperRect(&top, &left, &bottom, &right);