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

Side by Side 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, 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 27 matching lines...) Expand all
38 } 38 }
39 39
40 void WebRtcNsx_Process(NsxHandle* nsxInst, 40 void WebRtcNsx_Process(NsxHandle* nsxInst,
41 const short* const* speechFrame, 41 const short* const* speechFrame,
42 int num_bands, 42 int num_bands,
43 short* const* outFrame) { 43 short* const* outFrame) {
44 WebRtcNsx_ProcessCore((NoiseSuppressionFixedC*)nsxInst, speechFrame, 44 WebRtcNsx_ProcessCore((NoiseSuppressionFixedC*)nsxInst, speechFrame,
45 num_bands, outFrame); 45 num_bands, outFrame);
46 } 46 }
47 47
48 const uint32_t* WebRtcNsx_noise_estimate(const NsxHandle* nsxInst) { 48 const uint32_t* WebRtcNsx_noise_estimate(const NsxHandle* nsxInst,
49 int* domain) {
50 *domain = 11;
49 const NoiseSuppressionFixedC* self = (const NoiseSuppressionFixedC*)nsxInst; 51 const NoiseSuppressionFixedC* self = (const NoiseSuppressionFixedC*)nsxInst;
50 if (nsxInst == NULL || self->initFlag == 0) { 52 if (nsxInst == NULL || self->initFlag == 0) {
51 return NULL; 53 return NULL;
52 } 54 }
55 *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
53 return self->prevNoiseU32; 56 return self->prevNoiseU32;
54 } 57 }
55 58
56 size_t WebRtcNsx_num_freq() { 59 size_t WebRtcNsx_num_freq() {
57 return HALF_ANAL_BLOCKL; 60 return HALF_ANAL_BLOCKL;
58 } 61 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698