They will only ever be <=0 if the option is broken, in which case this
check is hiding a bug.

It's a hack which was created to allow for multiple options with
different defaults to refer to same field (e.g. 'b' vs 'ab'). There is
no need for it anymore.

Also change it to be more readable.

Specifically, trellis, nr, me_range, me_method, subq, b_strategy,
keyint_min, chroma_me and coder.
Change their defaults to -1 and apply them after
x264_param_default_preset() if the user explicitly set them.

Move setting it after x264_param_default_preset()
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Deprecate AVCodecContext.directpred

Deprecate AVCodecContext.partitions.

Deprecate AVCodecContext.complexityblur

Deprecate AVCodecContext.deblockalpha/deblockbeta

Deprecate AVCodecContext.bframebias.

Specifically: gop_size, max_b_frames, scenechange_threshold, qmin, qmax,
max_qdiff, qblur, qcompress and refs.

Change their default values to -1 and only use them if the user
explicitly set them. Otherwise x264 defaults are used.

Move setting those options after x264_param_default_preset(), so they
don't get overwritten by it.

priv_data is memset to 0.

Since we now always allocate it, it can simply be made part of the context
instead.

Update FATE references accordingly.

scale_coefficients() rather than in apply_channel_coupling()

The number of rematrixing bands still needs to be calculated.

This function should return number of samples decoded, not number of bytes
decoded.
Spotted by Uoti Urpala.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

After 039267f1, metadata specifiers are
passed without the leading ':'. Remove a ++ that didn't take this into
account.

mpeg12: propagate more real return values through chunk decode error return and fix some indentation
Signed-off-by: Anton Khirnov <anton@khirnov.net>

This fixes improper flushing in the cases when the same frame is decoded in
several iterations (for being too large to fit into output buffer) and flush is
called mid-decoding and it also resets context in case of decoding errors.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Change syntax -- -[vas]bsf are replaced by -bsf:[stream specifier], the
argument is now a comma-separated list of bitstream filters.

Make it work properly with multiple input files.

Add a -frames option that uses generic stream specifiers, change
-[vad]frames into aliases to it.

It is now possible to set metadata on streams and chapters.

It should be flags, not int.

Move stream maps to it.

Eventually all non-global options should be stored in it.

This allows options like -target, which are just shortcuts for other
options, to work without dummy function for all options they invoke.

This is the first step to removing the globals plague from avtools.

This will be useful in the following commit.