testclock: add support for waiting and releasing multiple GstClockIDs

In order to be deterministic, multiple waiting GstClockIDs needs to be
released at the same time, or else one can get into the situation that
the one being released first can add itself back again before the next
one waiting is released.

Test added for new API and old tests rewritten to comply.

pad: don't access unowned and possibly already freed event

Don't print the name of the event when ownership is given away.

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

inspect: print structure values of properties

input-selector: Hold pad object lock when changing tags

Avoid spurious crashes when tags are retrieved just as
new ones arrive.

gstbuffer: Fix range iteration

We want to iterate over items idx to idx + length

We use the len variable as the corrected number of memory to iterate
and then properly go over all items.

Fixes the issue where specifying any idx different from 0 had no effect

Spotted by clang static analyzer

structure: error out when trying to fixate a fraction near an invalid target

event: Update running time in QoS based on the pad offsets

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

pad: Apply pad offsets on all events, not just segment events

event: Add running-time-offset field to all events

Events passing through #GstPads that have a running time
offset set via gst_pad_set_offset() will get their offset
adjusted according to the pad's offset.

If the event contains any information that related to the
running time, this information will need to be updated
before usage with this offset.

utils: avoid dividing by zero when multiplying y/z by 0/x

The gcd of 0/x is 0, and this is then used as a denominator.

multiqueue: And actually run the other tests again

multiqueue: Wake up the queues if limits are changing in a way that would unblock the queue

multiqueue: Add test for checking if pads are waked up when limits are changed

baseparse: Fix memory leak

Queued frames were not released after being pushed, this
caused a leak of the GstBaseParseFrame structure.

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

queue2: use g_strerror() instead of strerror()

Need UTF-8 encoding.

Revert "check: only call setpgid on valid child PIDs"

This reverts commit b9313afc75b68d986e473b76b55543456857912b.

This should be fixed in upstream libcheck instead. We want
to keep diff of our local copy to upstream libcheck
to a minimum.

queue2: warn if we can't remove our temporary file

It's not fatal though, so do not error out.

Coverity 1037121

check: only call setpgid on valid child PIDs

Coverity 206186

filesrc: no need for a translated message for impossible error cases

The message is too technical anyway, the default message works
just fine here as well.

filesrc: catch failure to seek back to zero after seek test

This should never happen theoretically, but since a transient
failure would get us to silently read wrong data, it's worth
erroring out. And it silence this:

Coverity 206034

parse: Don't dist the bison and flex generated headers

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

debugutils: Handle caps field values being NULL

GST_DEBUG_BIN_TO_DOT_FILE() would cause a segfault whenever it encountered an
element's caps that had a field value being NULL. Such fields are successfully
handled e.g. by GST_*_OBJECT(), and with this patch so does
GST_DEBUG_BIN_TO_DOT_FILE(). Even if string fields with a NULL value are
not supposed to be valid in caps, such caps can be created.

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

baseparse: Remove always-true-checks

a gsize is guaranteed to be positive on all systems since it's an
unsigned value.

CID #1037147

adapter: Remove always-true-checks

a gsize is guaranteed to be positive on all systems since it's an
unsigned value.

CID #1037145
CID #1037146

gstvalue: Prevent division or modulo by zero

The step can end up being zero if the underlying value isn't a valid
range GValue.

In those cases, return FALSE.

We don't use g_return*_if_fail since it will already have been triggered
by the above-mentionned _get_step() functions.

Spotted by Coverity.

gstvalue: Fix comparison of int/int64 range

Checking step three times seems unnecessary.
A similar bug was fixed for double range in
commit 3ea6b04c10b10fde9d62190068f274b940edef07

inputselector: fix build with debug cached buffers enabled

gstinputselector.c:818:5: error: format not a string literal
and no format arguments [-Werror=format-security]

baseparse: Make sure to set the DISCONT flag on the first buffer of each GOP in reverse playback mode

basesrc: removing duplicated inner if

The inner if replicates the same code of the outer and is useless
as flag_segment will always be true.

Found by coverity.

gst-launch.1: Playbin2 is dead, long live playbin

Looks like that was the last remaining mention in core ...

gst-inspect: Add missing \n in output

scripts: gst-uninstalled: gst-plugins-gl libs got merged into -bad

