appsink: Reuse sample object in pull_sample if possible

Simple optimization to reduce memory allocations.

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

parsebin: Post STREAM_COLLECTION on EVENT_STREAM_COLLECTION

Otherwise decodebin won't get notified about STREAM_COLLECTION comming
from the sources and thus will never get informored about it. Without
being informed about the stream collection decodebin won't be able to
select any streams. It ends up not creating any output for the streams
defined from outside parserbin.

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

egl: fix build when using RPi EGL

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

meson: move some dependency checks to the top level

The newly-added build definitions for test/icles relied
on dependencies that were only defined when the examples
are enabled, thus breaking meson build -Ddisable_examples=true

video: Don't corrupt caption GstMeta

the meta initialization function is provided *after* the base implementation
fields have been set so do *NOT* reset them otherwise it would result
in corrupted GstMeta.

Instead explicitely set our fields to the default values we actually want.

Automatic update of common submodule

From f0c2dc9 to ed78bee

gl: Define default value for GST_GL_HAVE_WINDOW_GBM

Thus, silent compiler's warning:
"GST_GL_HAVE_WINDOW_GBM" is not defined, evaluates to 0 [-Wundef]

tag: fix some GIR annotations

Mostly related to out and array parameters.

tests: make getaddrinfo suppression more generic

So it works for me on debian sid.

valgrind supps: ignore leaks where pango is the first frame

valgrind supps: ignore intentional leak in GL test

valgrind supps: conditional moves in nvidia-glcore

Not much we can do here

valgrind supps: getaddrinfo leaks when resolving addresses

https://lists.debian.org/debian-glibc/2016/03/msg00243.html

audioaggregator: unref converted buffer after gst_buffer_replace

alsamidisrc: unref buffer_list before early return

tests/audiomixer: fix caps leak

valgrind supps: ignore bash leaks

gst_install_plugins_* can instantiate bash

urisourcebin: Avoid unreffing a pad we are not owning

expose_output_pad takes ownership of the pad.

uridecodebin3: free_play_items when READY_TO_PAUSED failed.

We will never go through the PAUSED_TO_READY transition if
that is the case, and thus never free the play items.

playbin3: fix leak of recursive mutex

valgrind suppressions: ignore XInitThreads leaks

As we load that symbol dynamically, valgrind gets confused
when it leaks and reports the leak against an unrelated library
and an unknown (??) symbol.

To address that, put the loading and calling of that symbol
in a separate function, and ignore any malloc leak happening
in that function.

valgrind supps: ignore leak in XextCreateExtension

Searching for that specific function in other suppression files
on the Internet shows a few projects ignoring it

eg: https://github.com/XCSoar/XCSoar/blob/master/valgrind.suppressions

vorbisenc: do not map input buffer in WRITE mode

meson: port tests/icles

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

audiovisualizer: Only fixate pixel-aspect-ratio if the field exists

It's optional.

appsrc: Documentation fix in gst_app_src_push_sample_internal

Make it clear that it doesn't take ownership of the sample.

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

audiovisualizer: Fixate pixel-aspect-ratio to the closest value to 1/1

playback: Add support for CEA 608/708 CC overlay elements

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

video: Add support for VANC and Closed Caption

This commits add common elements for Ancillary Data and Closed
Caption support in GStreamer:

* A VBI (Video Blanking Interval) parser that supports detection
  and extraction of Ancillary data according to the SMPTE S291M
  specification. Currently supports the v210 and UYVY video
  formats.

* A new GstMeta for Closed Caption : GstVideoCaptionMeta. This
  supports the two types of CC : CEA-608 and CEA-708, along with
  the 4 different ways they can be transported (other systems
  are super-set of those).

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

pbutils: Add Closed Caption caps description

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

audiobasesrc: posting errors should be always be safe

Don't try to signal an error in the ringbuffer if it hasn't been
allocated yet.

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

audioringbuffer: Don't spam INFO for every buffer

This makes GST_DEBUG=4 outputs too spammy, and such frequent messages
are meant to go into DEBUG or TRACE anyway.

tests: Enable tests for videodecoder

The tests pass fine here so don't see any reason to keep them disabled.

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

encodebin: Always respect encoding profile preset factory name

And fail if it is not present.

encodebin: Inform about missing elements in the "text" error field

Instead of the debug field, that information should be directly
exposed to end users.

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

