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 563962 fdb84091f2c2e12932a35918ce1e419c08c36db6
parent 563952 efd3de3df6e15da9cc3bc6d31417f732f39cbf63
child 563963 9200d40aeb5b8314cd4493de5e4abbf9f7d46c08
push id54487
push usercpeterson@mozilla.com
push dateTue, 18 Apr 2017 06:09:29 +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);