pad: Include event type in debug output when delaying a sticky event because of not-linked

baseparse: Fix splitting and reversing of GOPs in reverse playback mode

We iterate the current discont group backwards and push each GOP forwards,
starting from the last one. However if the first buffer in the current
discont group is a keyframe, we will keep it around until next time,
which is far from ideal. Just push it.

launch: place the deep-notify on the right pipeline

If the toplevel bin is not not a pipeline, we place the bin in a
pipeline. Also make sure that we connect to the deep-notify of this new
pipeline because we will g_signal_handler_disconnect() from it later.

basesink: copy last_sample on DRAIN

Make sure we don't hold a ref to a buffer from before the DRAIN query by
making a copy of the last_buffer.

buffer: don't clear TAG on NULL buffer

When the buffer fails to copy, don't clear the TAG on the NULL pointer.

queue2: fix event/preroll deadlock

The qlock is released between popping a buffer from the queue
and pushing it. When this buffer causes the sink to wait in
preroll, this lets a query see that the queue is empty, and
push the query then wait for it to be serviced. However, this
will not be done till after peroll, and this will thus block.
If upstream was waiting on buffering to reach 100% before
switching to PLAYING, a deadlock would ensue.

We fix it by refusing the query when buffering, as per Wim's
recommendation on IRC.

tests: make netclientclock test faster and less flaky

gst: init new flag types to pass make check

queue2: Update buffering status and maybe post buffering message right when enabling buffering

multiqueue: Update buffering status and maybe post buffering message right when enabling buffering

fakesink: Update positions we're checking for after a state is lost

basesink: Update start time when we lose our state

Otherwise we report not the correct position while the state is lost.

basesink: Don't clip the reported position in PAUSED to after the last buffer end timestamp

Otherwise we jump forward when pausing, and go backwards a bit again
when resuming playback.

win32: Update exports for GstToc loop

win32: Update export for gst*device symbols

devicemonitor: Use local includes and use gst_private before anything

Should fix build issues on BSD

gst-inpect: Print device monitor

device: Add "klass" to GstDevices

devicemonitor: Make classes into pure strings

Instead of having strings & flags, make them just strings

Fix extended message handling with gst_bus_pop_timed_filtered()

Make sure extended message types don't get accidentally matched
when not asked for in the mask

gstbin: Avoid pointless object lock forwarding messages.

Every instance of calling bin_do_message_forward() first took the
object lock, so that bin_do_message_forward() could drop it and
then reclaim. Instead, only take the object lock afterward where
needed.

collectpads: When seek flushed, immediately set eospads to 0

This prevents situations where a first branch would get seeked and
receive a buffer before all branches got seeked, and thus collected
would get called based on EOS from the previous segment.

As a consequence, during the process of seeking, don't decrease
the eospads number when a FLUSH_STOP is received.

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

collectpads: Unref peer pad

collectpads: Forward seek events to the peer directly

Taken from the adder seek handling code.

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

toc: expand GstTocEntry with loop fields

Add loop_type and repeat_count fields to GstTocEntry plus setters and getters.
This allows to represent edit-lists in a toc as well as loops in instruemnts (wav, xi).

API: gst_toc_entry_set_loop
API: gst_toc_entry_get_loop

globaldevicemonitor: update for new message API

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

globaldevicemonitor: Add device monitor aggregator

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

tests: add test for extended message types and gst_bus_timed_pop_filtered

message, bus: do extended message types slightly differently

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

gst: fix indentation

devicemonitor: Add GstDeviceMonitor and related

Also add GstDevice and GstDeviceMonitorFactory
And add code to the registry to save them

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

message: Add GST_MESSAGE_EXTENDED

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

pad: actually return data.ret

The return value from gst_pad_forward() is the aggregated return value from the callback and the callback returns FALSE to continue iterating.

pad: forward return value from gst_pad_forward

Instead of ignoring the return value and always return TRUE pass the already agregated result back.

win32: fix make-check by running 'make update-exports'

tests: multiqueue: fix eos count on test for not-linked case

From the test case:

/* This test creates a multiqueue with 2 streams. One receives
 * a constant flow of buffers, the other only gets one buffer, and then
 * new-segment events, and returns not-linked. The multiqueue should not fill.
 */

