On Wed, Nov 22, 2017 at 11:38 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2017-11-22 21:00 GMT+01:00 Michael Niedermayer <mich...@niedermayer.cc>: > >> diff --git a/libavcodec/sbrdsp_fixed.c b/libavcodec/sbrdsp_fixed.c >> index a0ef6859f1..0db932a105 100644 >> --- a/libavcodec/sbrdsp_fixed.c >> +++ b/libavcodec/sbrdsp_fixed.c >> @@ -133,7 +133,7 @@ static av_always_inline SoftFloat autocorr_calc(int64_t >> accu) >> >> round = 1U << (nz-1); >> mant = (int)((accu + round) >> nz); >> - mant = (mant + 0x40)>>7; >> + mant = (mant + 0x40ll)>>7; > > LL? >
More correctly, shouldnt this use one of those fancy integer constant macros, like INT64_C(0x40)? (I don't actually know if those are supposed to work with hex constants, but the fact that they exist seems to indicate that LL is not entirely portable) - Hendrik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
