aggregator: simplify pad_event_func for FLUSH_STOP events

We want to skip serialization for FLUSH_STOP events (apparently). We can
simplify the code to add it to the top-level conditions. There was nothing
done in the first code path if the event was FLUSH_STOP.

aggregator: drop special casing for eos

Just queue it like any other serialized event. This way we don't need to
check if there still are buffers in the queue.

Validated with the tests and gst-launch-1.0 pipelines.

aggregator: add a doc-blob for the event_func

aggregator: rename a local variable

The variable tracks wheter the queue is not empty, but num_buffers==0. That
means we have events or queries to process. Rename accordingly.

aggregator: remove commented code

The SEGMENT_DONE event does not require any special treatment. This is
commented out in 6efc106a67.

aggregator: move the comment for the locks to the lock macros

Looks like some code was inserted afterwards.

aggregator: improve section docs

Mention how data ends up in the queues. Document the relation of the pad
functions and the class vmethods to get events and queries.

audioaggregator: improve readability in offset calculation

Don't reuse the offset variables will contain a sample offset for an
intermediate time value. Instead add a segment_pos variable of type
GstClockTime for this. Use The clock-time macros to check if we got
a valid time.

aggregator: add two more tests for a sequence of data

This verifies that we handle events and queries at the head of the queue and
then buffers.

aggregator: refactor the test helper

Make the test helpers use a queue. This lets us also test sequences of events,
queries and data.

aggregator: test cleanup

Remove gst_init() from a few tests. Use _OBJECT variants in logging. Remove
arbitrary extra blank lines. Make push_event() more like push_buffer() - set
the event to NULL and add cleanup to _chain_data_clear().

audioaggregator: move comment to the place it is meant to be

This probably got shifted after some changes.

nvbaseenc: Fix Memory leak in error case

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

qt: update qmake .pro file

Update for renaming of plugin file, and add some
missing source files.

videoaggregator: use GstFlowReturn for flow returns

decklink: Print one "dropped N old frames" message, not one per frame

If we drop many frames at once, printing one message per video frame and
one per audio packet would cause a lot of disk IO. Just print a total at
the end.

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

kmssink: Fix crash on NULL dmabuf allocator pointer

Now that we are doing lazy allocation, we may endup calling _stop()
before the allocator was created. As a side effect, we need to nul-check
the pointer before calling it's method (_clear_cache()).

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

mxfdemux: Fix memory leak in error case

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

kms: Fix compilation error when libdrm < 2.4.68

DRM_RDWR was not defined until libdrm 2.4.68.  However,
in configure.ac we only require libdrm >= 2.4.55.

Seems silly to to bump minimum libdrm version for a simple
define.  Thus, define DRM_RDWR if it's not defined.

This fixes compilation error introduced in:

commit 922031b0f98bf5c7b01f4431d1c08617f65865f7
Author: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Date:   Tue Sep 12 12:07:13 2017 -0400

    kms: Export DMABuf from Dumb buffer when possible

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

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>

kmsallocator: Update GstVideoInfo.size when extrapolating

When we guess the strides, we need to also update the GstVideoInfo.size
otherwise the memory size will be set to something smaller then needed.
This was causing crash with the DMABuf exportation, since we would not
mmap() a large enough buffer.

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

kms: Export DMABuf from Dumb buffer when possible

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

kmssink: Move the mem cache into the allocator

No functional change, the cache will be later reused by the buffer
pool to keep track of the kmssink memory when exporting dmabuf.

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

kmsbufferpool: Don't check allocator pointer twice

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

kmsbufferpool: Removed unused member fd

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

kmssink: Check if we can prime export

This will be used later to decide if a DMABuf allocator should be
exposed.

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

audioaggregator: remove buffer!=NULL check

Acording to the logic this cannot happen (we already check this before). So
add a assert like we do above and remove the check. This make it clearer that
we check for the offset range.
Also remove a dead assignment since we reassign this a few lines below.

audioaggreator: update docs

Remove wrote references to collectpads. Document the units.

meson: remove vs_module_defs

The GST_EXPORT should handle it.

meson: Fix namespace and add some missing args in the gir generation

audioaggregator: pass blocksize to mix_buffer()

No need to recalc the value twice per run. Establishes that it is the same
value.

audioaggregator: rename _fill_buffer() to _queue_new_buffer()

It does not fill a buffer. Rename it and add a short comment.

aggregator: cleanup event forwarding

Don't copy the whole event struct. Set the input params when we call the
forwarding helper. Initialize the internal fields and return values in the
helper.

