generic-library/webrtc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Disables SW AEC when built-in AEC is enabled

Browse Source 
As of r7849 the built-in AEC on devicing supporting it is enabled by default.
Unfortunately, the SW AEC (AECM) was not disabled, hence running on top of the built-in one. This is not necessary. In fact it reduce double talk performance significantly.

BUG=4431
TESTED=manually
R=henrika@webrtc.org, mflodman@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/49419004

Cr-Commit-Position: refs/heads/master@{#8735}
git-svn-id: http://webrtc.googlecode.com/svn/trunk@8735 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
bjornv@webrtc.org 2015-03-16 14:22:03 +00:00
parent 8bd2f40a8c
commit 3f11823a1a
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
talk/media/webrtc/webrtcvoiceengine.cc
View File

@ -824,6 +824,7 @@ bool WebRtcVoiceEngine::ApplyOptions(const AudioOptions& options_in) {
// Disable internal software EC if device has its own built-in EC,
// i.e., replace the software EC with the built-in EC.
options.echo_cancellation.Set(false);
echo_cancellation = false;
LOG(LS_INFO) << "Disabling EC since built-in EC will be used instead";
}
}