Bug 1303411: Add parens to some logic in VideoConduit.cpp, to address clang -Wlogical-op-parentheses build warning. r=dminor
MozReview-Commit-ID: BMq48HVLUH7
--- a/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
+++ b/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
@@ -739,17 +739,17 @@ WebrtcVideoConduit::ConfigureSendMediaCo
mVideoCodecStat->Register(true);
mSendingWidth = 0;
mSendingHeight = 0;
mSendingFramerate = video_codec.maxFramerate;
// See Bug 1297058, enabling FEC when NACK is set on H.264 is problematic
bool use_fec = codecConfig->RtcpFbFECIsSet();
- if (mExternalSendCodec && codecConfig->mType == mExternalSendCodec->mType
+ if ((mExternalSendCodec && codecConfig->mType == mExternalSendCodec->mType)
|| codecConfig->mType == webrtc::kVideoCodecH264) {
if(codecConfig->RtcpFbNackIsSet("")) {
use_fec = false;
}
}
if (use_fec)
{