aggregator: simplify src_event

Avoid extra ref/unref, we have a ref and do_seek unrefs. Just return the result
as we have. This lets us remove the local var plus the label.

meson: Define GST_USE_UNSTABLE_API in the build system

This way meson and autotools both do the same thing and we don't need to
modify the code after all.

Fix autotools build broken by 25dee2f2cb21d3fdd

Build fails on GST_USE_UNSTABLE_API being redefined

Use proper GtkDoc notation for NULL/FALSE/TRUE

adaptivedemux: fix typos

meson: Build tsparser example

tsparses: Define GST_USE_UNSTABLE_API

This example is entirely based on unstable API, this avoids the expected
compilation warning.

gl: Use GstGLDisplayEGL directly instead of creating a GstGLDisplayVIVFb subclass

This simplifies the code a lot without any functional changes apart from
not closing the display connection. Closing the display connection is
not safe to do as it is shared between all other code in the same
process and no reference counting or anything happens at the platform
layer.

audioaggregator: reduce variable scope

This is a non-functional change that makes the code more alike to the previous
check. I should be more obvious when we drop a buffer.

audioaggregator: update comment for aggregate

Replace collect-pads left-over. Remove first paragraph, we're not doing this.
Remove 3), 4) since this is not per pad.

glcontext_egl: Add gstglwindow header

Otherwise, compiler complains implicit function declaration warning.

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

audiobuffersplit: Drain pending samples if the caps are changing

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

audiobuffersplit: Use new GstAudioStreamAlign API

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

glfilter: Unmap video frame in error case

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

decklinkaudio/videosrc: Put hardware reference timestamp in a reference timestamp meta

This can be useful to know on multi-channel cards which frames from
different channels were captured at the same time.

glmixer: Unmap video frame in error case

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

webpenc: Unmap video frame in failed case

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

autoconvert: Fix two memory leaks on error conditions

Both the sink_chain and sink_chain_list will neither pass
or free the buffer if the internal src pad doesn't exist yet.

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

decklink: Free the correct memory in our own buffer pool

The buffer itself is 128 bytes into the allocated memory area, to be
able to store the size and other metadata before it. Freeing the buffer
directly will make malloc moderately unhappy.

tests/check: disable all ipcpipeline tests until I find a way to make them run reliably

glvidemixer: need reconfigure output gemotry after caps renegotiated

Revert "glvideomixer: need update output geometry after src caps reconfigure"

This reverts commit d6e538dc5651fb03c85d7c7614bcf6c689f2db2f.

glvideomixer: need update output geometry after src caps reconfigure

Need update output geometry when sink caps changed and use
gst_structure_set to update caps if structure is fixed

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

ttmlparse: Ensure default showBackground behaviour is enacted

Ensure that region backgrounds are always show when tts:showBackground
is not explicitly set, in accordance with the default behavour given in
the TTML spec.

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

nvdec: Correctly set the discontinuity flag

Instead of clearing the "valid timestamp" flag, which would result in a
timestamp mismatch after a discontinuity.

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

gl/viv-fb: transform screen coordinate to viewport coordinate

In y direction, screen coordinate are opposite to viewport coordinate.

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

gl/viv-fb: fix wrong pos (x,y) calculate in queue_resize

queue_resize viewport coordinate calculate is wrong and
force queue_resize when do _set_render_rectangle in case
user input pos changed

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

glimagesink: expose should do redisplay all the time

when using internal window, window resize should work
when pause state, but expose only do redisplay when
window_id is valid. So expose should do redisplay all
the time.

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

audioaggregator: fix typo in comment

aggregator: register func for do_events_and_queries

This fixes logging the func ptr from _iterate_sinkpads().

aggregator: only set clipped_buffer to NULL if needed

aggregator: rename check_events

This function also handles queries. Update the code to loop until all events and
queuries are handled.

aggregator: add a few more comments to PadPrivate struct

aggregator: rename buffers field to data

The queue stores buffers, events and queries.

aggregator: documentaion fixes

Fix typos and remove params docs, where the param was moved.

tests: simplify audiomixer test

Use _link_many() and reuse a helper to reduce the test code.

mssdemux: fix caps leak in error code path

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

player: introduce helper method to remove seek source

this incidentally fixes a missing g_source_destroy() call in _main()

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

player: remove duplicate state_ret check in play internal

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

qmlglsink: Expose itemInitialized as property

Instead of just signalling when ready exposing the state
as a property allows us to bind at any time if player is
loaded async.

kmssink: Don't leak the dumb buffer on drain

