Chromium Code Reviews

Unified Diff: webrtc/modules/audio_coding/main/test/TestStereo.cc

Issue 1406123011: Let AudioCodingModule::SendCodec return Maybe<CodecInst> (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: review fix Created 5 years, 1 month 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/modules/audio_coding/main/test/TestStereo.cc
diff --git a/webrtc/modules/audio_coding/main/test/TestStereo.cc b/webrtc/modules/audio_coding/main/test/TestStereo.cc
index 69cc3272bb0df8e2cada93637c5e6d2b4072c662..bb38fac7388ab5651582fc33c472ebcf7b1bb76e 100644
--- a/webrtc/modules/audio_coding/main/test/TestStereo.cc
+++ b/webrtc/modules/audio_coding/main/test/TestStereo.cc
@@ -823,14 +823,15 @@ void TestStereo::OpenOutFile(int16_t test_number) {
}
void TestStereo::DisplaySendReceiveCodec() {
- CodecInst my_codec_param;
- acm_a_->SendCodec(&my_codec_param);
+ auto send_codec = acm_a_->SendCodec();
if (test_mode_ != 0) {
- printf("%s -> ", my_codec_param.plname);
+ ASSERT_TRUE(send_codec);
+ printf("%s -> ", send_codec->plname);
}
- acm_b_->ReceiveCodec(&my_codec_param);
+ CodecInst receive_codec;
+ acm_b_->ReceiveCodec(&receive_codec);
if (test_mode_ != 0) {
- printf("%s\n", my_codec_param.plname);
+ printf("%s\n", receive_codec.plname);
}
}
« no previous file with comments | « webrtc/modules/audio_coding/main/test/TestAllCodecs.cc ('k') | webrtc/modules/audio_coding/main/test/TestVADDTX.cc » ('j') | no next file with comments »

Powered by Google App Engine