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

Unified Diff: webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c

Issue 2746903002: iSAC fix entropy coder: Recently added DCHECK could in fact trigger (Closed)
Patch Set: don't let new function be caught by #ifdefs 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
« no previous file with comments | « webrtc/common_audio/signal_processing/include/spl_inl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c
diff --git a/webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c b/webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c
index 10e05bc31a2e08e759f2d9fb27a4eb2699af57d5..e4d399b61ff4fa1b3deb00aac8671e36f2f760fe 100644
--- a/webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c
+++ b/webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c
@@ -234,8 +234,7 @@ static void CalcInvArSpec(const int16_t *ARCoefQ12,
for (n = 0; n < FRAMESAMPLES/8; n++) {
const int64_t p =
(WebRtcIsacfix_kCos[k][n] * (int64_t)CorrQ11[k + 1] + 2) >> 2;
- RTC_DCHECK_EQ(p, (int32_t)p); // p fits in 32 bits
- CurveQ16[n] += (int32_t)p;
+ CurveQ16[n] += WebRtcSpl_SatW64ToW32(p);
}
}
« no previous file with comments | « webrtc/common_audio/signal_processing/include/spl_inl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698