identity: send gap events when dropping buffers

preset: fix incorrect preset version comparison

Use app_version if there are no system presets, so that if the
application presets are newer than the user presets they are merged.

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

inputselector: Don't dereference NULL pointer

CID 1262286

Automatic update of common submodule

From f2c6b95 to bc76a8b

preset: fix sorting presets

The glib docs are not clear on this, but the qsort man-page is - the
GCompareDataFunc does not get the strings, but pointers to them.

inputselector: fix silly GQueue iteration code

Not active by default though.

preset: don't return empty preset lists

Add a shortcut for the cases where an element implements the preset iface but
has no presets and return NULL instead of an empty list in that case.

gst-inspect: only print presets line if num-presets > 0

Also check for an empty strv.

gst-inspect: fix output for -a

Use n_print to ensure all lines are prefixed with the element name.

docs: ignore libcheck headers that use doxygen comments

docs: fix two gtk-doc warnings

One by correcting the end-of-comment marker and one by making sure the function
prototype in the header is in sync with the c file and doc-blob.

docs: Add missing interfaces to documentation

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

iterator: Fix outdated example code and accompanying documentation

GstIterator no longer returns a refcounted gpointer

inputselector: Use the same waiting function for EOS and non-EOS waiting

inputselector: Wake up all waitings pads directly if we forward the EOS event

Otherwise they might wait a bit longer unnecessarily.

Also do some minor cleanup.

inputselector: Block when receiving an EOS event on a deactivated pad

... and only unblock when either a) the pad becomes active and the event
should be forwarded or b) the active pad went EOS itself.

Otherwise it can happen that we switch from a longer track that is not EOS yet
to a shorter track that already is EOS, but the shorter track won't have any
possibility to send its EOS event downstream anymore.

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

inputselector: Keep a ref of the currently active sinkpad around

Otherwise we can't be sure that the pointer points to a still existing
pad instance after releasing the lock.

inputselector: Get the active sinkpad again after taking the lock when handling events

It might have changed in the meantime.

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

basetransform: Short-circuit CAPS query handling if transform_caps returns EMPTY caps

Both for the peer filter caps and the converted caps based on the peer caps.

If the peer filter caps are EMPTY, the peer caps query will also return
EMPTY. There's no ned to both downstream/upstream with this query.

MAINTAINERS: Update my mail address

debugutils: use a constant for the max param length

Improve readability by using a define for the max-chars. Also use the unicode
ellipsis as dot files are utf-8.

gst-inspect: print preset names

If the element supports presets and ships some, print them.
Fixes #741427

gstinfo: Add new maximum level debugging

API: GST_LEVEL_MAX

By compiling gstreamer (or plugins) with GST_LEVEL_MAX defined, only
the debugging statements at or below that level will be compiled in.

This allows compiling in some debugging (like errors and warnings) which
helps in debugging, but without the full cpu/memory overhead of all debugging
enabled.

gstcheck: fix GI annotation

Add missing : to annotation

baseparse: jump over large skips in pull mode

This bypasses the dumping of buffers we still have to do in push mode.

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

debugutils: Truncate parameter values that are too long

This removes some information from the dumps, but improves readability.

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

Automatic update of common submodule

From ef1ffdc to f2c6b95

structure/caps: Add gst_{structure,caps}_filter_and_map_in_place()

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

structure: Add simple unit test for foreach() and map_in_place()

caps: Add gst_caps_foreach() and gst_caps_map_in_place()

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

po: update for new translatable strings

check: Have autotools generate internal-check.h

Previously GStreamer got access to the libcheck interface by including
libs/gst/check/check.h which in turn included internal-check.h in the
same directory. internal-check.h was generated by copying
libs/gst/check/libcheck/check.h which in turn was generated from
check.h.in in the same directory. In this case generating
libs/gst/check/libcheck/check.h is unnecessary, in addition this file
was accidentally distributed in generated project tarballs.

