avvidec: Report the latency once we're fully configured

Several decoders will only be able to report a real latency (has_b_frames)
once they're actually initialized (i.e. when they return their first frame).

Doing it earlier (in set_format) doesn't guarantee that the AVCodecContext
has_b_frames has been properly initialized.

https://bugzilla.gnome.org/show_bug.cgi?id=766362

avcodecmap: Add mapping for G729 caps

https://bugzilla.gnome.org/show_bug.cgi?id=766333

libav: Update to ffmpeg n3.0.2

avauddec: Finish frames if they are header buffers only and don't produce any output

Otherwise we will consider them as one frame of raw audio that is still
pending, and shift all timestamps by the amount of time spent with header
buffers.

https://bugzilla.gnome.org/show_bug.cgi?id=765797

avauddec: If decoding a frame failed, skip it

Otherwise the next successfully decoded frame will get its timestamp and we
will slowly let a/v sync drift apart.

https://bugzilla.gnome.org/show_bug.cgi?id=765797

avaudenc: Set all required fields in the AVFrame

Various functions in libavcodec need them, like the format, sample rate, etc.
and just having them in the context is not enough.

This fixes draining for codecs like MP2 that require a fixed frame size and
require libav to pad the last frame if required.

avcfg: Use av_strdup() instead of g_strdup() for strings owned by ffmpeg

It has its own allocator that depending on the configuration is incompatible
with GLib's and just causes a segmentation fault. Like on Windows.

https://bugzilla.gnome.org/show_bug.cgi?id=760266

avdeinterlace: Register all filters and only disable all except for yadif during compilation

Otherwise avdeinterlace won't find any filters, and we only use yadif in
there.

https://bugzilla.gnome.org/show_bug.cgi?id=765319

libav: Update to n3.0.1

Automatic update of common submodule

From 6f2d209 to ac2f647

avvid{dec,enc}: remove dead gobject macros

The GObject macros either for GstFFMpegVidDec and GstFFMpegVidEnc can
break the compilation because they are not GTypes, since each av video
elements are registered in runtime.

https://bugzilla.gnome.org/show_bug.cgi?id=764162

libav: use new gst_element_class_add_static_pad_template()

https://bugzilla.gnome.org/show_bug.cgi?id=763084

Back to development

Release 1.8.0

configure: Restore CPPFLAGS after the last check

The next checks can also set CPPFLAGS.

https://bugzilla.gnome.org/show_bug.cgi?id=763940

Release 1.7.91

avcodecmap: Only set the bitrate if we actually got it from the caps

Otherwise we set a random value from the stack as bitrate. The default (0)
is a better choice and doesn't upset valgrind.

Release 1.7.90

Automatic update of common submodule

From b64f03f to 6f2d209

avdeinterlace: Don't assume that get_current_caps() returns non-NULL caps after has_current_caps()

Remove calls to gst_pad_has_current_caps() which then go on to call
gst_pad_get_current_caps() as the caps can go to NULL in between. Instead just
use gst_pad_get_current_caps() and check for NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=759539

win32: remove outdated build cruft

This hasn't been touched for generations, doesn't work,
and is just causing confusion. We also don't want to
maintain these files manually.

Don't use exported but undeclared core debug category symbols

It's not right and won't work on Windows with MSVC.

Back to development

Release 1.7.2

avcfg: rc-min-rate and rc-max-rate are now 64bit integers

Switch the gobject properties and internal handling to support that

avvidenc: Set width/height and format in the AVFrame we pass to the encoder API

docs: Update docs to not have unstaged changes after build again

There are some new codec IDs that we probably would like to map too, e.g.
Dirac/VC2 and Cineform HD.

libav: Update to n3.0

Automatic update of common submodule

From 86e4663 to b64f03f

avaudenc: Also use av_free() instead of g_free()

avaudenc: Use av_malloc() instead of GLib malloc() wrappers for memory freed by ffmpeg

It has its own allocator that is not necessarily doing the same as malloc and
will then usually crash. E.g. on Windows or when memalign() is available.

avvidec: Bring mpeg video decoders up to PRIMARY rank

We should have done that ages ago ...

https://bugzilla.gnome.org/show_bug.cgi?id=574461

libav: Ensure to link with -lavfilter

avviddec: Use AVFrame API instead of deprecated AVPicture API

av: Use av_image_get_buffer_size() instead of deprecated avpicture_get_size()

avdemux: Use av_packet_unref() instead of deprecated av_free_packet()

avvidenc: Use AVPacket.flags for detecting keyframes

AVCodec.coded_frame is deprecated and will disappear at some point
in the future.

avdeinterlace: Chain up to parent class' dispose()

https://bugzilla.gnome.org/show_bug.cgi?id=757498

libav: Bitrate field changed from int to int64_t, fix compiler warnings

Cast it to a gint64 for now though, as otherwise we will fail compilation
with ffmpeg 2.8.

