Bug 1356843 - Fix -Wcomma warning in netwerk/protocol/http/Http2Stream.cpp. r=jhao draft
authorChris Peterson <cpeterson@mozilla.com>
Mon, 27 Mar 2017 21:42:21 -0700
changeset 563951 2caaedd75d43b6ce4364d01d75823c43c621e82a
parent 563950 d25658a39026821bf0b455967872001fb4816da5
child 563952 efd3de3df6e15da9cc3bc6d31417f732f39cbf63
push id54485
push usercpeterson@mozilla.com
push dateTue, 18 Apr 2017 05:57:33 +0000
reviewersjhao
bugs1356843
milestone55.0a1
Bug 1356843 - Fix -Wcomma warning in netwerk/protocol/http/Http2Stream.cpp. r=jhao clang's -Wcomma warning warns about suspicious use of the comma operator such as between two statements. netwerk/protocol/http/Http2Stream.cpp:436:59 [-Wcomma] possible misuse of comma operator here MozReview-Commit-ID: AsmNPTjKlYB
netwerk/protocol/http/Http2Stream.cpp
--- a/netwerk/protocol/http/Http2Stream.cpp
+++ b/netwerk/protocol/http/Http2Stream.cpp
@@ -428,17 +428,18 @@ Http2Stream::ParseHttpRequestHeaders(con
     MOZ_ASSERT(false);
     return rv;
   }
 
   nsAutoCString requestURI;
   head->RequestURI(requestURI);
 
   mozilla::OriginAttributes originAttributes;
-  mSocketTransport->GetOriginAttributes(&originAttributes),
+  mSocketTransport->GetOriginAttributes(&originAttributes);
+
   CreatePushHashKey(nsDependentCString(head->IsHTTPS() ? "https" : "http"),
                     authorityHeader, originAttributes, mSession->Serial(),
                     requestURI,
                     mOrigin, hashkey);
 
   // check the push cache for GET
   if (head->IsGet()) {
     // from :scheme, :authority, :path
@@ -1546,11 +1547,10 @@ Http2Stream::Finish0RTT(bool aRestart, b
     nsHttpTransaction *trans = mTransaction->QueryHttpTransaction();
     if (trans) {
       trans->Refused0RTT();
     }
   }
   return rv;
 }
 
-
 } // namespace net
 } // namespace mozilla