Retyped the frequency estimate of the comfort noise for the higher band to harmonize the AEC code(#4

webrtc/modules/audio_processing/aec/aec_core_mips.c

Index: webrtc/modules/audio_processing/aec/aec_core_mips.c
diff --git a/webrtc/modules/audio_processing/aec/aec_core_mips.c b/webrtc/modules/audio_processing/aec/aec_core_mips.c
index dfd4dc606a009853450809d3a60333ed59ae58c3..7d64c2a836d10774f6075e5d92487ec0d994bfc1 100644
--- a/webrtc/modules/audio_processing/aec/aec_core_mips.c
+++ b/webrtc/modules/audio_processing/aec/aec_core_mips.c
@@ -26,7 +26,7 @@ extern const float WebRtcAec_overDriveCurve[65];
 
 void WebRtcAec_ComfortNoise_mips(AecCore* aec,
                                  float efw[2][PART_LEN1],
-                                 complex_t* comfortNoiseHband,
+                                 float comfortNoiseHband[2][PART_LEN1],
                                  const float* noisePow,
                                  const float* lambda) {
   int i, num;
@@ -274,7 +274,7 @@ void WebRtcAec_ComfortNoise_mips(AecCore* aec,
   noiseAvg = 0.0;
   tmpAvg = 0.0;
   num = 0;
-  if ((aec->sampFreq == 32000 || aec->sampFreq == 48000) && flagHbandCn == 1) {
+  if (aec->num_bands > 1 && flagHbandCn == 1) {

[minyue-webrtc, 2015/12/04 09:49:44]

Interesting, MIPS was different here, was there a reason?

[peah-webrtc, 2015/12/04 23:02:48]

I think it is just an error. As far as I know, num_bands are only greater than
one when the sample rate is larger than 16000.  So as long as the native sample
rates are used, the statements are equivalent.

Anyway, there cannot be any reason for these functions to behave differently.

     for (i = 0; i < PART_LEN; i++) {
       rand[i] = ((float)randW16[i]) / 32768;
     }
@@ -314,10 +314,14 @@ void WebRtcAec_ComfortNoise_mips(AecCore* aec,
 
     for (i = 0; i < PART_LEN1; i++) {
       // Use average NLP weight for H band
-      comfortNoiseHband[i][0] = tmpAvg * u[i][0];
-      comfortNoiseHband[i][1] = tmpAvg * u[i][1];
+      comfortNoiseHband[0][i] = tmpAvg * u[i][0];
+      comfortNoiseHband[1][i] = tmpAvg * u[i][1];
     }
   }
+  else {

[minyue-webrtc, 2015/12/04 09:49:44]

321 to 320

[peah-webrtc, 2015/12/04 23:02:48]

Done.

+    memset(comfortNoiseHband, 0,
+           2 * PART_LEN1 * sizeof(comfortNoiseHband[0][0]));

[hlundin-webrtc, 2015/12/04 11:01:39]

... and sizeof(comfortNoiseHband)

[peah-webrtc, 2015/12/04 23:02:48]

As stated in the other place for this, that does not work, the compiler
complains. N

[hlundin-webrtc, 2015/12/08 11:52:52]

Acknowledged.

+  }
 }
 
 void WebRtcAec_FilterFar_mips(