app: GST_EXPORT -> GST_APP_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

allocators: GST_EXPORT -> GST_ALLOCATORS_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

audioaggregator: fix channel-mask negotiation

When outputting more than two channels, a channel-mask has to be
specified in the output caps.

We follow the same heuristic as other cases, when downstream
does not specify a channel-mask, we use that of the first
configured pad, and if there was none we generate a fallback
mask.

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

playback: Remove subtitle-encoding property on uridecodebin3

For now, until we found a more future-proof solution

docs: video: fix typo and add since marker

docs: libs: add another missing symbol

docs: add video region of interest add/get parameter api to docs

audio: Add audioaggregator.h in audio.h

docs: add GstPhysMemoryAllocator to docs

Revert "alsasrc: lock calls to snd_pcm_delay() with mutex as in alsasink"

This reverts commit 4f88125b3ddc15367d528e850cc27739f5bb27fa.

This seems to have undesirable side-effects and needs more
investigation first.

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

gl: only enable gbm backend if gudev is available

gudev/gudev.h is included unconditionally, so only enable
the gbm backend if gudev was actually found. This also
matches the meson build behaviour.

Should fix build on GNOME SDK builder.

Release 1.13.90

Update docs

pkgconfig: Add private zlib dependency for libgsttag

Fixes issues with static application builds (which might not know
they need to link with zlib when using a libgsttag that was compiled
with it).

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

meson: enable more warnings

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

videodecoder: Reset QoS time after pushing segment

This fixes playbin gapless playback. An ancient QoS time was used and
would lead to all frames being dropped.

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

Port to latest GstAggregator segment API

The aggregator segment is now exposed on the src pad

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

audioaggregator: remove GstAudioAggregator->info

As we now require subclasses to use a subclass of
GstAudioAggregatorPad, we can reuse its info field

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

tests: tags: add unit test for ID3 v2.4 extended headers

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

id3v2: re-fix handling of v2.4 extended headers

The various id3v2 specs handle the extended header sizes differently
(because hey, it wouldn't be fun otherwise).

http://id3.org/id3v2.3.0 states:
"Where the 'Extended header size', currently 6 or 10 bytes, excludes
 itself."

http://id3.org/id3v2.4.0-structure states:
  Extended header size   4 * %0xxxxxxx
     Number of flag bytes       $01
     Extended Flags             $xx

   Where the 'Extended header size' is the size of the whole extended
   header, stored as a 32 bit synchsafe integer. An extended header can
   thus never have a size of fewer than six bytes.

So in id3v2.4.0 it's the *whole* extended header size (a-la ISOBMFF
atom), whereas in id3v2.3.0 it's the extended header size *excluding*
those 4 initial bytes.

And for other versions, god knows..

Fixes regression introduced in commit da607005.

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

audioaggregator: refactor conversion API

For the rationale, see:

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

Also test audiomixer conversion of current output buffer

glcolorconvert: Fix a typo in a debug message

glfilter: Support non-2D texture targets

Allow for sub-classes to change pad templates to
support other texture targets, and bind input textures
accordingly.

When setting the caps, also store the texture target.

By default, glfilter only reports 2D texture targets
in the default caps, but sub-classes can change that
and it would be nice if they could easily find out
which texture targets were negotiated.

This adds 2 fields to the public struct, but since
it's unreleased -base API, it's not an ABI break.

audioaggregator: Document that the pad's audio info is read-only and needs the object lock

Also fix indentation in the header a bit.

m4: fix variable value access

roi-meta: duplicate params when copying meta

When copying the meta the params list should be copy too into the
new meta.

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

meson: Don't install GL GBM headers

gl: Move DRM_CFLAGS into gst-gl.m4

gl: Add missing gbm directory to dist dir

This fixes make distcheck

gl: Add DRM cflags

This prevents cross compilation errors like:

usr/include/xf86drm.h:40:10: fatal error: drm.h: No such file or directory

These are caused because gstgldisplay_gbm.h includes xf86drm.h .

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

pbutils: Fix compilation of last commit

pbutils: Move locale dir initialization to a separate function and do lazy initialization

It is the only thing gst_pb_utils_init()  does and it could be
automatically called from the places in pbutils it is needed.

After 1.14 we should deprecate gst_pb_utils_init().

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

gl: Add meson support for GBM backend

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

gl: Add Mesa3D GBM backend

This makes it possible to use the GStreamer OpenGL elements without a
windowing system if a libdrm- and Mesa3D-supported GPU is present

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

configure: Add configure checks for libdrm and gudev

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

glimagesink: Correct PAR in output caps when transforming

