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
--- 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