kmssink: Don't assume buffer pool won't touch buf pointer

Just for extra safety, let's not assume that gst_buffer_pool_acquire()
won't ever touch the buf pointer.

kms: fix crash if bo allocation failed when copying to dumb buffer

If bo allocation failed we destroy the buffer and return GST_FLOW_ERROR,
but the @buffer pointer was still pointing to the address of the
destroyed buffer. gst_kms_sink_copy_to_dumb_buffer() was then trying to
unref it when bailing out causing a crash.

Leave @buffer untouched if allocation failed to fix the crash.
Also remove the check on *buffer being not NULL as gst_buffer_new()
will abort if it failed.

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

qt: fix build with qmake

Move the package defines for GST_PLUGIN_DEFINE from the
command line into the source file to avoid quoting issues
(-DPACKAGE_NAME="foo" means the quotes won't actually make
it to the compiler and then it no longer gets a string constant).

dfbvideosink: Pass the VideoInfo to gst_video_info_from_caps

Fixing the build

Request minimum buffer even if need_pool is FALSE

When tee is used, it will not request a pool, but still it wants to
know how many buffers are required.

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

hlsdemux: fix compilation with OpenSSL 1.1.0

OpenSSL 1.1.0 no longer allows stack-allocated structures;
it hides the implementation behind typedefs

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

check: Fix usage of dual probes

Using two (or more) probes on the same pad where one of the probe
returns HANDLED or DROP is tricky since the other probes might
not be called.

Instead use regular probes and a proper pad (the sinkpad already existed,
it only required to be activated and have a dummy chain function for
the events/buffers to be received/handled properly)

amc: actually use the provided application class loader

For the camera and sensor

Fixes a couple of ClassNotFound java exceptions when initializing GStreamer
off the main thread.

gl/wayland: call eglTerminate() before wl_display_disconnect()

Calling these two functions in the wrong order will result in
use-after-free inside wayland.

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

gtkglsink: expose the created display and context correctly

1. Propagate the GstGLDisplay we create
2. Add the created GstGLContext to the propagated GstGLDisplay

Otherwise with multi-branch GL pipelines involving gtkglsink, things
will fall apart and errors will be genarated somewhere.

sbcenc: Fix typo in docs

plugin: Rename libde265 to de265 for consistency

And avoids it being blacklisted

examples: Makefile.am: add ipcpipeline in SUBDIRS if enabled

ipcpipeline: cleanup header includes

We are only using read(), write(), memcpy(), strlen() and errno
in ipcpipelinecomm.c. Everything else is glib/gstreamer.

ipcpipeline: use GstPoll instead of select() to watch for socket activity

... and make that code more readable in the process

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

examples: ipcpipeline: get rid of yet another use of SOCK_NONBLOCK

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

opencv: allow compilation against 3.3.0

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

qt: Only include qtgui-config.h on qt >= 5.9.0

The file does not exist in previous versions

tsdemux: Make jp2k handling more robust and efficient

* Avoid copying the pending data and instead create a buffer directly from
  that data with the appropriate offset.
* Locate the jp2k magic to determine the exact location of the (first) frame
  data instead of assuming that the header is of an expected size

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

tsdemux: Handle quirk in jp2k es header handling

The jp2k specification (ITU-T T.800) specifies that the 'brat' box
has two fields and the second one (AUF2) can be set to 0 for progressive
streams.

The problem is that the mpeg-ts specification (ITU-T H.222.0 06/2012)
says that the AUF2 field is only present if the stream is interlaced

In order to cope with both situation, accept those next 32bit if the
stream is marked as progressive and those bits contain 0

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

examples: ipcpipeline: do not use the linux-specific SOCK_NONBLOCK flag

Use fcntl() instead to set O_NONBLOCK, which is portable.

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

tests: ipcpipeline: provide pipe2() on systems that don't have it

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

qt: the defines for QT_OPENGL_ES_2 have moved

Update the includes to account for that

qt: ensure GL_DRAW_FRAMEBUFFER

tests: ipcpipeline: make the state_changes test more deterministic

Watching the STATE_CHANGED messages is way more deterministic than
polling the state.

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

tests: ipcpipeline: fix broken exclusivity checks

In most cases we want to stop the pipeline just once, but we have
to do this from code that runs in the streaming threads and in case
we have multiple streams, we need to make sure that we do this only
once. The previous checks were broken, this should fix it.

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

tests: ipcpipeline: attempt to make tags test more deterministic

Instead of using a timeout and pushing the tags from an outside
thread, use a pad probe and push them from the streaming thread.

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