Remove Plan 9 support

Supporting the system was a nice joke for the 9 release, but it has
run its course. Nowadays Plan 9 receives no testing and has no
practical usefulness.

build: Drop support for old versions of libdc1394

The libdc1394 API transition was finished close to a decade ago.

build: Add separate flags for checkheaders targets

This allows filtering out some warning noise.

build: Add -Wpedantic and -Wextra to extra warning flags

Also drop -Winline from set of warning flags as it only produces noise.

mpeg4audio: correctly propagate meaningful error values

Signed-off-by: James Almer <jamrial@gmail.com>

mmaldec: initialize refcount using atomic_init()

This is how we initialize refcount in libavutil/buffer.c.

Signed-off-by: Wan-Teh Chang <wtc@google.com>
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>

configure: Be more helpful when requiring options set

Be explicit about which options should be set.

examples: fix a typo in an error message

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

lavu: Document the color properties enumeration values origin

options_table: Do not rely on enum size as option bound

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

lavc: Use a stricter check for the color properties values

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

cabac: x86: Give optimizations header a more meaningful name

aarch64: vp9itxfm: Skip empty slices in the first pass of idct_idct 16x16 and 32x32

This work is sponsored by, and copyright, Google.

Previously all subpartitions except the eob=1 (DC) case ran with
the same runtime:

vp9_inv_dct_dct_16x16_sub16_add_neon:   1373.2
vp9_inv_dct_dct_32x32_sub32_add_neon:   8089.0

By skipping individual 8x16 or 8x32 pixel slices in the first pass,
we reduce the runtime of these functions like this:

vp9_inv_dct_dct_16x16_sub1_add_neon:     235.3
vp9_inv_dct_dct_16x16_sub2_add_neon:    1036.7
vp9_inv_dct_dct_16x16_sub4_add_neon:    1036.7
vp9_inv_dct_dct_16x16_sub8_add_neon:    1036.7
vp9_inv_dct_dct_16x16_sub12_add_neon:   1372.1
vp9_inv_dct_dct_16x16_sub16_add_neon:   1372.1
vp9_inv_dct_dct_32x32_sub1_add_neon:     555.1
vp9_inv_dct_dct_32x32_sub2_add_neon:    5190.2
vp9_inv_dct_dct_32x32_sub4_add_neon:    5180.0
vp9_inv_dct_dct_32x32_sub8_add_neon:    5183.1
vp9_inv_dct_dct_32x32_sub12_add_neon:   6161.5
vp9_inv_dct_dct_32x32_sub16_add_neon:   6155.5
vp9_inv_dct_dct_32x32_sub20_add_neon:   7136.3
vp9_inv_dct_dct_32x32_sub24_add_neon:   7128.4
vp9_inv_dct_dct_32x32_sub28_add_neon:   8098.9
vp9_inv_dct_dct_32x32_sub32_add_neon:   8098.8

I.e. in general a very minor overhead for the full subpartition case due
to the additional cmps, but a significant speedup for the cases when we
only need to process a small part of the actual input data.

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

arm: vp9itxfm: Skip empty slices in the first pass of idct_idct 16x16 and 32x32

This work is sponsored by, and copyright, Google.

Previously all subpartitions except the eob=1 (DC) case ran with
the same runtime:

                                     Cortex A7       A8       A9      A53
vp9_inv_dct_dct_16x16_sub16_add_neon:   3188.1   2435.4   2499.0   1969.0
vp9_inv_dct_dct_32x32_sub32_add_neon:  18531.7  16582.3  14207.6  12000.3

By skipping individual 4x16 or 4x32 pixel slices in the first pass,
we reduce the runtime of these functions like this:

vp9_inv_dct_dct_16x16_sub1_add_neon:     274.6    189.5    211.7    235.8
vp9_inv_dct_dct_16x16_sub2_add_neon:    2064.0   1534.8   1719.4   1248.7
vp9_inv_dct_dct_16x16_sub4_add_neon:    2135.0   1477.2   1736.3   1249.5
vp9_inv_dct_dct_16x16_sub8_add_neon:    2446.7   1828.7   1993.6   1494.7
vp9_inv_dct_dct_16x16_sub12_add_neon:   2832.4   2118.3   2266.5   1735.1
vp9_inv_dct_dct_16x16_sub16_add_neon:   3211.7   2475.3   2523.5   1983.1
vp9_inv_dct_dct_32x32_sub1_add_neon:     756.2    456.7    862.0    553.9
vp9_inv_dct_dct_32x32_sub2_add_neon:   10682.2   8190.4   8539.2   6762.5
vp9_inv_dct_dct_32x32_sub4_add_neon:   10813.5   8014.9   8518.3   6762.8
vp9_inv_dct_dct_32x32_sub8_add_neon:   11859.6   9313.0   9347.4   7514.5
vp9_inv_dct_dct_32x32_sub12_add_neon:  12946.6  10752.4  10192.2   8280.2
vp9_inv_dct_dct_32x32_sub16_add_neon:  14074.6  11946.5  11001.4   9008.6
vp9_inv_dct_dct_32x32_sub20_add_neon:  15269.9  13662.7  11816.1   9762.6
vp9_inv_dct_dct_32x32_sub24_add_neon:  16327.9  14940.1  12626.7  10516.0
vp9_inv_dct_dct_32x32_sub28_add_neon:  17462.7  15776.1  13446.2  11264.7
vp9_inv_dct_dct_32x32_sub32_add_neon:  18575.5  17157.0  14249.3  12015.1