When doing a 3D/multiview transformation and rescaling to
match the output window size, the resulting PAR may
not match the input any more and needs recalculating,
or else the GstSample reported to client-draw has the
wrong PAR.

roi-meta: Fix bad doc comment syntax

videometa: add support for downstream parameters to ROI meta

The current GstVideoRegionOfInterestMeta API allows elements to detect
and name ROI but doesn't tell anything about how this information is
meant to be consumed by downstream elements.
Typically, encoders may want to tweak their encoding settings for a
given ROI to increase or decrease their quality.
Each encoder has its own set of settings so that's not something that
can be standardized.

This patch adds encoder-specific parameters to the meta which can be
used to configure the encoding of a specific ROI.

A typical use case would be: source ! roi-detector ! encoder
with a buffer probe on the encoder sink pad set by the application.
Thanks to the probe the application will be able to tell to the encoder
how this specific region should be encoded.

Users could also develop their specific roi detectors meant to be used with a
specific encoder and directly putting the encoder parameters when
detecting the ROI.

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

rtpbuffer.h: Add new RTPBuffer flags

These flags will be used for Forward Error Correction purposes

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

meson: fix multi-line string

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

pbutils: add some missing gi annotations

For allow-none parameters

tests: videoscale: simplify test and make it more robust

test_negotiation would occasionally time out, for unknown reasons.
Simplify the test setup and get rid of the main loop, busses, and
notify signals. With this I can no longer easily reproduce the
timeout. Fingers crossed.

appsrc: don't signal the GCond if no one is waiting on it

Performance optimisation: Keep track whenever the streaming
thread or the application thread are waiting on the GCond for
more space or new data, and only signal on the GCond if someone
is actually waiting. Avoids unnecessary syscalls and thus
context switches.

appsrc: use GstQueueArray instead of GQueue for internal item queue

Performance optimisation.

Avoids alloc/free churn for the queue's list nodes.

Depends on new API in GstQueueArray in core.

appsink: don't signal the GCond if no one is waiting on it

Performance optimisation: Keep track whenever the streaming
thread or the application thread are waiting on the GCond
for more space or new data, and only signal on the GCond if
someone is actually waiting. Avoids unnecessary syscalls and
thus context switches.

appsink: use GstQueueArray instead of GQueue for internal item queue

Performance optimisation.

Avoids alloc/free churn for the queue's list nodes.

icles: add appsink and appsrc benchmarks

These are very much artificial of course, but got to
measure something. appsink one contains lots of buffer
creation/free overhead, while appsrc one does not.

m4: gmodule-no-export-2.0 is needed for everything using the g_module_*() API

While it is pulled in via the GLib check, GLIB_LIBS is not pulled into
any of the places using the API.

gl: Link to libgstallocators for the dmabuf allocator

gl/wayland: Memory leak when display couldn't connect

When trying to create a wayland display, it may fail because there
is not actually display to connect. It this case NULL is returned
but the created instance is not freed.

This patch unrefs the failed display.

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

audio-converter: fix declaration-after-statement

audio-converter: add a convenience conversion method

This is useful from python bindings

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

gst_audio_converter_new: update annotations

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

AudioConverter: register as boxed type

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

audio-info: annotate gst_audio_info_set_format

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

Back to development

Release 1.13.1

docs: update plugin docs

configure: fix build with --disable-external

po: update translations

Update disted backup ORC files

gstaudiopack.orc: pack_u32be_swap: actually swap

Fixes:

gst-launch-1.0 audiotestsrc ! audio/x-raw, format=U32BE ! \
audioconvert ! autoaudiosink

doc: Remove obsolete Since 0.10.X marks

doc: Add per version newly added API indexes

doc: Fix since marker in dmabuf to match a stable release

doc: Remove extra . after Since marker

doc: Fix Since 1.X marker on new video formats

doc: Include new GstVideoOverlay API

playbin3: fix source-setup signal emission

Previous code would pass 0 as source element.

vorbisparse: error out when headers are missing

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

audioaggregator: remove declaration for function that doesn't exist

docs: add GstAudioAggregator to docs

docs: add audiomixer + audiointerleave to plugin docs

docs: update plugin docs for recent changes

audiomixer, audiointerleave: add to build

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

GstAudioAggregator: hook up to build

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

audioaggregator, audiomixer, audiointerleave: move from -bad to -base

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

vorbisdec: Improve "new headers while initialized" handling

If new headers arrive after we are initialized, we need to make
sure that they are indeed valid.

A vorbis bitstream always begins with three header packets and must
be in order.

Also some streams have unframed (invalid?) headers that might
confuse and disrupt the decoding process.

