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

Unified Diff: webrtc/modules/audio_coding/main/include/audio_coding_module.h

Issue 1425133002: ACM/NetEq: Restructure how post-decode VAD is enabled (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@acm-recv-nack
Patch Set: Actually doing something with the new config Created 5 years, 1 month 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/main/include/audio_coding_module.h
diff --git a/webrtc/modules/audio_coding/main/include/audio_coding_module.h b/webrtc/modules/audio_coding/main/include/audio_coding_module.h
index b145cf423e594a34b36b5c9feaa03812fce34fe1..94b1271dd9c009867d701bdca459c8a136f01223 100644
--- a/webrtc/modules/audio_coding/main/include/audio_coding_module.h
+++ b/webrtc/modules/audio_coding/main/include/audio_coding_module.h
@@ -60,7 +60,11 @@ class AudioCodingModule {
public:
struct Config {
- Config() : id(0), neteq_config(), clock(Clock::GetRealTimeClock()) {}
+ Config() : id(0), neteq_config(), clock(Clock::GetRealTimeClock()) {
+ // Post-decode VAD is disabled by default in NetEq, however, Audio
+ // Conference Mixer relies on VAD decisions and fails without them.
+ neteq_config.enable_post_decode_vad = true;
+ }
int id;
NetEq::Config neteq_config;
« no previous file with comments | « webrtc/modules/audio_coding/main/acm2/acm_receiver.cc ('k') | webrtc/modules/audio_coding/neteq/include/neteq.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698