[FFmpeg-soc] [PATCH 1/5] Use av_clip_int16() in FIX15(). Output will differ now in some cases, but the AC3 regtest is unchanged.

Ronald S. Bultje rsbultje at gmail.com
Wed Nov 17 03:58:40 CET 2010


Hi,

On Wed, Nov 17, 2010 at 2:37 AM, Justin Ruggles
<justin.ruggles at gmail.com> wrote:
> +/** convert float in range [-1..1] to int16_t in range [-32768..32767] */
> +#define FIX15(a) ((int16_t)av_clip_int16((int)(a * (float)(1 << 15))))

-32768, so you might want to update the comment. Also, is the cast at
the start needed (int16_t)?

Ronald


More information about the FFmpeg-soc mailing list