WIP: Revert "Revert "timecodestamper: Modify ltc-add to tc-add""

This reverts commit 05426d9298431c149807fb435cd1d632e9fd061f.

WIP: Revert "Revert "timecodestamper: LTC from audio""

This reverts commit 1998ccf1fbd586ef1dc4b1e7256bad7af8136f13.

WIP: Revert "Revert "debugutils: Added new jitterer element""

This reverts commit ccc1624d097cbfdaf9f99526c033c642c863400c.

WIP: Revert "Revert "decklink: Detect gaps on incoming stream times, issue warnings""

This reverts commit 07819afda461f5e4d77c9be9f484a7aae3c5ec8c.

openh264dec: Remove useless check

We are sure to have a frame at this point

CID #1427138

hlssink2: Remove useless if statement

message will always be present

CID #1409853

kateenc: Fix memory leak in error cases

CID #1427096
CID #1427114

srt: Remove platform dependent socket headers

SRT modules entrust `gnetworking.h` with finding right headers
for the platforms.

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

nalutils: Fix Exponential-Golomb 'ue' parsing

The algorithm we use only supports up to a maximum exponent of 31
(for a maximum resulting of 2**32 -1)

CID #1427097

opencv: allow all future 3.4.x versions

Fix-up for previous patch which I modified incorrectly.

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

opencv: allow compilation against 3.4.x

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

player: transfer ownership of info properties

The previous code would emit GObject critical warnings when the info properties
are NULL.

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

srtp: get ssrc for Application-Defined RTCP Packet

Application-defined RTCP(204) packets also have a ssrc in their header.
To find a valid stream from the packet, the ssrc should be used.

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

meson: skip translations if gettext is not available

Fix typo in gstreamer-bad-audio-1.0.pc

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

gl: local libs should come first in Makefile.am

And keep consistent order in CFLAGS and LIBS.

gl: arrange for finding headers and libs that have moved to -base

In file included from ../../../gst-plugins-bad/ext/gl/gstopengl.c:47:0:
../../../gst-plugins-bad/ext/gl/gstglmixerbin.h:25:29: fatal error: gst/video/video.h: No such file or directory

festival: Don't forward all queries

This fixes issues where wavparse would query the file size upstream
and assert because the file size is way smaller then what the WAVE
header says. This patch disable or cane a handful of queries that
make no sense to forward.

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

audioaggregator: Don't leak pads

all audioaggregator subclasses were leaking the first sink pad :)

adaptivedemux: fix pending_segment log

pending_segment shall be logged as GST_PTR_FORMAT, it's an event

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

shmsink: Block in preroll_wait on unlock

The correct behaviour of anything stuck in the ->render() function
between ->unlock() and ->unlock_stop() is to call
gst_base_sink_wait_preroll() and only return an error if this returns an
error, otherwise, it must continue where it left off!

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

audioaggregator: implement input conversion

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

smhsink: Check return values of functions working on memory

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

applemedia: don't include private cocoa gl header

gtk example: Fix cflags in Makefile.am

gl: enable openglmixers plugin in autotools again too

applemedia: fix build after gl move to -base

meson: gl: fix build of openglmixers plugin

meson: fix dependency type name to check for

meson: fix build with GstGL from gst-plugins-base as subproject

- use right fallback variable
- get platform/winsys variables from subproject instead of pkg-config

With contributions from Thibault Saunier.

gtk: don't include uninstalled header

allocatorsbad: physmem moved to -base

gl: update plugins to use GstGL from -base

gl: most of opengl plugin has moved to -base, keep mixer bits in -bad for now

Since they rely on GstVideoAggregator.

Fix detection of things again, and Meson

gl: remove GStreamer OpenGL integration library and move to -base

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

ladspa: Add Windows/OSX path support

This is to mimic LV2 and what is commonly documented over the
web. We also completely track these directories when updating
the cache now. Unlike LV2, the plugins are flat in the plugin
directories, so no need for the recursive lookup. This also fixes
support for Fedora and other architecture using lib64 as a libdir.

meson: Enable ladspa plugins compilation

lv2: Try and reflect better lilv default path

While keeping it simple, this patch tries and mimic lilv default path.
It does not matter if some path are duplicated due to symlink because in
the end it's lilv that will walk these paths. The worst case is that we
update our cache more often then strictly needed.

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

lv2: Recursively monitor the lv2 plugin path

Otherwise we might not update the cache when needed.

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

Revert "New element 'proxy' to send data to in-process pipelines"

This reverts commit 8a056af05e23ad5b81cd1dc2699ae20abcf9b91a.

Accidentally pushed this element, oops! Progress on this element is
being tracked at: https://bugzilla.gnome.org/show_bug.cgi?id=788200

meson: Use cdata.set_quoted and set LIBDIR correctly

LIBDIR must include the prefix. Also use set_quoted instead of
manually quoting because it's clearer what the intention is.

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

New element 'proxy' to send data to in-process pipelines

This plugin is useful when you want to pipe arbitrary data to
a different pipeline within the same process. Buffers, events, and caps
are transmitted as-is without copying or manipulation.

