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

Unified Diff: webrtc/modules/audio_coding/neteq/audio_decoder_impl.cc

Issue 1928293002: NetEq: Use a BuiltinAudioDecoderFactory to create decoders (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rejigger Opus config Created 4 years, 8 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
Index: webrtc/modules/audio_coding/neteq/audio_decoder_impl.cc
diff --git a/webrtc/modules/audio_coding/neteq/audio_decoder_impl.cc b/webrtc/modules/audio_coding/neteq/audio_decoder_impl.cc
index c9b9b6ae671f5c37941fabe4b903b8a7bfc9cf90..762c3859837172f0e151966c01267c089e0c91a7 100644
--- a/webrtc/modules/audio_coding/neteq/audio_decoder_impl.cc
+++ b/webrtc/modules/audio_coding/neteq/audio_decoder_impl.cc
@@ -137,67 +137,4 @@ int CodecSampleRateHz(NetEqDecoder codec_type) {
}
}
-AudioDecoder* CreateAudioDecoder(NetEqDecoder codec_type) {
- if (!CodecSupported(codec_type)) {
- return NULL;
- }
- switch (codec_type) {
- case NetEqDecoder::kDecoderPCMu:
- return new AudioDecoderPcmU(1);
- case NetEqDecoder::kDecoderPCMa:
- return new AudioDecoderPcmA(1);
- case NetEqDecoder::kDecoderPCMu_2ch:
- return new AudioDecoderPcmU(2);
- case NetEqDecoder::kDecoderPCMa_2ch:
- return new AudioDecoderPcmA(2);
-#ifdef WEBRTC_CODEC_ILBC
- case NetEqDecoder::kDecoderILBC:
- return new AudioDecoderIlbc;
-#endif
-#if defined(WEBRTC_CODEC_ISACFX)
- case NetEqDecoder::kDecoderISAC:
- return new AudioDecoderIsacFix();
-#elif defined(WEBRTC_CODEC_ISAC)
- case NetEqDecoder::kDecoderISAC:
- case NetEqDecoder::kDecoderISACswb:
- return new AudioDecoderIsac();
-#endif
- case NetEqDecoder::kDecoderPCM16B:
- case NetEqDecoder::kDecoderPCM16Bwb:
- case NetEqDecoder::kDecoderPCM16Bswb32kHz:
- case NetEqDecoder::kDecoderPCM16Bswb48kHz:
- return new AudioDecoderPcm16B(1);
- case NetEqDecoder::kDecoderPCM16B_2ch:
- case NetEqDecoder::kDecoderPCM16Bwb_2ch:
- case NetEqDecoder::kDecoderPCM16Bswb32kHz_2ch:
- case NetEqDecoder::kDecoderPCM16Bswb48kHz_2ch:
- return new AudioDecoderPcm16B(2);
- case NetEqDecoder::kDecoderPCM16B_5ch:
- return new AudioDecoderPcm16B(5);
-#ifdef WEBRTC_CODEC_G722
- case NetEqDecoder::kDecoderG722:
- return new AudioDecoderG722;
- case NetEqDecoder::kDecoderG722_2ch:
- return new AudioDecoderG722Stereo;
-#endif
-#ifdef WEBRTC_CODEC_OPUS
- case NetEqDecoder::kDecoderOpus:
- return new AudioDecoderOpus(1);
- case NetEqDecoder::kDecoderOpus_2ch:
- return new AudioDecoderOpus(2);
-#endif
- case NetEqDecoder::kDecoderCNGnb:
- case NetEqDecoder::kDecoderCNGwb:
- case NetEqDecoder::kDecoderCNGswb32kHz:
- case NetEqDecoder::kDecoderCNGswb48kHz:
- RTC_CHECK(false) << "CNG should not be created like this anymore!";
- case NetEqDecoder::kDecoderRED:
- case NetEqDecoder::kDecoderAVT:
- case NetEqDecoder::kDecoderArbitrary:
- default: {
- return NULL;
- }
- }
-}
-
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698