zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Correcting the check for the return code produced by

Browse Source 
AudioProcessing::ProcessReverseStream().
Before the change, only -1 was considered to be an error.

Allthough the error code scheme for AudioProcessing definitely
could be discussed, the current scheme have many error codes that
differ from -1 and thus were not caught by the old code.

BUG=webrtc:5237

Review URL: https://codereview.webrtc.org/1515073004

Cr-Commit-Position: refs/heads/master@{#11003}
This commit is contained in:
peah
2015-12-13 23:02:33 -08:00
committed by Commit bot
parent 1d5c19d23e
commit e5ae6f8237
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
webrtc/voice_engine/output_mixer.cc
View File

@ -550,7 +550,7 @@ OutputMixer::DoOperationsOnCombinedSignal(bool feed_data_to_apm)
// ----------------------------------------------------------------------------
void OutputMixer::APMProcessReverseStream() {
if (_audioProcessingModulePtr->ProcessReverseStream(&_audioFrame) == -1) {
if (_audioProcessingModulePtr->ProcessReverseStream(&_audioFrame) != 0) {
WEBRTC_TRACE(kTraceError, kTraceVoice, VoEId(_instanceId, -1),
"AudioProcessingModule::ProcessReverseStream() => error");
}