https://bugzilla.gnome.org/show_bug.cgi?id=757498

libav: Remove usage of deprecated API

https://bugzilla.gnome.org/show_bug.cgi?id=757498

avdeinterlace: Port non-deprecated AVFilter API

https://bugzilla.gnome.org/show_bug.cgi?id=757498

libav: Update to ffmpeg n2.8.5

av: canonicalise av plugin name more consistently in more places

Use g_strdelimit(), make sure to include comma as well, which
isn't included in G_STR_DELIMITERS.

https://bugzilla.gnome.org/show_bug.cgi?id=734451

Back to development

Release 1.7.1

avcodecmap: Add special mapping for mono channel layouts

In ffmpeg this is the same as FRONT_CENTER, but we distinguish between
FRONT_CENTER and MONO in GStreamer. Add an explicit mapping for this special
case in the translations functions.

https://bugzilla.gnome.org/show_bug.cgi?id=759846

avviddec: remove realvideo slice_offset handling

Handling slice_offset in avviddec is resulting in invalid memory read.
Since rv decoders anyways handle slice_offset, removing the same to fix
memory mishandlings

https://bugzilla.gnome.org/show_bug.cgi?id=758726

libav: Update to ffmpeg n2.8.4

configure: Use -Bsymbolic-functions if available

While this is more useful for libraries, some of our plugins with multiple
files and some internal API can also benefit from this.

docs: update to git

Automatic update of common submodule

From b319909 to 86e4663

Require libav provided by FFmpeg at run-time

Error out if system's libav* libraries are not
provided by FFmpeg. Libav-incompatible changes
were introduced to support the latter so we
can no longer support both.

https://bugzilla.gnome.org/show_bug.cgi?id=758183

Require libav provided by FFmpeg at build-time

Libav-incompatible changes were introduced to support
FFmpeg and we can no longer properly support Libav.

FFmpeg micro versions for libav* start at 100 (this was
done to differentiate from Libav builds). We use this
to bail at configure time if the system libav* libraries
are not provided by FFmpeg.

https://bugzilla.gnome.org/show_bug.cgi?id=758183

libav: Update to n2.8.3

avviddec: Make sure to use a buffer pool with the correct width/height configured on it for pushing buffers downstream

If downstream does not provide a (usable) pool, we would use our internal
pool. But the internal pool might be configured with a different width/height
because of padding, which then will cause problems if we push buffers from it
directly downstream.
Instead create a new pool if the width/height is different.

This prevents crashes with vaapisink and d3dvideosink for example.

Based on the debugging results and discussions with
Nicolas Dufresne <nicolas.dufresne@collabora.com>

https://bugzilla.gnome.org/show_bug.cgi?id=758344

avviddec: Fix structure memory leak

gst_structure_copy already takes a reference for config_copy.
No need to take another reference while setting it to buffer pool

https://bugzilla.gnome.org/show_bug.cgi?id=758512

Remove unnecessary NULL checks before g_free()

g_free() is NULL-safe

(Includes unrelated formatting change to go through
indent checker commit hook)

libav: Update to n2.8.2

avviddec: Don't fallback on allocation failure

Allocation should keep working, falling back causes the stride to
change which is not supported in direct rendering.

https://bugzilla.gnome.org/show_bug.cgi?id=756028

avvidenc: Fix frame memory leak

The frame being passed to handle_frame should be unref'ed in all cases

https://bugzilla.gnome.org/show_bug.cgi?id=757453

avdemux: Fix string memory leaks

https://bugzilla.gnome.org/show_bug.cgi?id=757454

avviddec: _drain and _finish need slightly different handling

... since they handle separate cases in video decoder with different requirements.
Consider e.g. x264enc ! rtph264pay ! identity drop-probability=0.1 ! rtph264depay
to illustrate a need for such separation.

Automatic update of common submodule

From b99800a to b319909

Use new GST_ENABLE_EXTRA_CHECKS #define

https://bugzilla.gnome.org/show_bug.cgi?id=756870

Automatic update of common submodule

From 9aed1d7 to b99800a

avviddec: Fix bufferpool config double free by taking a copy as needed

Also free the config in two code paths where we leaked it before.

libav: Update to n2.8.1

avcodecmap: add jpeg2000 decoder mapping

Map the 3 possible jpeg2000 caps to JPEG2000 codec id

avcodecmap: Don't allocate dummy codec data anymore, it causes crashes nowadays

Multithreaded encoders are going to free this dummy codec data twice, e.g.
with this pipeline
  gst-launch-1.0 videotestsrc num-buffers=40 ! \
    videoconvert ! avenc_mjpeg ! fakesink

avvidenc: Pass the correct user_data to gst_buffer_new_wrapped_full()

Fixes crash with e.g.

  gst-launch-1.0 videotestsrc num-buffers=40 ! \
    videoconvert ! avenc_huffyuv ! fakesink

