Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(656)

Unified Diff: webrtc/modules/audio_coding/acm2/audio_coding_module_unittest.cc

Issue 2389263003: Fix lint errors in audio_coding_module_unittest.cc (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/audio_coding/acm2/audio_coding_module_unittest.cc
diff --git a/webrtc/modules/audio_coding/acm2/audio_coding_module_unittest.cc b/webrtc/modules/audio_coding/acm2/audio_coding_module_unittest.cc
index 1843f0d1e8a89288d833abc487893b5bab4da500..b6fcd01b63a0df8c8abbd85475de89e79ba7c5f6 100644
--- a/webrtc/modules/audio_coding/acm2/audio_coding_module_unittest.cc
+++ b/webrtc/modules/audio_coding/acm2/audio_coding_module_unittest.cc
@@ -1051,7 +1051,7 @@ TEST_F(AcmReceiverBitExactnessOldApi, 48kHzOutputExternalDecoder) {
// public.
class DecodeForwarder {
public:
- DecodeForwarder(AudioDecoder* decoder) : decoder_(decoder) {}
+ explicit DecodeForwarder(AudioDecoder* decoder) : decoder_(decoder) {}
int Decode(const uint8_t* encoded,
size_t encoded_len,
int sample_rate_hz,
@@ -1550,8 +1550,7 @@ TEST_F(AcmSetBitRateOldApi, Opus_48khz_20ms_10kbps) {
Run(10000, 9288);
#else
Run(10000, 9024);
-#endif // WEBRTC_ANDROID
-
+#endif // WEBRTC_ANDROID
}
TEST_F(AcmSetBitRateOldApi, Opus_48khz_20ms_50kbps) {
@@ -1560,7 +1559,7 @@ TEST_F(AcmSetBitRateOldApi, Opus_48khz_20ms_50kbps) {
Run(50000, 47960);
#else
Run(50000, 49544);
-#endif // WEBRTC_ANDROID
+#endif // WEBRTC_ANDROID
}
// The result on the Android platforms is inconsistent for this test case.
@@ -1642,7 +1641,7 @@ TEST_F(AcmChangeBitRateOldApi, Opus_48khz_20ms_10kbps) {
Run(10000, 32200, 5176);
#else
Run(10000, 32200, 5456);
-#endif // WEBRTC_ANDROID
+#endif // WEBRTC_ANDROID
}
TEST_F(AcmChangeBitRateOldApi, Opus_48khz_20ms_50kbps) {
@@ -1651,7 +1650,7 @@ TEST_F(AcmChangeBitRateOldApi, Opus_48khz_20ms_50kbps) {
Run(50000, 32200, 24768);
#else
Run(50000, 32200, 24848);
-#endif // WEBRTC_ANDROID
+#endif // WEBRTC_ANDROID
}
TEST_F(AcmChangeBitRateOldApi, Opus_48khz_20ms_100kbps) {
@@ -1661,10 +1660,10 @@ TEST_F(AcmChangeBitRateOldApi, Opus_48khz_20ms_100kbps) {
Run(100000, 32200, 51152);
#else
Run(100000, 32200, 51248);
- #endif // WEBRTC_ARCH_ARM64
+ #endif // WEBRTC_ARCH_ARM64
#else
Run(100000, 32200, 50584);
-#endif // WEBRTC_ANDROID
+#endif // WEBRTC_ANDROID
}
// These next 2 tests ensure that the SetBitRate function has no effect on PCM
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698