video: Fix/Add section documentation

docs/libs: The big spring cleanup

* Explicitely specify which headers aren't to be included in gtkdoc-scan
  This is essentially all the headers that are not installed and only
  for internal/local usage. This also includes the orc-generated headers.
* Remove all symbols/sections that are no longer present (due to accurately
  scanning only the headers we need).
* Add or expose sections which weren't previously exposed
* Make sure the "unified" library headers (ex: gst/video/video.h) are used
  everywhere applicable. Only use the specific headers where applicable
  (such as the GL-implementation-specific objects)
* Add all documentation which was not previously exposed in the right sections
* Update 'types' file to get as many runtime information as possible

This brings down the number of unused symbols to 15 (from over 300).

libs: Documentation cleanup

* Fix wrong naming, wrong types and typos
* Add missing sections
* Add missing documentation for entries
* Explicitely mark private structure entries
* Remove items that never existed

tools: Fix discoverer deprecation

The only information provided by "misc" was the missing plugins which
is already handled with another API

libs: Add deprecation guard

Those functions have been marked as deprecated

check: Remove old structures from ABI check

Just in case someone still tries to run it on 32bit osx

videorate: In reverse playback mode, don't output the first buffer with ts=segment.stop

Instead go backwards before segment.stop based on the framerate or the
next buffers end timestamp. Otherwise the first buffer will usually be
dropped because outside the segment.

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

gl: pick up GstVideo-1.0.gir from local build dir

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

tools: play: fix leaving STDIN in non-blocking mode after exit

gst-play-1.0 sets STDIN to non-blocking mode to have the input
characters read as soon as they arrive.

However, when gst_play_kb_set_key_handler() gets called from
restore_terminal() it forgets to restore the STDIN blocking status.

This can result in broken behavior for cli command executed in the same
terminal after gst-play-1.0 exited.

It turns out that putting STDIN in non-blocking mode is not even the
proper way to achieve the desired effect, instead VMIN and VTIME in
struct termios should be set to 0.

Let's do that, and don't mess with the STDIN blocking mode now that it's
not necessary.

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

encoding-profile: fix gi annotation for gst_encoding_profile_find

gl: Disable glmixerbin for the time being too

Otherwise we have one copy in gst-plugins-bad and one (unused) here,
which makes static linking unhappy.

encodebin: Use gst_object_unref() instead of g_object_unref()

... for better debugging (e.g., tracing refcount)

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

audio-aggregator: Check return values

And copy over already-parsed information

CID #1427140

meson: libs: use gnome.mkenums_simple() to generate enumtypes files

This way we no longer need custom wrapper scripts or template files.

gl/gbm: Initialize window handle (= gbm surface) like other window systems

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

viv-fb: Include gstglfuncs.h to fix cross compilation errors

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

dmabufallocator: Fix build if LINUX_DMA_BUF_H is missing

allocators: Add DMABuf synchronization

When doing CPU Access, some architecture may require caches to be
synchronize before use. Otherwise, some visual artifact may be
visible, as the CPU modification may still resides in cache.

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

video: Set correct value in g-i annotations for tile related mask constants

video: Include gstvideoaffinetransformationmeta.h in video.h

pbutils: Include gstaudiovisualizer.h in pbutils.h

Back to development

Release 1.14.0

Update docs

oggstream: protect against out-of-bounds read

We need at least 17 bytes of data for a valid flac header

oss-fuzz #6974

tests: audiomixer: remove unistd.h include

Not needed and breaks the build with MVSC.

audiostreamalign: Mark the whole type as new in 1.14

pkg-config: Add orc-0.4 to Requires.private of gstreamer-audio and gstreamer-video

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

Release 1.13.91

Update docs

meson: install new audio-prelude.h

docs: update api decorators to ignore

video: GST_EXPORT -> GST_VIDEO_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.

tag: GST_EXPORT -> GST_TAG_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.

sdp: GST_EXPORT -> GST_SDP_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.

rtsp: GST_EXPORT -> GST_RTSP_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.

rtp: GST_EXPORT -> GST_RTP_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.

riff: GST_EXPORT -> GST_RIFF_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.

pbutils: GST_EXPORT -> GST_PBUTILS_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.

gl: GST_EXPORT -> GST_GL_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.

fft: GST_EXPORT -> GST_FFT_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.

audio: GST_EXPORT -> GST_AUDIO_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.

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.