I.e. in general a very minor overhead for the full subpartition case due
to the additional loads and cmps, but a significant speedup for the cases
when we only need to process a small part of the actual input data.

In common VP9 content in a few inspected clips, 70-90% of the non-dc-only
16x16 and 32x32 IDCTs only have nonzero coefficients in the upper left
8x8 or 16x16 subpartitions respectively.

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

arm: vp9itxfm: Only reload the idct coeffs for the iadst_idct combination

This avoids reloading them if they haven't been clobbered, if the
first pass also was idct.

This is similar to what was done in the aarch64 version.

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

vp9dsp: add DC only versions for idct/idct.

before:

time ./avconv -v 0 -nostats -threads 1 -i sintel_vp9_500kbps.webm -f null -
real    0m11.125s
user    0m11.059s
sys     0m0.050s

time ./avconv -v 0 -nostats -threads 1 -i sintel_vp9_500kbps.webm -f null -
real    0m10.944s
user    0m10.819s
sys     0m0.064s

after:

time ./avconv -v 0 -nostats -threads 1 -i sintel_vp9_500kbps.webm -f null -
real    0m8.153s
user    0m8.034s
sys     0m0.050s

time ./avconv -v 0 -nostats -threads 1 -i sintel_vp9_500kbps.webm -f null -
real    0m8.038s
user    0m7.980s
sys     0m0.039s

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

hevc: Eliminate pointless variable indirection

hevc: Drop pointless av_unused attribute

metasound: Drop unused tables

configure: Integrate X11 checks into vaapi/vdpau checks

configure: Do not add newlines in filter()/filter_out() functions

configure: Move hardware-accelerated codec deps out of hwaccel section

configure: MMAL-related decoders should depend on, not select, mmal

mjpegdec: Check return values of functions that may fail

dxva2: Adjust printf length modifiers where appropriate

avisynth: Cast to the right type when loading avisynth library functions

Fixes a number of related warnings.

lavc: move decoding-related code from utils.c to a new file

lavc: move encoding-related code from utils.c to a new file

aac_adtstoasc_bsf: validate and forward extradata if the stream is already ASC

Fixes AAC AudioSpecificConfig passthrough.

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

mss2: only use error correction for matching block counts

This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2
with coded_width/coded_height larger than width/height.

Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

avconv: Fix the audio next dts computation

Use the correct timebase.

CC: libav-stable@libav.org

ac3enc: Avoid unnecessary macro indirections

ac3enc: Reshuffle functions to avoid forward declarations

ac3enc: Reshuffle some float/fixed-mode ifdefs to avoid a dummy function

hwcontext_vaapi: Don't abort on failing to allocate from a fixed-size pool

tta: avoid undefined shifts

Signed-off-by: Diego Biurrun <diego@biurrun.de>

tta: use get_unary() instead of a custom implementation

Signed-off-by: Diego Biurrun <diego@biurrun.de>

build: Drop gcrypt support

GnuTLS in combination with gcrypt has been deprecated since 2010.

configure: Use correct libm linker flag during math function checks

configure: Add missing asyncts filter, movie filter, and output example deps

Also add a missing avcodec.h #include in the movie filter.

configure: Use correct variable name in libsnappy test

configure: Remove old avisynth support leftover

arm: warn/error on movrelx usage problematic with PIC on ELF

The warning has false positives but our asm does not trigger it. For
new code false positives can only be avoided by changing the register
allocation.

configure: Disable warning C4703 with MSVC

This disables warnings about potentially uninitialized local pointer
variables.  Disabling the warning is in line with what we do for gcc.

w32pthreads: Fix function pointer casts

This eliminates a handful of warnings at every inclusion of the header.

qt-faststart: Do not try to use fancy 64-bit seeking functions on mingw32ce

These functions are not available on mingw32ce.

