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

Unified Diff: webrtc/modules/audio_processing/ns/noise_suppression_x.c

Issue 1821443003: Fix normalization of noise estimate in NoiseSuppressor (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Combine in a function Created 4 years, 9 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_processing/ns/noise_suppression_x.c
diff --git a/webrtc/modules/audio_processing/ns/noise_suppression_x.c b/webrtc/modules/audio_processing/ns/noise_suppression_x.c
index efe8a5bf2b5d2134172925cc6cae93a8ff6f8546..011b7d12bc5ad72610e2e9933fa193ba1efc5773 100644
--- a/webrtc/modules/audio_processing/ns/noise_suppression_x.c
+++ b/webrtc/modules/audio_processing/ns/noise_suppression_x.c
@@ -45,11 +45,14 @@ void WebRtcNsx_Process(NsxHandle* nsxInst,
num_bands, outFrame);
}
-const uint32_t* WebRtcNsx_noise_estimate(const NsxHandle* nsxInst) {
+const uint32_t* WebRtcNsx_noise_estimate(const NsxHandle* nsxInst,
+ int* domain) {
+ *domain = 11;
const NoiseSuppressionFixedC* self = (const NoiseSuppressionFixedC*)nsxInst;
if (nsxInst == NULL || self->initFlag == 0) {
return NULL;
}
+ *domain += self->prevQNoise;
peah-webrtc 2016/03/31 20:50:57 What is the Q value of prevNoiseU32? From the name
aluebs-webrtc 2016/04/01 01:52:42 Yes, I agree that it is confusing, but it is in fa
return self->prevNoiseU32;
}

Powered by Google App Engine
This is Rietveld 408576698