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.

Reviewed-by: Stefano Sabatini <stefasab@gmail.com>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

The file name can be set by setting the FFREPORT environment
variable to "file=something".

Fix trac ticket #1823.

This reverts commit 841bf0ef24.
This does not fully work as the file is used before the filename is parsed
so not all information is written in the specified file.

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

This fixes compilation with libswscale disabled.

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

It appears such construct has undefined behavior

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

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

Some systems require sys/time.h being explicitly included before
sys/resource.h.  The configure check already does this.

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

This is needed for setting both contexts -err_detect at once.

Reviewed-by: Clément Bœsch <ubitux@gmail.com>
Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

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

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

Fixes CID732166.