Use <> for system headers, add needed math.h, drop unnecessary avutil.h.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

The width and height must be copied from the input before
being used.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

Also remove now unused variable.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Also remove now unused variables.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Rename it to "avctx", the old name was confusing as the function is
used also with decoder contexts.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

It's more readable and less prone to breakage.

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

CONFIG_SMALL is always defined as 0 or 1.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

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

libavcodec/libspeexdec.c:108: warning: passing argument 2 of ‘speex_bits_read_from’ discards qualifiers from pointer target type
/usr/include/speex/speex_bits.h:80: note: expected ‘char *’ but argument is of type ‘const uint8_t *’

libavcodec/libopenjpeg.c:100: warning: passing argument 2 of ‘opj_cio_open’ discards qualifiers from pointer target type
/usr/include/openjpeg.h:770: note: expected ‘unsigned char *’ but argument is of type ‘const uint8_t *’

User-provided data should never trigger an assert; return error instead.
Also fix an instance of get_bits where get_bits_long should have been used.

This eliminates the following warning:
libavcodec/vaapi_mpeg2.c:112:14: warning: variable 'start_code' set but not used

An official specification now exists and it is both more detailed and
easier to understand than the incomplete notes available here.

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

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