Now libs/gst/check/internal-check.h is generated directly from
libs/gst/check/libcheck/check.h.in by configure. This means that the
libcheck source must include internal-check.h instead of the previously
generated libs/gst/check/libcheck/check.h. However the unnecessary
intermediate step is now skipped.

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

bufferpool: Don't check size in config validation

Pools are allowed to change the size in order to adapt padding. So
don't check the size. Normally pool will change the size without
failing set_config(), but it they endup changing the size before
the validate method may fail on a false positive.

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

bufferpool: log reason for discarded buffers

PERFORMANCE log the reason why a buffer could not be recycled in the
bufferpool.

check: Update version number of included libcheck

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

win32: update exports

docs: add new preset api

preset: add gst_preset_is_editable()

Add a function to check if the preset iface implementation is editable and
document this from the implementers perspective.
API: gst_preset_is_editable()

win32: Update def file

debugutils: Fix compiler warning

gstdebugutils.c: In function 'gst_debug_bin_to_dot_data':
gstdebugutils.c:683:530: error: 'return' with no value, in function returning non-void [-Werror]
   g_return_if_fail (GST_IS_BIN (bin));

debugutils: Add a gst_debug_bin_to_dot_data() method

This provides the dot file as a string, rather than dumping to a file.

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

typefind: minor cosmetic change

No nee to abbrev variab nam here, nicer to read if full.

typefind: use GST_BUFFER_OFFSET_NONE for buffer offset

check: Avoid requring (u)intmax_t in macros

Previously embedded libcheck versions did not depend on (u)intmax_t and
doing so would require projects using GStreamer's check framework to add
AX_CREATE_STDINT_H to their configure.ac. A workaround is to fallback to
glib types. This patch assumes that glib.h is always included before
internal-check.h which is ok since everything Gstreamer would include
gst/gstcheck.h instead of directly including internal-check.h.

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

check: Fix compilation error for iOS

libcheck includes CoreServices for its compat for clock_gettime(),
even though it never uses anything it declares. Let's remove it.

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

check: Apply GStreamer-specific patches

Reintroduced patches:

 * Make sure that fail_if(1) actually fails
   from commit 9f99d056a263e71a5e6181224829def906cf0226

New patches due to updated libcheck (based on 0.9.14):

 * Checks in m4/check-checks.m4 to cater for new dependencies
 * Conditional compile-time compat POSIX fallbacks for libcheck
 * Avoid relative paths for libcheck header files
 * Make timer_create() usage depend on posix timers, not librt
 * Rely on default AX_PTHREAD behavior to allow HAVE_PTHREAD to be used
   when checking for types and functions (like clock_gettime())
 * Avoid double declaration of clock_gettime() when availabe outside of
   librt by making compat clock_gettime() declaration conditional
 * check 0.9.9 renamed _fail_unless() and 0.9.12 later renamed it again
   to _ck_assert_failed(), so ASSERT_{CRITICAL,WARNING}() now calls this
   function
 * Remove libcheck fallback infrastructure for malloc(), realloc(),
   gettimeofday() and snprintf() since either they appear to be
   available or they introduce even more dependencies.

The result is an embedded check in gstreamer that has been tested by
running check tests in core, -base, -good, -bad, -ugly and rtsp-server
on Linux, OSX and Windows.

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

check: Import version 0.9.14

This lifts the files almost verbatim (the changes being running though
gst-indent and fixing the FSF address) from the upstream respository.
Therefore this commit reverts some GStreamer-specific patches to check
that will be reintroduced next.

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

typefind: Propagate input buffer offset

The initial buffers might have non-default offsets, make sure they get
propagated if present.

basesink: clamp reported position based on direction

When using a negative rate (rate being segment.rate * segment.applied_rate),
we will end up reporting decreasing positions, therefore adjust the clamping
against last reported value accordingly.

Fixes positions getting properly reported with applied_rate < 0.0

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

Don't compare booleans for equality to TRUE and FALSE

