Fixes -Wunused-const-variable from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Works only with video stream.
First pass without seeking -- counts crcs of a frames and store it in an array.
After that it seeks a lot in different places and checks if crcs of these frames and crcs of frames in array are the same.
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes -Wunused-const-variable from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>
Reviewed-by: N"Ronald S. Bultje" <rsbultje@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes -Wsometimes-uninitialized from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>
Reviewed-by: N"Ronald S. Bultje" <rsbultje@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Forgotten in c5d0148c. Found by Thomas Langhorst.
Signed-off-by: NLou Logan <lou@lrcd.com>

Fixes -Wenum-conversion from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes part of ticket #4786.

Fixes Ticket4777
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes arm cross build on osx
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Reviewed-by: NAndreas Cadhalpun <andreas.cadhalpun@googlemail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NLou Logan <lou@lrcd.com>

Signed-off-by: NLou Logan <lou@lrcd.com>
Reviewed-by: NGanesh Ajjanagadde <gajjanag@mit.edu>

If the return value doesn't fit in a single register rdx/edx can in some
cases be used in addition to rax/eax.

Doesn't affect any of the existing checkasm tests but might be useful later.

Also comment the relevant code a bit better.

Now we no longer have to rely on function pointers intentionally
declared without specified argument types.

This makes it easier to support functions with floating point parameters
or return values as well as functions returning 64-bit values on 32-bit
architectures. It also avoids having to explicitly cast strides to
ptrdiff_t for example.

This fixes fate when FF_API_ERROR_RATE=0.

Reviewed-by: NHendrik Leppkes <h.leppkes@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NGwenole Beauchesne <gwenole.beauchesne@intel.com>

Invalid buffer ids are defined by VA_INVALID_ID. Use that through out
vaapi_*.c support files now that we have private data initialized and
managed by libavcodec. Previously, the only requirement for the public
vaapi_context struct was to be zero-initialized.

This fixes support for 3rdparty VA drivers that strictly conform to
the API whereby an invalid buffer id is VA_INVALID_ID and the first
valid buffer id can actually be zero.
Signed-off-by: NGwenole Beauchesne <gwenole.beauchesne@intel.com>

Move libavcodec managed objects from the public struct vaapi_context
to a new privately owned FFVAContext. This is done so that to clean up
and streamline the public structure, but also to prepare for new codec
support, thus requiring new internal data to be added in there.

The AVCodecContext.hwaccel_context, that holds the public vaapi_context,
shall no longer be accessed from within vaapi_*.c codec support files.
Signed-off-by: NGwenole Beauchesne <gwenole.beauchesne@intel.com>

Deprecate older VA pixel formats (MOCO, IDCT) as it is now very unlikely
to ever be useful in the future. Only keep plain AV_PIX_FMT_VAAPI format
that is aliased to the older VLD variant.

This is an API change.
Signed-off-by: NGwenole Beauchesne <gwenole.beauchesne@intel.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixing small leaks that can occur when mkv_write_tracks fails in mkv_write_header
(e.g., if video track has unknown codec). Also changing mkv_write_seekhead to take
the MatroskaMuxContext to avoid having dangling pointers.
Signed-off-by: NNeil Birkbeck <neil.birkbeck@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

otherwise it can fail when a different idct is used on some platform
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes ticket #4784 as found by rodarmor.
Signed-off-by: NLou Logan <lou@lrcd.com>
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Currently, when forcing an I frame, via API, or via the ffmpeg cli,
using -force_key_frames, we still let x264 decide what sort of
keyframe to user. In some cases, it is useful to be able to force
an IDR frame, e.g. for cutting streams.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

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

This fixes compilation with FF_API_UNUSED_MEMBERS=0.

When building SDL with MinGW, it sets -mwindows with the
assumption that the application is a GUI application. If this
is linked without passing -mconsole to configure via
--extra-ldflags, stdout will be silenced from cmd.exe while
running FFmpeg.

The -mwindows flag that causes this behavior is included in the
sdl_libs variable, so append -mconsole there rather than create
an sdl_ldflags case just to insert it (especially if -mconsole
must come *after* -mwindows in order to be effective).
Reviewed-by: NTimothy Gu <timothygu99@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>