AVID streams - currently handled by the AVRN decoder - can be (depending
on extradata contents) either MJPEG or raw video. To decode the MJPEG
variant, the AVRN decoder currently instantiates a MJPEG decoder
internally and forwards decoded frames to the caller (possibly after
cropping them).

This is suboptimal, because the AVRN decoder does not forward all the
features of the internal MJPEG decoder, such as direct rendering.
Handling such forwarding in a full and generic manner would be quite
hard, so it is simpler to just handle those streams in the MJPEG decoder
directly.

The AVRN decoder, which now handles only the raw streams, can now be
marked as supporting direct rendering.

This also removes the last remaining internal use of the obsolete
decoding API.

src/libavfilter/vf_vif.c: In function ‘process_frame’:
src/libavfilter/vf_vif.c:542:20: warning: ‘main’ is usually a function [-Wmain]
     AVFrame *out, *main = NULL, *ref = NULL;
                    ^~~~
Signed-off-by: Guo, Yejun <yejun.guo@intel.com>

The build warning message:
src/libavfilter/vf_ssim.c: In function ‘ssim_plane_16bit’:
src/libavfilter/vf_ssim.c:246:24: warning: ‘main’ is usually a function [-Wmain]
         const uint8_t *main = td->main_data[c];
                        ^~~~
src/libavfilter/vf_ssim.c: In function ‘ssim_plane’:
src/libavfilter/vf_ssim.c:289:24: warning: ‘main’ is usually a function [-Wmain]
         const uint8_t *main = td->main_data[c];
                        ^~~~
Signed-off-by: Guo, Yejun <yejun.guo@intel.com>

It is simpler and more efficient.
Suggested-by: NJames Almer <jamrial@gmail.com>

Current code is very confused and confusing. It uses two different
reference frames - "previous" and "last" - when only one is really
necessary. It also confuses the two, leading to incorrect output with
APNG_DISPOSE_OP_PREVIOUS mode.

Fixes #9017.

FF_CODEC_CAP_ALLOCATE_PROGRESS makes no sense because the decoder does
not support frame threading.
FF_CODEC_CAP_SKIP_FRAME_FILL_PARAM makes no sense because the decoder
does not handle skip_frame.

It shares very little code with pngdec, so keeping them together only
makes the code harder to read.

This is needed to implement x86 SIMD.

Note that >32 codes are no longer supported, give
proper error code if such scenario ever happens.

Fixes floating point exception.

No buffer will be fetched from the pool after it's uninitialized, so there's
no benefit from waiting until every single buffer has been returned to it
before freeing them all.
This should free some memory in certain scenarios, which can be beneficial in
low memory systems.

Based on a patch by Jonas Karlman.
Reviewed-by: NAnton Khirnov <anton@khirnov.net>
Signed-off-by: NJames Almer <jamrial@gmail.com>

It is unused and coincides with ff_modified_quant_tab.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Unused since 80944df7.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

The last user of g15Mask, r15Mask, g16Mask and r16Mask was disabled
in 77a416e8 and finally removed in
36e8de07; b15Mask and b16Mask were
apparently always unused (except for in_asm_used_var_warning_killer,
a function that only existed to make the compiler not optimize ASM
constants away).
w10 is unused since d604bab9, w02
since ef423a66.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

mask24hh etc. are unused since f099fbf5,
mask32b and mask32r since 296609f8,
mask32g since b38d4874 and mask32 since
f8a138be.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

mmx_grnmask is unused since 531f97b0,
the other constants since e934194b.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

The code using ff_exp2 (namely ff_acelp_decode_gain_code) use it only if
G729_BITEXACT is defined. So disable it if not.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

This allowed to remove forward declarations. Because compilers expect
declarations for all functions they encounter even when it is within
blocks disabled via "if (0 && foo)", one has to use a real #if in
ff_diracdsp_init_x86.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Unused since a1f3b18b, yet as nonstatic
functions the compiler can't detect this, so that these functions aren't
stripped and no warning is emitted.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

ff_acelp_decode_8bit_to_1st_delay3, ff_acelp_decode_4bit_to_2nd_delay3
and ff_acelp_decode_5_6_bit_to_2nd_delay3 are all only used once (by
g729dec) whereas ff_acelp_decode_9bit_to_1st_delay6 and
ff_acelp_decode_6bit_to_2nd_delay6 are completely unused; with the
possible exception of ff_acelp_decode_4bit_to_2nd_delay3, these
functions are so small that inlining them is appropriate; and as long as
ff_acelp_decode_4bit_to_2nd_delay3 is only called once, this is also
true for it.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

It is only used there.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Unused since 46246567.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Added in 0c79b140 to use it
in a function that was never used and was itself removed in
676f1f53.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

It is only used inside code guarded by #ifdef DEBUG, i.e. it is normally
unused.
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Clang infers from the existence of a default case that said case can be
taken. In case of libavcodec/bitstream.c said default case consisted of
an av_assert1 that evaluates to nothing in case of the ordinary assert
level. In this case (that doesn't happen) a variable wouldn't be
initialized, so Clang emitted Wsometimes-uninitialized warnings.
Solve this by making sure that the default path also initializes
the aforementioned variable.
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

The counter for the number of styles is written on two bytes, ergo
anything > UINT16_MAX is invalid. This also fixes a compiler warning
because of a tautologically true check on 64bit systems.
Reviewed-by: NPhilip Langdale <philipl@overt.org>
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>

Commit 0d1229f1 factored the main part
of the voc demuxer's read_packet function out; yet when this Libav
commit was merged in f99195d5, the
dependency of the other users of this function on vocdec.o was
unnecessarily kept. This commit fixes this.

While just at it, also disable the data only used by the voc demuxer
and muxer in voc.c if both of them are disabled.
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NAndreas Rheinhardt <andreas.rheinhardt@gmail.com>