TRUE is 1, but every other non-zero value is also considered true. Comparing
for equality with TRUE would only consider 1 but not the others.

Also normalize booleans in a few places.

plugins: fix build on windows

gstelements_private.c: In function 'gst_writev_buffers':
gstelements_private.c:236:51: error: 'EWOULDBLOCK' undeclared

filesink: use writev() in ::render() to write out memories without merging them

filesink: implement ::render_list() function that uses writev()

fdsink: use writev() in ::render() to write out memories without merging them

fdsink: implement ::render_list() using writev()

Write out multiple buffers possibly containing multiple
memories with one writev() call, without merging the
buffer memories first, like ::render() does currently.

plugins: add helper function for writing buffers out with writev()

baseparse: update the duration variable before emitting the bus

Otherwise the application might still get the old value if it asks
between the message and the real update.

element: Fix doc and default implementation of send_event

The documentation states that gst_element_send_event is to "send an event
to an element".

Therefore we *send* upstream events to a source pad and downstream events
to a sink pad

element: Figure default send_event direction handling

If we get a downstream event we want to send it to a random SINK pad
(and vice-versa).

basetransform: Compare correct caps variable against NULL before comparing caps

Automatic update of common submodule

From f32cfcd to ef1ffdc

scripts:uninstalled: Make sur the GES TestManager is registered

So that whenever user work with GstValidate they can run GES tests
within the gst-uninstalled environment

build: Update ax_pthread.m4 and move it to common

Has some updates for Clang support (might not work with newer Clang
properly, yet), AIX support, and some misc fixes.

triggercontrolsource: Fix short description for the docs

docs: add GST_GL_* environment variables to 'Running GStreamer' section

queue2: percentage is relative to high-percent

When comparing percentage values, compare with 0-100 scale as it
has already been made relative to 0-high_percent, otherwise we mark
the queue as not buffering and report a 50% to the user. This leads to
a buffering stall as the user assumes the queue is still buffering but
it thinks it isn't.

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

multiqueue: percentage is an absolute value

multiqueue's queues stored percent value is the percentage from 0
to 100 (max-size-*) and should be compared with the requested limit
(high_percentage) set by the user and not with 100% to check if
buffering should stop. Otherwise we are only stopping buffering when the
queue gets completely full.

basetransform: Fix caps equality check

Instead of checking if our outcaps are equivalent to the previous incaps, and
if that is the case not setting any caps on the pad... compare against our
previous outcaps because that's what we care about.

Fixes some cases where the outcaps became equivalent to the previous incaps,
but the previous outcaps were different and we were then sending buffers
downstream that were corresponding to the caps we forgot to set on the pad.
Resulting in crashes or image corruption.

common: update for bison version check patch

Fix configure check with bison development version.

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

structure: don't overread input when searching for "

When searching for the string terminator don't read past the ending
0-byte when escaping characters.
Add unit test for various escaping cases.

pad: fail dropped queries

Previously, dropping a query from a pad probe would deem the
query succeeded, and the caller might then assume the query's
results are valid, and thus dereference an invalid object
such as a GstCaps.

We now assume dropped queries did not succeed. Dropped events
and buffers are still deemed a success.

Added back after previous revert, as it's been double checked.

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

Revert "pad: fail dropped queries"

This was pushed by mistake along with an unrelated patch.

This reverts commit c7103ce4b8c1da7dcfbcf2ec83a42a376fb896e1.

baseparse: allow skipping more data than we currently have

This can be useful for skipping large unwanted data, such as
large album art, when we know the size of it from a metadata
header.

pad: fail dropped queries

Previously, dropping a query from a pad probe would deem the
query succeeded, and the caller might then assume the query's
results are valid, and thus dereference an invalid object
such as a GstCaps.

We now assume dropped queries did not succeed. Dropped events
and buffers are still deemed a success.

task: Fix pause/stop race condition

