This also involves making some function static and changing the name
prefixes of some functions and structures.

A nontrivial public function such as this should most certainly NOT be
inline.

libavcodec/txd.c:49:60: warning: variable ‘mipmap_count’ set but not used

libavcodec/mmvideo.c:87:9: warning: variable ‘i’ set but not used

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

This is useful for writing unit tests.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

This fixes a double-free crash if lists are the same due to the two
merge_ref() calls at the end of the (useless) merging that happens.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Rationale is the same as for video width/height etc.

ZeroCodec relies on the keyframe flag being set in the container, and
prev is the previously decoded frame. A keyframe flags incorrectly set
will lead to this condition.
Signed-off-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

Our code should be terse and clear.

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Signed-off-by: Kostya Shishkov <kostya.shishkov@gmail.com>

This fixes a number of unused-but-set gcc warnings.

libavcodec/dvdec.c:344:12: warning: assignment discards ‘const’ qualifier from pointer target type

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

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

The function is not implemented (and possibly useless).

The scale filter is used for internal colorspace conversions, so it must
always be present.

Fixes https://bugzilla.libav.org/show_bug.cgi?id=244

The strtol() interface makes it difficult to use with
const-qualified pointers.  With this change, although
the const is still lost, the compiler does not warn
about it.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Signed-off-by: Mans Rullgard <mans@mansr.com>

This is logical and also fixes checking for the fourth plane offset.

Signed-off-by: Mans Rullgard <mans@mansr.com>

This is required for SIMD optimisations.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Signed-off-by: Mans Rullgard <mans@mansr.com>

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

This avoids memory leaks if there actually was some extradata
set before.
Signed-off-by: Martin Storsjö <martin@martin.st>

This removes one level of indentation.
Signed-off-by: Martin Storsjö <martin@martin.st>