Therefore if ever we see new headers, we accumulate them and once
we get a non-header packet we check them to make sure that:
* We have at least 3 headers
* They are the expected ones (identification, comments and setup)
* They are in order
* Any other "header" is ignored

If those conditions are met, we reset and reconfigure the decoder

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

playbin3: Re-enable buffering message handling

Buffering messages are only sent for the active group (in case there
is more than one).

If the inactive group posts buffering messages we keep the last one
around and will post it once it becomes the playing one.

decodebin3: high cpu usage after eos

After eos, decodebin3 enters a loop sending eos events which causes high cpu usage.

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

decodebin3: Handle dual-output of STREAM_START/EOS

In order to flush out multiqueue, we send again a STREAM_START and
then a EOS event.

The problem was that was that we might end up pushing out on the
output of multiqueue (and therefore decodebin3) a series of:
* EOS / STREAM_START / EOS

Apart from the uglyness of such output, If decodebin3 is used with
elements such as concat on their output, they might potentially
block on that second STREAM_START.

In order to make sure we don't end up in that situation we send
a custom STREAM_START event when refreshing multiqueue (which we
drop on the output) and we don't special case EOS events on streams
on which we already got EOS.

At worst we now end up sending at most two EOS on the output of
multiqueue (and decodebin3).

playbin3: Implement gapless playback

Similar in vein to the playbin2 architecture except that uridecodebin3
are prerolled much earlier and all streams of the same type are
fed through a 'concat' element.

This keeps the philosphy of having all elements connected as soon
as possible.

The 'about-to-finish' signal is emitted whenever one of the uridecodebin
is about to finish, allowing the users to set the next uri/suburi.

The notion of a group being active has changed. It now means that the
uridecodebin3 has been activated, but doesn't mean it is the one
currently being outputted by the sinks (i.e. curr_group and next_group).
This is done via detecting GST_MESSAGE_STREAM_START emission by playsink
and figuring out which group is really playing.

When the current group changes, a new thread is started to deactivate
the previous one and optionnaly fire 'about-to-finish'.

playbin3: Use uridecodebin3 and link/reconfigure immediately

Apologies for the big commit, but it wasn't really possible to split it
in anything smaller.

* Switch to uridecodebin3 instead of managing urisourcebin and decodebin3
ourselves. No major architectural change with this.

* Reconfigure sinks/outputs when needed. This is possible thanks to the
various streams-related API. Instead of blocking new pads and waiting
for a (fake) no-more-pads to decide what to connect, we instead reconfigure
playsink and the combiners to whatever types are currently selected. All of
this is done in reconfigure_output().
  New pads are immediately connected to (combiners and) sinks, allowing
immediate negotiation and usage.

* Since elements are always connected, the "cached-duration" feature is gone
and queries can reach the target elements.

* The auto-plugging related code is currently disabled entirely until
we get the new proper API.

* Store collections at the GstSourceGroup level and not globally

* And more comments a bit everywhere

NOTE: gapless is still not functional, but this opens the way to be able
to handle it in a streams-aware fashion (where several uridecodebin3 can
be active at the same time).

urisourcebin: Add 'about-to-finish' signal

With push-based sources, urisourcebin will emit this signal when
the stream has been fully consumed.

This signal can be used to know when the source is done providing
data.

playback: New uridecodebin3 element

In the same vein as old uridecodebin except that it also
accepts a suburi and uses urisourcebin and decodebin3 internally

playbin3: Remove wrong 'notify'

Those properties doesn't exist on playbin3, don't emit a notify for that

playbin3: Remove setting 'subtitle-encoding' on decodebin

That property doesn't exist

playbin3: Clarify documentation of combiner properties

playbin3: Remove unused define

decodebin3: Use GST_GROUP_ID_INVALID

decodebin3: Don't forward already-handling SELECT_STREAMS

Upstream might respond negatively to the event, whereas we actually
handled it.

decodebin3: Add new about-to-finish signal

decodebin3: Remove unused definition

decodebin3: Don't take the lock when creating a new input

We only need to take the input lock when adding/removing
inputs from the list.

playbin3: Remove unused variable

The lock is never used

urisourcebin: Remove auto-plugging signals

They were never used and we need a better system

urisourcebin: Remove ASYNC behaviour

It is not needed in the new streams-aware world

urisourcebin: Remove 'unknown-type' signal

It was never used and makes no sense in the new streams-based world

playbin3: Disable autoplug-related code

The signals were never emitted from decodebin3. This needs
switching to a new signalling system

playbin3: fix "no-more-pads" handling

That signal is never emitted by decodebin3 and is handled differently