If a task thread is calling pause on it self and the
controlling/"main" thread stops the task, it could end in a race
where gst_task_func loops and then checks for paused after the
controlling thread just changed the task state to stopped.
Hence the task would actually call func again even though it was
both paused and stopped.

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

gstobject: Don't check booleans for equality in the unit test

Every value other than 0/FALSE is TRUE, == TRUE will only check for 1.

gstobject: Add gst_object_has_parent()

Adds gst_object_has_parent, which works like gst_object_has_ancestor
but does not ascend further.

API: gst_object_has_parent()

basetransform: Don't bother the subclass with setting the same caps multiple times

basesink: Don't bother the subclass with setting the same caps multiple times

basesrc: Don't bother the subclass with setting the same caps multiple times

structure: remove conditional for G_VALUE_COLLECT_INIT

This API is in glib since 2.24 and we currently require 2.32 and already use
this unconditionally elsewhere.

preset: remove commented code

The GQuark was never used.

gstconfig: Put gstconfig.h into $(libdir)/gstreamer-1.0/include

It's architecture dependent and should not be placed into the include
directory as the assumption is that all those headers are architecture
independent.

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

uri: Fix gobject-introspection warnings

gsturi.c:997: Error: Gst: Skipping invalid GTK-Doc comment block:
/** private GstUri functions **/
^
gsturi.c:1179: Error: Gst: Skipping invalid GTK-Doc comment block:
/** RFC 3986 functions **/
^

dataqueue: Fix gst_data_queue_new() description.

Reword the function docs, which haven't made any sense since
gst_data_queue_new_full() was removed a few years ago.

basesink: Answer the query position when receiving it from upstream

Currently we are just returning FALSE, but we do have the information
we should just answer the query the same way as when answering through
the GstElement.query vmethod default implementation.

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

capsfilter: Add an optional delayed caps change mode

In this mode we accept previously set filter caps until
upstream renegotiates to something that is compatible
to the current filter caps.

This allows dynamic caps changes in the pipeline even
if there is a queue between any conversion element
and the capsfilter. Without this we would get not-negotiated
errors if timing is bad.

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

toc: minor code clean-up

And get rid of g_list_prepend/g_list_reverse
anti-pattern while we're at it.

gst: ensure GStreamer initialization debug message is displayed

The GST_INFO ("initialized GStreamer succesfully") is currently at the end of
gst_init_check which isn't guaranteed to be run since GStreamer can be
initialized by using init_pre and init_post directly from GOptionContext like
gst-launch does. Ensure this message is displayed by moving it to init_post.

doc: Do not use deprecated gtk-doc 'Rename to' tag

GObject introspection GTK-Doc tag "Rename to" has been deprecated, changing to
rename-to annotation.

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

gst-inspect: add G_PARAM_DEPRECATED to known flags

Display 'deprecated' instead of flag value when using G_PARAM_DEPRECATED
in element properties.

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

tests: refactor tools check a little

Use an array of constant strings so if arguments get
removed from it they are not considered leaked, and
valgrind is happy. Still some stuff leaking in GLib
though.

tests: fix out-of-bounds memory access in bytereader unit test

Caught by -fsanitize=address / libasan.

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

gst: make gst_init() thread-safe

Because we can, and there isn't really any
reason not to do so.

tests: fdsrc: don't ignore return value of write()

Causes compiler warnings on some systems.

tests: fix fdsrc test corner case

Make pipe socket non-blocking, so we don't
end up being blocked in a write on the pipe
while the src is eos and not reading data
any more, and thus we never unblock and never
notice that we're done. This would happen
quite reliably on the rpi.

Automatic update of common submodule

From 84d06cd to 7bb2bce

debugutils: Trivial typo fix

basesink: don't unlock mutex that is not locked

Fixes 'Attempt to unlock mutex that was not locked'
warning with newer GLibs when sink is shut down in
certain situations. Triggered by the decodebin
test_reuse_without_decoders unit test in -base
sometimes, esp. on slower machines.

win32: update .def for new _get_type() function for GstControlPoint

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