If one of the queues goes EOS and the other returns NOT_LINKED the stream
can be considerered EOS as a NOT_LINKED means that one of the branches has no
sink downstream that will block the EOS message posting.

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

adapter: Adapt gst_adapter_copy() for bindings

This is done by introducing a new gst_adapter_copy_bytes() call that
returns a GBytes structure.

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

docs: fix multiqueue docs for new template names foo_%d -> foo_%u

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

docs: update plugin docs

pad: simplify gst_pad_link_get_name() and fix Since marker

Has added benefit that compiler might warn if more values
are added to the enum.

pad: add debug helper for GstPadLinkReturn names

Add a helper like gst_flow_get_name() for GstPadLinkReturn. Use this in core.
API: gst_pad_link_get_name()

tee: use store_sticky events add add more logging

Use the pad as object for logging to get more context. Use
gst_pad_store_sticky_event() instead of sending the event. This avoids a warning
as here the pad is not yet linked and we actually don't want to send anyway.

queue2: if buffering is disabled while buffering, post 100% message

Avoids stall waiting for buffering to reach 100%

queue2: remove unused variable

buffering_iteration was never used

queue: queue2: preserve last flow result when pushing events

Avoids mistakenly returning _OK when downstream is still
_NOT_LINKED on subsequent received pad pushes

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

multiqueue: if buffering is disabled while buffering, post 100% message

Avoids stall waiting for buffering to reach 100%

multiqueue: do not reset last push result when pushing events

Use the last result as a default when pushing a item from a single queue,
otherwise the status gets reset to _OK when pushing events.

This causes problems when mistakenly activating a not-linked stream
that is being ignored upstream as it is not being used (adaptive
scenarios), it will make the multiqueue post a buffering message
on a pad that won't receive buffers

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

buffer: invert meaning of GST_BUFFER_FLAG_TAG_MEMORY

It's nicer to only have it set when something noteworthy
happened and otherwise unset.

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

pad: fix gst_pad_add_probe() return value docs

Also fix comment typos and add more detail in the logs.

fakesink, identity, fakesrc: fix debug printing if TAG_MEMORY buffer flag

The IN_CAPS flag does not exist any more.

ghostpad: use gst_pad_get_peer to acquire a reference to the target pad

This ensures that the lock of the internal pad is held while referencing
it's peer (= the target pad), which ensures that the peer is not
going to be unlinked/destroyed in the meantime.

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

ghostpad: hold a reference to the target pad while unlinking it

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

.gitignore: Ignore gcov intermediate files

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

docs: Fix typos and remove unknown annotations

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

docs: use the new markdown for ordered list.

This was plain text that had all list items one after the other (including a
repeated number). Now it will atleast look good when processed with gtk-doc
1.20.

Automatic update of common submodule

From fe1672e to bcb1518

docs: fix problems introduced by c068b225fef5a9bf0

- Fix failing build
- Drop added trailing whitespace

buffer: add return values to g_return_if_fail

FIxes previous commit.

docs: convert the examples to use gtk-doc markup, instead of docbook

The gtk-doc markup is less intrusive and better handled when creating docs for
language bindings. The titles (where used) where not adding much.

docs: add some more new API do docs

bufferpool: only release buffers with writable memory

Check if the memory is writable before releasing the buffer into the
pool again.
Add unit test for this scenario.

buffer: add function to check writability of memory

Check if memory is writable in a buffer and thus is exclusively owned by
this buffer.

bufferpool: Use TAG_MEMORY to check memory before releasing

Tag allocated buffers with TAG_MEMORY. When they are released later,
only add them back to the pool if the tag is still there and the memory
has not been changed, otherwise throw the buffer away.
Add unit test to check various scenarios.

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

buffer: add a new flag to track memory changes

Add a flag to check if the memory changed in a buffer.

buffer: remove wrong comment

Refcount and writability are not related for memory objects.

bufferpool: refactor free_buffer

Make a do_free_buffer method to also decrements the number of allocated
buffers. Stop will now be successful when all buffers are freed.

Automatic update of common submodule

From 1a07da9 to fe1672e

debugutils: add a legend to pipeline dumps

We use a couple of symbols to represent states/flags. Add a short explanation for them.