The mingw win32 atomics appear to be faulty, so they should not be used
if the gcc ones are available.
Signed-off-by: Martin Storsjö <martin@martin.st>

Fixes video playback in applications still using reordered_opaque.

Return an error on incomplete frames.

Fixes a memleak due to avcodec_free_frame not freeing the frame data.

planes - FF_ARRAY_ELEMS would be evaluated as unsigned and underflow
instead of being negative as was intended.

Avoid buffer overflow in buffer_needs_copy()
Signed-off-by: Xi Wang <xi.wang@gmail.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

SDL 2 is API incompatible.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

On the current code, armcc will fail with:
"libavutil/atomic_gcc.h", line 52: Error:  #2771: first argument must be
a pointer to integer or enumeration type

Fixes valgrind uninitialized memory errors when decoding png.

CC:libav-stable@libav.org

CC:libav-stable@libav.org

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

Print an error and abort when the option is of the wrong type (decoding
for output file or vice versa), since this could never be correct for
any input or output configuration.

Print a warning and continue when the option is of the correct type,
just unused, so same commandlines can be reused for different kinds of
input or output files.

The -{vas}<option> syntax is now obsolete and deprecated, and has never
been documented or encouraged.

It is an encoding option, it has no effect when applied to decoding.

This is shorter and more readable.

This is shorter and more readable.

I.e. input options to output files or vice versa.

Not all gcc configurations have an implementation of all the atomic
operations, and some gcc configurations have some atomic builtins
implemented but not all.

Thus check for the most essential function, whose presence should
indicate that all others are present as well, since it can be used
to implement all the other ones.
Signed-off-by: Martin Storsjö <martin@martin.st>

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

This makes them pass standalone compilation tests. Previously,
they included atomic.h which included themselves again, leading to
double definitions.
Signed-off-by: Martin Storsjö <martin@martin.st>

Fixes build without threads after
759001c5.

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