Signed-off-by: Diego Biurrun <diego@biurrun.de>

rtmpdh: Do global initialization before running the test

The rtmpdh code can use crypto libraries which may require
a process global init. (gcrypt is one of the libraries
where the rtmpdh test code can fail if global init hasn't been
done, depending on gcrypt version.)

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

aarch64: vp9itxfm: Don't repeatedly set x9 when nothing overwrites it

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

rdt: Convert to the new bitstream reader

ogg: Convert to the new bitstream reader

mpegts: Convert to the new bitstream reader

xsubdec: Convert to the new bitstream reader

xan: Convert to the new bitstream reader

wnv1: Convert to the new bitstream reader

vima: Convert to the new bitstream reader

vble: Convert to the new bitstream reader

utvideodec: Convert to the new bitstream reader

twinvq: Convert to the new bitstream reader

tscc2: Convert to the new bitstream reader

truespeech: Convert to the new bitstream reader

tiertex: Convert to the new bitstream reader

truemotion2: Convert to the new bitstream reader

svq1dec: Convert to the new bitstream reader

smacker: Convert to the new bitstream reader

sipr: Convert to the new bitstream reader

rtjpeg: Convert to the new bitstream reader

ra288: Convert to the new bitstream reader

ra144: Convert to the new bitstream reader

arm: vp9itxfm: Rename a macro parameter to fit better

Since the same parameter is used for both input and output,
the name inout is more fitting.

This matches the naming used below in the dmbutterfly macro.

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

arm/aarch64: vp9itxfm: Fix indentation of macro arguments

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

checkasm: vp9dsp: benchmark all sub-IDCTs (but not WHT or ADST).

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

Revert "checkasm: vp9dsp: Benchmark the dc-only version of idct_idct separately"

This reverts commit 81d7f0bbca837afda1f7e60d3ae52ab1360ab44b.

Instead of just benchmarking dc separately, test all relevant subparts
(in the next commit).

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

matroska: use av_stream_add_side_data() for stereo3d side data

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>

mov: Use av_stream_add_side_data() for displaymatrix side data

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>

utils: Add av_stream_add_side_data()

Functionally similar to av_packet_add_side_data(). Allows the use of an
already allocated buffer as stream side data.

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>

fate.sh: Allow setting other make flags for running tests

If makeopts_fate is set, these makeopts are used for running the
tests instead of the normal makeopts. If it isn't set, the normal
makeopts variable is used as before.

This is useful if remote testing on a lesser machine where a large
number of parallel jobs might be undesireable, while wanting to speed
up the build with many parallel processes.

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

avpacket: fix leak on realloc in av_packet_add_side_data()

If realloc fails, the pointer is overwritten and the previously allocated buffer
is leaked, which goes against the expected functionality of keeping the packet
unchanged in case of error.

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

examples/decode_video: allocate the packet dynamically

AVPackets on stack are discouraged.

examples/decode_video: switch to the new decoding API

examples/decode_video: use a parser for splitting the input

Do not rely on the decoder handling this, as it's not guaranteed to
work.

examples/encode_video: allocate the packet dynamically

AVPackets on stack are discouraged.

examples/encode_video: switch to the new encoding API

examples/decode_audio: flush the decoder

examples/decode_audio: allocate the packet dynamically

AVPackets on stack are discouraged now.

examples/decode_audio: handle planar audio now produced by the MP2 decoder

examples/decode_audio: use the new audio decoding API

examples/decode_audio: use a parser for splitting the input

Do not rely on the decoder handling this, as it's not guaranteed to
work.

examples/encode_audio: switch to the new audio encoding API

examples/qsvdec: convert to the new decoding API

examples/qsvdec: switch to the hwcontext API

The code now does not depend on VA and will work on windows as well.

libopusdec: default to stereo for invalid number of channels

This fixes an out-of-bounds read if avc->channels is 0.

Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

dvbsub: cosmetics: Group all debug code together

dvbsub: Check for errors from system()

libavcodec/dvbsubdec.c:145:5: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result]
libavcodec/dvbsubdec.c:148:5: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result]

als: Restructure DEBUG ifdefs to avoid unused function parameter warnings

ac3enc: Restructure DEBUG ifdefs to avoid unused function parameter warnings

ape: Restructure DEBUG ifdefs to avoid unused function parameter warnings

doc: Add libxavs section

rtpdec_mpeg4: const correctness for parse_fmtp_config()

libavformat/rtpdec_mpeg4.c:282:38: warning: passing argument 2 of ‘parse_fmtp_config’ discards ‘const’ qualifier from pointer target type

Drop some bogus Doxygen documentation.

Fix some mismatches between function parameter and doxygen parameter names.