It does not need to be unsigned.

It is already unconditionally set to 0 prior to this, so we can modify it
only when needed.

x_modified is just unnecessary, and final_val can be removed by simplifying
the unsigned-to-signed conversion.

avoids some unnecessary arithmetic in certain situations

reduces the number of parameters to decode_scalar() and slightly simplifies
the code

Use the ALACContext fields directly instead.

It is used to store a value read by avio_rb64().

according to the CAF specification:
"... the value for mChunkSize can be greater than the actual valid content
of the packet table chunk"

This provides a more precise identification of the compiler used.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Reversing the lpc coefficient order simplifies indexing in
the filter.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Signed-off-by: Mans Rullgard <mans@mansr.com>

This moves all VP3-specific function pointers from dsputil to a
new vp3dsp context.  There is no reason to ever use the VP3 IDCT
where an MPEG2 IDCT is expected or vice versa.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Signed-off-by: Mans Rullgard <mans@mansr.com>

It turns out that the reference decoder subtracts 128 from DC during block
decode but adds it back during reordering block with zigzag pattern.
Transforming block with incorrect DC caused heavy visual artifacts for
many quantisers.

Based on FFmpeg version from
commit 713a7854Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

Signed-off-by: Martin Storsjö <martin@martin.st>

Signed-off-by: Martin Storsjö <martin@martin.st>

Also exit in such a case if -xerror is used.

Fixes bug 329.

This fixes decoding some TSCC2 files with large quantisers.