[FFmpeg-devel] [PATCH][RFC] Lagarith Decoder.

[Reimar Döffinger]

On Mon, Aug 10, 2009 at 11:42:19PM -0600, Nathan Caldwell wrote:
> +    scaled_cumul_prob = cumulative_target - scaled_cumul_prob;
> +
> +    if (scaled_cumul_prob > INT_MAX) {

While FFmpeg requires it in a few places, I think this is ugly code
because it assumes two's complement without good reason.
If speed is critical I guess this should be a debug-only assert,
otherwise just
if (scaled_cumul_prob > cumulative_target)
with those code lines swapped of course.