meson: Enable building LV2

Note that I only support lilv 0.22+ for now, because it's simple and old
enough. If anyone needs to use the 0.16 support it will need to be
added.

adaptivedemux: fix log integer format

range_start/range_end shall be logged as G_GINT64_FORMAT

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

dashdemux: parse xs:dateTime's timezone indicator

The lexical form of xs:dateTime is YYYY-MM-DDThh:mm:ss[Z|(+|-)hh:mm]
timezone indicator (+|-)hh:mm] needs to be parsed.

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

player: Fix-up set_seek_accurate() configuration to take a player config instead of a player instance

player: Consistently use GstVideoMultiviewFramePacking instead of Mode

The latter needs additional metadata, and the former was already used
everywhere except for the functions: in properties in playbin itself.

av1enc: Add debugging of configuration settings

In preparation of making such properties configurable
via gobject properties.

av1enc: Remove g_print for errors

And use standard debugging system for it

Add aom plugin with av1enc and av1dec element

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

opencv: handdetect: fix typo in example launch line

decklinkaudiosink: Start audio pre-rolling if the output is not started yet in render()

This seems to allow to schedule audio samples correctly at their right
times already.

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

decklinkvideosink: Only start if both sinks are set to PLAYING already

Not only if the video sink is set to PLAYING so far. Also give more
useful debug output about why we don't start, and don't start if already
started.

Also refactor the function to early-return instead of having a huge
if-else block over the whole function.

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

decklinkaudiosink: Include final flow return in debug output

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

decklinkaudiosink: Check also against the clock if we run ahead of the clock too much

The buffer level as reported by the Decklink driver is completely
unreliable, and we could otherwise easily run ahead >1s if we're
unlucky.

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

decklinkaudiosink: Implement resampling/buffer reversing for trick-modes

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

decklinkaudiosink: Re-implement around GstBaseSink instead of GstAudioBaseSink

The Decklink and GstAudioBaseSink APIs don't fit very well together,
which causes various problems due to inaccuracies in the clock
calculations and the actual ringbuffer and GStreamer's copy getting of
sync.

Problems are audio drop-outs and A/V sync getting wrong after
pausing/seeking.

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

Automatic update of common submodule

From e8c7a71 to 3fa2c9e

vtenc: fix a warning when building in MacOS 10.12

vtenc.c:564:20: error: incompatible pointer types passing 'GstVTEnc *' (aka 'struct _GstVTEnc *')
to parameter of type 'GstVideoEncoder *' (aka 'struct _GstVideoEncoder *')

cocoa: fix a warning when building in MacOS 10.12

gstglwindow_cocoa.m:186:60: error: incompatible pointer types sending 'GstGLContextCocoa *'
(aka 'struct _GstGLContextCocoa *') to parameter of type 'GstGLContext *' (aka 'struct _GstGLContext *')

gl: display: demote error level at display creation error

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

avwait: Added "avwait-status" element message

"avwait-status" is posted when avwait starts or stops passing through
data (e.g. because target-timecode and end-timecode respectively have
been reached). The attached structure includes a "dropping" boolean (set
to TRUE if we are currently dropping data, FALSE otherwise), and a
"running-time" GST_CLOCK_TIME which contains the running time of the
change.

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

meson test: Enable libs tests

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

pkgconfig: Move fixes for move of aggregator to core

h264parse: make caps writable before modifying them

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

srt: Post error message for NULL host

... instead of crash without any information

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

waylandsink: Fix memory leak of shm allocator

This fixes conflict resolution error introduced in commit:

  816d115317c522c87297109de781c63b16146493

http://bugzilla.gnome.org/show_bug.cgi?id=790042

waylandsink: Fix memory leak of shm allocator.

gst_allocator_find() needs gst_object_unref() after usage.

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

dashdemux: fix coredump issue with external AdaptationSet

fix wrong AdaptationSet list iterator

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

m3u8: Parse and use AVERAGE-BANDWIDTH attribute if available

