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

Unified Diff: webrtc/modules/audio_coding/neteq/normal.h

Issue 2763273003: Changed OLA window for neteq. Old code didnt work well with 48khz (Closed)
Patch Set: Created 3 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_coding/neteq/normal.h
diff --git a/webrtc/modules/audio_coding/neteq/normal.h b/webrtc/modules/audio_coding/neteq/normal.h
index 23887f5134389591ebdba24f2f6ca2ddf9cca6fd..5a38536ef9edc4aa2253269ccb2e0e32c472a12d 100644
--- a/webrtc/modules/audio_coding/neteq/normal.h
+++ b/webrtc/modules/audio_coding/neteq/normal.h
@@ -39,6 +39,9 @@ class Normal {
decoder_database_(decoder_database),
background_noise_(background_noise),
expand_(expand) {
+ // Setup Overlapp Add to 1ms
+ ola_win_length_ = fs_hz / 1000;
hlundin-webrtc 2017/03/23 11:41:51 Make these two member variables const, and set the
hlundin-webrtc 2017/03/23 11:41:51 rtc::CheckedDivExact(fs_hz_, 1000)
+ ola_win_slope_Q14_ = (1 << 14)/ola_win_length_;
hlundin-webrtc 2017/03/23 11:41:51 Add spaces on both sides of /. I suggest you run "
}
virtual ~Normal() {}
@@ -61,6 +64,9 @@ class Normal {
const BackgroundNoise& background_noise_;
Expand* expand_;
hlundin-webrtc 2017/03/23 11:41:52 No blank line, please.
+ int ola_win_length_;
hlundin-webrtc 2017/03/23 11:41:51 const size_t samples_per_ms_;
+ int16_t ola_win_slope_Q14_;
hlundin-webrtc 2017/03/23 11:41:51 const int16_t default_win_slope_Q14_;
+
RTC_DISALLOW_COPY_AND_ASSIGN(Normal);
};

Powered by Google App Engine
This is Rietveld 408576698