Flag added in a few simple filters. A bunch of other filters can likely
use the feature as well.

Example:
Filter overlay
  Overlay a video source on top of the input.
    Inputs:
       #0: main (video)
       #1: overlay (video)
    Outputs:
       #0: default (video)

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

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

It will be useful outside of avprobe.

Removes an arbitrary hardcoded limit on the number of filters.

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

A similar error message is generated by the calling function, avoid
duplication and spurious error message occurring with negated boolean
options introduced in 8bb5680, which can be reproduced for example with
the command:
ffmpeg -f lavfi -i testsrc=d=0.1 -nostdin -f null -

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

The memleak fix in a1bcc76e was causing problems when the original
string wasn't NULL or av_malloc'ed, thus was disabled. It seems the only
OPT_STRING occurence where it needed to be fixed is in ffserver. This
commit fixes that specific problem and so re-enable the original code.

The definition of the include are needed for builds even without pp.
This should fix FATE with --disable-gpl after 63edd2f9.

The check is already performed by the PRINT_LIB_INFO macro.

Even if not directly supported by FFmpeg, it is still useful to see the
linked version of libavresample.

I.e. input options to output files or vice versa.

Remove now unused cmdutils get_buffer() implementation.

Make sure the behavior does not change with the locale.

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

As far as I can tell the code should not change behaviour
depending on locale in any of these places.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>

In this case, no encoder specific options are filtered, only
options specific to that codec type in general.

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

avprobe and avserver use cmdutils and do not strictly require libavresample.

This avoids per parameter changes in ffmpeg at the cost of making access
somewhat more annoying.

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

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

There may be more decoders that rely on this. Those should be found and
fixed.

Signed-off-by: Anton Khirnov <anton@khirnov.net>
CC:libav-stable@libav.org

CC:libav-stable@libav.org

The bottom line was invalid before.

CC:libav-stable@libav.org

Setting it to zero (instead of 128, as the default get_buffer() does)
also produces more correctly-looking output.

It's not relying on get_buffer() initializing the frame since
99e36ddd3e.

Fixes Ticket2127

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

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

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

The old implementation is no longer possibly due to limitations
of the new command line parser.

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

This makes the code simpler and avoids mixing designated and
non-designated initializers in a potentially unsafe way in avconv.

It results in spurious errors when using the new preparser.

It splits the commandline into a more convenient internal representation
for further parsing.

This will allow e.g.
- processing global options first independently of their location on the
  commandline, eliminating ugly hacks for processing e.g. cpuflags
  first
- better options validation and error reporting. It is now possible for
  the parser to know that it's applying an input option to an output
  file or vice versa and act accordingly.

Separate the code that searches for the option to use and the code that
actually writes it. The writing code will be reused by the new options
parser.

This is required because there are some "holes" in the list for
compatibility with the fork.

The commit also removes the now unecessary check from cmdutils.

Found-by: wm4

There is no point in duplicating this code in every get_buffer()
implementation.

This simplifies the condition to avoid hardcoding the systems
where the function exists. This also simplifies support for
newer Windows API subsets where this function doesn't exist,
such as Windows Phone 8 and the "metro" API subset of Windows 8.

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

The first parameter is a pointer and NULL is more correct

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

Fix coverity issue 743443, which was a harmless leak.

Do not skip the end delimiter.
Reserve positive return values.
This is an API break, but the function was introduced less than
two weeks ago.

Deprecate avcodec_get_chroma_sub_sample.