The later may yield incorrect code for on-stack variables.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

The later may yield incorrect code for on-stack variables.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

The later may yield incorrect code for on-stack variables.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This is the remaining error, the output on the SPX samples,
respectively csi_miami_stereo_128_spx.eac3 and
csi_miami_5.1_256_spx.eac3, goes from:
stddev:    8.71 PSNR: 77.52 MAXDIFF:  235
stddev:24270.51 PSNR: 22.17 MAXDIFF:47166
to:
stddev:    0.12 PSNR:114.12 MAXDIFF:    1
stddev:    0.12 PSNR:114.73 MAXDIFF:    1
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

It was set to 1 instead of sqrt(3)
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Should also improve decoding, but actually doesn't...
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

When the decoder is deactivated, the x86-optimized versions are
not compiled, resulting in a link error.

The C version is unaffected, as it is part of the idctdsp
subsystem.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Also reduce the table duplication with SSE2 code, remove duplicated
macro parameters.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

The AC3DecodeContext has a float (USE_FIXED=0) and an integer
(USE_FIXED=1) variant, both of which can be present in the same binary.
This is not only very confusing, but it also breaks horribly, when one
variant is used by code expecting the other.

This currently happens, because eac3dec.c is only compiled for the float
variant, but also used from ac3dec_fixed.c, which uses the integer
variant.

The result is memory corruption, leading to crashes.

So compile eac3dec.c once for each variant and adapt it, so that it
works with the integer variant.

A loss of precission and scaling bug has been fixed by the committer
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes assertion failure and race conditions
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>

No global variables are used and the VLC tables are allocated without
static elements. This will allow using a JPEG decoding context within
other decoders.

This field is designed for marking codec properties useful to lavc internals.
Two internal capabilities are added:
 - FF_CODEC_CAP_INIT_THREADSAFE: codec can be opened without locks;
 - FF_CODEC_CAP_INIT_CLEANUP: codec frees memory if initialization fails.

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

The main difference consists in renaming properly labels, and
letting yasm select the gprs for skipping 1D transforms.
Previous-version-reviewed-by: NJames Almer <jamrial@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Also add av_cold to ff_hevcdsp_init_arm.
Signed-off-by: NJames Almer <jamrial@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Reviewed-by: N"Ronald S. Bultje" <rsbultje@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Reviewed-by: N"Ronald S. Bultje" <rsbultje@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

So far it is only set in roq_encode_frame, but it is used in
roq_encode_end to free the coded_frame. This currently segfaults if
roq_encode_frame is not called between roq_encode_init and
roq_encode_end.

CC:libav-stable@libav.org
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Also moves the shift out of the inner loop

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This also simplifies the code

Found-by: Clang -fsanitize=shift
Reported-by: NThierry Foucu <tfoucu@google.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

The fourth substream is being discarded, since its not raw audio data,
but an encoded Atmos stream which needs a specialized decoder.

Fixes decoding of the true hd stream from Transformers\ -\ Age\ of\ Extinction\ 2014\ 1080P-003.mkv
Signed-off-by: NAnton Khirnov <anton@khirnov.net>