Remove now redundant (and broken/undocumented) opt_programid.

Also document it and replace undocumented and inconsistent
'-sample_fmt list' syntax with -sample_fmts.

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.

Allows cmdutils to call each tool's own cleanup function.

Replace it with newly introduced libavutil API.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

It will be further developed with a few incompatible changes.

ffmpeg.c will stay as is for some time, so any scripts using it won't be
broken.

This will allow the incoming -timecode option to be handled in both
encoders (mpeg12enc in that case) and demuxers (dv, mxf, etc.).

Avoid brittle and obfuscating reference to a global.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Split them when codec id is known.

Deprecate avcodec_alloc_context/2.

Signed-off-by: Stefano Sabatini <stefano.sabatini-lala@poste.it>

All option parsing functions now match the function pointer signature through
which they are called (int f(const char *, const char *), thereby working
reliably on all platforms.
Prefix all option processing functions with opt_

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Also add the public interface libavfilter/vsink_buffer.h.

It stores options in a dictionary to be passed to new open calls.

It will replace opt_default once all the pieces are in place.

Fix ffplay -i FILE, which was recently broken.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

The -i INPUT option can be implemented more cleanly by using a
function option, which can easily be done now that the
parse_arg_function passed to parse_options has a standard signature.

This is required for a pending simplification.

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

Make ff* tools only accept opt_* functions taking two arguments.

The distinction between functions with one and two arguments is quite
pointless. Simplify parse_options() code.

Make ff* tools only accept opt_* functions taking two arguments.

The distinction between functions with one and two arguments is quite
pointless. Simplify parse_options() code.

Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>

av_freep(swsContext) will leak all memory potentially
allocated within the swsContext.

Fix warnings:
cmdutils.c: In function ‘opt_default’:
cmdutils.c:304: warning: initialization discards qualifiers from pointer target type
cmdutils.c: In function ‘set_context_opts’:
cmdutils.c:431: warning: passing argument 2 of ‘alloc_priv_context’ discards qualifiers from pointer target type
cmdutils.c:414: note: expected ‘struct AVClass *’ but argument is of type ‘const struct AVClass *’

Avoid the presence of an invalid pointer, fix a crash in case of
get_filtered_frame() failure.