libav: G_CONST_RETURN is deprecated since GLib 2.30

Add configure check for GLib for consistency with other modules

avviddec: only free config when pool doesn't take ownership

Since gst_buffer_pool_set_config() takes ownership of the config structure,
it is only necessary to free the structure before using it when the true
branch of if (gst_buffer_pool_config_validate_params) hasn't run.

gst_buffer_pool_set_config() always takes ownership of the structure
regardless of success or failure. Which means the return, checked with
if (!working_pool), has no relation to the state of the structure.

avviddec: fix memory leak

Free config before overwriting.

CID #1320708

Back to development

Release 1.6.0

Release 1.5.91

avvidec: increase default alignment to 32 bytes

Change default alignment from 16 to 32 bytes, which fixes crashes
when decoding H.265 using AVX2-based decoder code paths and when
using ximagesink/glimagesink.

https://bugzilla.gnome.org/show_bug.cgi?id=754120

avvideodec: ensure required mem alignment fixing avdec_h265 crashes with ximagesink/glimagesink

Make sure the alignment requirement in GstAllocationParams
matches the GstVideoAlignment requirements. This fixes
issues with avdec_h265 crashing in the avx2 code path when
used with playbin and ximagesink/glimagesink as videosink.

The internal video pool would allocate buffers with an
alignment of 15 even though GstVideoAlignment specified
a stride_align requirement of 31 (which comes from ffmpeg).

https://bugzilla.gnome.org/show_bug.cgi?id=754120

libav: Update to ffmpeg n2.8

https://bugzilla.gnome.org/show_bug.cgi?id=754920

avvid/audenc: Set some tags in the output for downstream's info

Add the codec name and bitrate into the output for informational
purposes. Bitrate in particular is now used by flvmux to set
videodatarate and audiodatarate in the resulting stream

avviddec: Fix pool reallocation logic

Some check where incorect and also unsafe. The only reliable information
in get_buffer2 is the picture width/height really. The side effect is
that the width/height of the internal pool endup padded, so when we
switch we also need to switch to the a new width/height, hence we save
the pool info.

https://bugzilla.gnome.org/show_bug.cgi?id=753869

Release 1.5.90

aviddec: Re-enable direct rendering

This is achieved by using a tempory internal pool. We can then switch to a
downstream pool if the downstream pool buffer have matching strides.

https://bugzilla.gnome.org/show_bug.cgi?id=752802

avvidenc: use template subset check for accept-caps

It is faster than doing a query that propagates downstream and
should be enough

avaudenc: use pad template caps for accept-caps

use template subset check for accept-caps

It is faster than doing a query that propagates downstream and
should be enough

avaudenc: remove useless getcaps method

It just calls the exact same function as the default handler

avvidenc: remove useless getcaps method

It just calls the exact same function as the default handler

avauddec: let audiodecoder call the default pad query handler

Avoids repeating the same handling in many decoders

avviddec: let videodecoder call the default pad query handler

Avoids repeating the same handling in many decoders

libav: Update to n2.7.2

avviddec: use template caps intersection for accept-caps

Avoid doing downstream caps queries when accept-caps should just
do a shallow caps check on the element itself

https://bugzilla.gnome.org/show_bug.cgi?id=753623

avauddec: use template caps intersection for accept-caps

Avoid doing downstream caps queries when accept-caps should just
do a shallow caps check on the element itself

https://bugzilla.gnome.org/show_bug.cgi?id=753623

Map ffmpeg metadata to GStreamer tags

Update to the metadata API ffmpeg has had in
place for a long time now, and reenable output
of GStreamer tags from the demuxer.

https://bugzilla.gnome.org/show_bug.cgi?id=566605

avviddec: Remove unused force parameter

This parameter has been always false for a long time.

avviddec: Use caps-like size in get_buffer2()

The size in the AVFrame in get_buffer2 don't match the output size,
instead they match ffmpeg's memory requirements, so we can't compare
them from the values of the output AVFrame. Those are comparable to
the values in the passed AVCodecContext.

avviddec: Reset current_dr on close and when not chosing it

https://bugzilla.gnome.org/show_bug.cgi?id=753189

avviddec: Disable direct rendering before a frame has been produces

ffmpeg doesn't provide the final's image width & height in the get_buffer2()
callback, so it's not possible to create an output state for GstVideoDecoder
at this stage. So only try to do direct rendering if the buffer pool has already
been negotiated based on the final decoded size.

This partially reverts the effects of 2e621f8db

https://bugzilla.gnome.org/show_bug.cgi?id=752802

Revert "For static builds, add the ffmpeg library directories to the linker flags"

This reverts commit ac3437153dc6cb3d82cdda0d1f60ffc5600fdf67.

Doesn't actually make sense as it will put the (uninstalled) library paths
into the installed .la files. How does this all work?

For static builds, add the ffmpeg library directories to the linker flags