The AVERAGE-BANDWIDTH attribute in the EXT-X-STREAM-INF tag represents
the average segment bit rate of the Variant Stream, while the BANDWIDTH
attribute represents the peak segment bit rate of the Variant Stream.
(https://tools.ietf.org/html/draft-pantos-http-live-streaming-23#section-4.3.4.2)
Using the average bit rate instead of the peak bit rate for variant switching
is more efficient and appropriate. Sometimes due to VBR encoding,
the BANDWIDTH may represent a value way above the average bit rate,
which could result to players not switching to that variant stream
 although network bandwidth is sufficiently available.

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

examples: audiomixmatrix: Remove unused part

matrix-value-array property is removed since
18917de9562d0d33cf80ca68c071eee3572d4483 commit applied

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

kmssink: Fix CropMeta support

We copy the meta's from the original buffer to the wrapper or copied
buffer.

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

kmssink: Enforce pixel aspect ratio when we cannot scale

When we cannot scale, we need to enforce the pixel aspect ratio.
This was partly implemented in the previous patch. Doing this
simplify some of the code.

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

decklinkvideosink: Wait until scheduled playback is actually stopped before continuing

Otherwise we might "start" again, just to get finally stopped and then
getting errors all over the place.

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

srt: Fix warning error

gstsrt.c: In function ‘gst_srt_client_connect_full’:
gstsrt.c:151:6: error: ‘sock’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
   if (sock != SRT_INVALID_SOCK) {

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

androidmedia: when flushing, better handle IllegalStateException received from getOutputBuffer

1. Similar to 880f3d8, don't consider not getting an output buffer as
   an error during flushing. I've seen the following sometimes when
   encoding:

   W GStreamer+amcvideoenc: java.lang.IllegalStateException
   W GStreamer+amcvideoenc:         at android.media.MediaCodec.getBuffer(Native Method)
   W GStreamer+amcvideoenc:         at android.media.MediaCodec.getOutputBuffer(MediaCodec.java:2886)

2. For amcvideodec/enc, call _find_nearest_frame (which grabs a fresh
   reference on a GstVideoCodecFrame) after we have an output buffer,
   so as to not leak the reference, in case getting an output buffer
   fails.

   Otherwise, if we get an error grabbing the output buffer, we leak
   the reference to the frame. This can cause issues with a
   v4l2bufferpool feeding the encoder not being able to clean itself
   up properly due to buffers still being marked as in-use.

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

gldownload: fix wrong enum

When compiling with clang, an enum conversion error is triggered
since GstVideoFrameFlags are not GstVideoFlags.

This patch sets GST_VIDEO_FRAME_FLAG_NONE to the added video meta.

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

docs: aggregator has moved to base

gldownload: Add missing ifdef for dmabuf and egl

This fixes the build for platforms that don't support one or the other.

gldisplay: Prefer wayland over X11

As most Wayland compositors supports XWayland, X11 backend get
selected. This also realign better GStreamer decision to what
happens with GTK and other stack out there.

gldisplay: Add missing GL API to the doc

gldownload: Add dmabuf exporting

This patch adds code to gldownload to export the image as a
dmabuf if requested.  The element now exposes memory:DMABuf as
a cap feature, and if it is selected, the element exports the
texture to an EGL image and then a dmabuf. It also implements a
fallback to system memory download in case the exportation failed.

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

Remove GstAggregator from -bad, moved to core

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

kmssink: Add display-width/height properties

This is to be used with gst_video_overlay_set_render_rectangle()
so the application can calculate a rectangle that fits inside
the display. The property changes are notify in a way that you
can watch either notify::display-width or notify::display-height
and both will be up-to-data when this is called back. Before the
element is started, the size will be 0x0.

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

kmssink: support videooverlay interface

Implement videooverlay interface in kmssink, divided into two cases:
when driver supports scale, then we do refresh in show_frame(); if
not, send a reconfigure event to upstream and re-negotiate, using the
new size.

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

kmssink: add can-scale property

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

dashdemux: remove duplicated code

remove duplicated code

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

dashdemux: fix memory leak

GPtrArray in GstActiveStream is leaked in some cases.
gst_mpdparser_free_active_stream shall be invoked to free GstActiveStream

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

meson: don't generate or install pkg-config file for gstreamer-wayland

which is not for public consumption currently (headers not installed).

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

Automatic update of common submodule

From 3f4aa96 to e8c7a71

gl/caopengllayer: use public GstGLContext instead of Cocoa-specific one

Allows keeping the GstGLCAOpenGLLayer public but not the winsys-specific
context/display/window.

Revert "gl: cocoa: sprinkle some GST_EXPORT"

This reverts commit 94d798c3337013bd902c0fd0a66f7f6842243eeb.

cocoac headers don't need to be public as all the functionality can be
provided by the base class

gl: cocoa: sprinkle some GST_EXPORT

Undefined symbols for architecture x86_64:
  "_gst_gl_context_cocoa_get_type", referenced from:
      __create_layer in libgstopengl_la-caopengllayersink.o

Might need some more in other headers, but first need to
clarify what exactly should be exported, there are some
inconsistencies (installed header files vs. funcs in docs).

autotools: remove check-exports target dep which no longer exists

win32: remove .def file with exports

They're no longer needed, symbol exporting is now explicit
via GST_EXPORT in all cases (autotools, meson, incl. MSVC).

autotools: stop controlling symbol visibility with -export-symbols-regex

Instead, use -fvisibility=hidden and explicit exports via GST_EXPORT.
This should result in consistent behaviour for the autotools and
Meson builds.

codecparsers: mpegvideoparser: remove API that was deprecated 5 years ago

Libraries in -bad are not covered by our API/ABI stability
guarantees, and to the best of our knowledge everyone using
this API has moved to the replacement APIs ages ago.

waylandsink: Add prototype for gst_wayand_pool_get_type()

waylandsink: Rollback video info changes when copying

We change the video info base on the received buffer. We need to
rollback these changes whenever we want to copy into our internal
pool of buffers.

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

waylandsink: Validate strides and offset when using FD as SHM

As SHM interface only support 1 stride, and 1 offset, we need to
make sure that there is no padding between planes for planar formats.

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