fixes issue2449

patch submitted by xvid_fan freenet de

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

This is part of the Great Evil Plan to simplify swscale.

Inline functions are easier to read, maintain, modify and test,
which justifies the slightly increased source size. This patch
also adds support for non-native endianness RGB15/16 and fixes
isSupportedOutput() to no longer claim that we support writing
non-native RGB565/555/444.

This way the code in the file is less cluttered all-over-the-
place.

The 'wfex' is just a Microsoft WaveFormatEx struct.

Make ff_get_wav_header() free existing extradata before allocing a new
buffer.

Inline functions are slightly larger in source code, but
are easier to handle in source code editors. The binary code
generated is the same.

Inline functions are slightly larger in source code, but
are easier to handle in source code editors. The binary code
generated is the same.

This is part of the Great Evil Plan to simplify swscale.

This generates better code on some non-x86 architectures.

This reduces source code size without affecting the binary.

Remove inline keyword from functions that are never inlined.
Use av_always_inline for functions that should be force-inlined
for performance reasons. Use av_cold for init functions.

Remove inline keyword for functions that are only called through
their function pointers (and thus cannot be inlined); add av_cold
keyword to init function, and use av_always_inline instead of
inline for functions that must be inlined for performance reasons.

This fixes a potential double free.
Signed-off-by: Martin Storsjö <martin@martin.st>

This might happen if there was an error before priv_data was allocated
and result in segfault.

This prevents the following compiler warnings: "warning:
initialization from incompatible pointer type". Since the
variables are only ever used in inline assembly, their type
is actually irrelevant (so the part where it was wrong did
not invoke any buggy behaviour).

This way, they look like regular code, which is easier to
understand.

Use of this wrapper was removed in a previous patch, but I
forgot to actually remove the function itself.

This fixes the warning:
libavformat/options.c:62:1: warning: missing braces around initializer [-Wmissing-braces]

Rename it to AVDictionary, since it will be used as such.  Tags
documentation and metadata conversion API is lavf-specific, so remains
there.

Change indentation and whitespace; also move HAVE_YASM blocks.
Signed-off-by: Diego Biurrun <diego@biurrun.de>

The AC-3 encoder unconditionally references some symbols from the E-AC-3
encoder; make those references conditional to fix linking.

read_sequence_header can change width/height; therefore, re-initialize
all tables if width/height changed
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Deprecate av_get_bits_per_sample_fmt(), which was a misnamed function.

For the moment we don't have sample formats with a non-integer number
of bytes, in that case we may need to create a new
av_get_bits_per_sample() function. In the meanwhile we prefer to adopt
this variant, since avoids divisions by 8 all over the place.

The current filter implementation should only have the cx coefficients
divided by gain in order to give the correct output scale.