Chromium Code Reviews

Unified Diff: webrtc/pc/remoteaudiosource.cc

Issue 2988153003: Replace CHECK(x && y) with two separate CHECK() calls (Closed)
Patch Set: fix link failure Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: webrtc/pc/remoteaudiosource.cc
diff --git a/webrtc/pc/remoteaudiosource.cc b/webrtc/pc/remoteaudiosource.cc
index 8d1a5d0fd4caa7072e9868264f56f08ae7c69d67..f6f23c3f24c19a0167a53186b07a8e67a27a0c76 100644
--- a/webrtc/pc/remoteaudiosource.cc
+++ b/webrtc/pc/remoteaudiosource.cc
@@ -96,7 +96,8 @@ bool RemoteAudioSource::remote() const {
}
void RemoteAudioSource::SetVolume(double volume) {
- RTC_DCHECK(volume >= 0 && volume <= 10);
+ RTC_DCHECK_GE(volume, 0);
+ RTC_DCHECK_LE(volume, 10);
for (auto* observer : audio_observers_)
observer->OnSetVolume(volume);
}

Powered by Google App Engine