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

timedvaluecontrolsource: Add some signals about values changes

In order for user to be able to track changes in the value set in
GstTimedValueControlSource the following signals have been added:
 * value-added
 * value-removed
 * value-changed

To be able to use a GstControlPoint to be marshalled into the signals,
the GstControlPoint structure is now registerd as a GBoxed type.

New API:
~~~~~~~
  * GstTimedValueControlSource::value-added
  * GstTimedValueControlSource::value-removed
  * GstTimedValueControlSource::value-added

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

Automatic update of common submodule

From a8c8939 to 84d06cd

message: remove duplicate gst_message_get_type() in init

Spotted by: Jan Steffens

Automatic update of common submodule

From 6e75498 to a8c8939

identity: include the actual delta in the message

Including the actual delta in the message makes it easy to see, if the new
buffer is behind or ahead and how much.

gstvalue: Tidy initialisation

Use some macros to make our value functions setup a bit
tidier, and micro-optimise a few reallocs by setting an
initial size for the global type arrays.

gst-indent: Run indent twice. Once is not idempotent, twice seems to be.

multiqueue: Wake up any waiting streams if the current one goes EOS

Otherwise we might have unlinked streams waiting.

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

typefind: simplify registration code

Remove a useless assert (we just instantiated this type). Drop the free'ing of
the extension array. As we just created the instance this is always NULL.

docs: pwd: fix typos

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

tests: Add a test for removing a bus watch

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

bus: Add a function to remove a bus watch

If a bus watch is added to the non default main context it's not
possible to remove it using g_source_remove().

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

docs: Update GstQOSType documentation a bit

Correction for who is producing data too fast, and some other minor
clarifications.

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

docs: pwg: fix two typos

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

docs: pwg: fix typo in 'Dynamic negotiation' section

The point of this example is to show how to set caps
on the source pad once it has been set on the sink pad.
So, in passthrough mode, the caps is just copied to the
source pad.

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

multiqueue: don't lock multiqueue when pushing serialized queries

If we are pushing a serialized query into a queue and the queue is
filled, we will end in a deadlock. We need to release the lock before
pushing and acquire it again afterward.

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

collectpads: Use GST_PTR_FORMAT in debug to output buffer details

Use %GST_PTR_FORMAT instead of %p in debug output so all the buffer
details are output

systemclock: fix multi-thread entry status issue

Running two threads, one executing the timer and one unscheduling it, the
unscheduled status set by the second thread is sometimes overwritten by the
first one.

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

inputselector: fix compilation

input-selector: extract some common code into helpers

input-selector: small code cleanups

Rename TIMESTAMP -> PTS. Move a var down to the scope where it is used. Use
g_queue_free_full().

inputselector: fix printf format

The padcount is uint. Also add comments to the instance vars.

baseparse: don't leak caps in gst_base_parse_process_streamheader

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

tests: baseparse: set_sink_caps vfunc should't take ownership of the caps

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

fakesrc: mark the pattern property as unused

Revert the previous commit which removes the pattern property of fakesrc because
doing so will break ABI. Bringing the property back but marking it as unused
in the property string.

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

Revert "baseparse: don't leak caps in gst_base_parse_process_streamheader"

This reverts commit 5e8b4bf085180f7a4c7ae6ec0f525baeaedd4df8.

This causes refcounting criticals in the baseparse unit test.

fakesrc: removing unused pattern option

Eventhough the "pattern" property of fakesrc can be set, it is never used. The
only pattern supported is the default 0x00 -> 0xff, and if a pattern is set by
the user it is ignored. Removing the unused property and variable.

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

queue: Add missing break in switch

queue: update segment position on GAP events to calculate levels properly

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

queue2: update segment position on GAP events to calculate levels properly

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

multiqueue: update segment position on GAP events to calculate levels properly

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

baseparse: don't leak caps in gst_base_parse_process_streamheader

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

capsfilter: Push pending events before a buffer also if upstream never configured caps but we have srcpad caps already

Otherwise we never send pending events downstream that arrive after we
configured caps on the srcpad.

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

uri: Don't unconditionally use g_list_copy_deep()

We don't depend on GLib 2.34 yet and just for this seems a bit useless.

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

uri: Include our own BSD licensed copy of strcasestr() for Windows and others

uri: Fix compiler warnings with gcc

These are actually not true.

gsturi.c: In function '_gst_uri_string_to_table.constprop':
gsturi.c:1316:27: error: 'pct_kv_sep' may be used uninitialized in this function [-Werror=maybe-uninitialized]
             for (next_sep = strcasestr (value, pct_kv_sep); next_sep;
                           ^
gsturi.c:1283:24: error: 'pct_part_sep' may be used uninitialized in this function [-Werror=maybe-uninitialized]
               next_sep = strcasestr (next_sep + 1, pct_part_sep)) {
                        ^

uri: Fix memory leak in gst_uri_join()

The merged path segments are a deep-copied list and we need to free the
contained strings too instead of just the list nodes themselves.

GstUri: Add GstUri miniobject to handle URIs in an RFC 3986 compliant fashion

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

scripts: add gst-rpicamsrc to gst-uninstalled

fixme: bump leftover 0.11 fixme comments

event: 'newsegment' to 'segment' in the docs

Brings the api-docs in sync with the 1.0 api rename.

basesrc: move the quick return up

Don't assign local vars if we skip anyway. Add logging for failure conditio

tests: parallelise 'make valgrind'

Use $(MAKE) instead of 'make' inside the Makefile,
otherwise the make will run as if -j1 had been
specified and complain about the job server not
being available, and with $(MAKE) in inherits the
parent make's settings it seems.

Upgrade common submodule for parallel check-valgrind.

Let this settle a bit before upgrading the other modules.

win32: update .def file

It's sorted. If it's unsorted, make check-exports fails.

info: remove confusing warning about running under valgrind

We're not actually doing anything differently anywhere when
we detect that we're running under valgrind, so let's not
print that confusing message that makes people wonder how
they can switch it off so they can valgrind the normal
code paths. Seeing that we're not doing that nor have done
so in the last 10 years we might just as well remove the
entire check actually.

tests: fix caps leak in baseparse unit test

flowcombiner: add a gst_flow_combiner_clear() method

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

API: gst_flow_combiner_clear()

scripts: Handle gst-python in gst-uninstalled

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

capsfilter: Remove EOS event from pending_event list on FLUSH_STOP

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

docs: Fix GstBuffer typo "memory bock" -> "memory block"

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

docs: Improve gst_buffer_get_meta() to clear up confusion

I was confused by the existence of `gst_buffer_get_meta` as it suggested
to me that you should only attach one of any type of GstMeta to a buffer.
It's perfectly fine to attach multiple from a single API so I'm
documenting that here.

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

datetime: added missing include directives

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

queue: Do not forget to release the QUEUE_LOCK in the out_flow_error path

Avoiding deadlocks!

docs: fix a small contradition in the docs

The vmethod get_size() shall return the size in 'format' as configured by
_set_format().

queue: Do not hold GST_QUEUE_LOCK while posting ERROR messages

This might create deadlocks and we need to avoid holding element
specific lock while posting messages

For example a deadlock will happen if while posting the message,
someone connected on the bus (sync) tries to DOT the pipeline.

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

queue2: do not post buffering messages holding the lock

It might cause deadlocks to post messages while holding the queue2
lock. To avoid this a new boolean flag is set whenever a new
buffering percent is found. The message is posted after the lock
is released.

To make sure the buffering messages are posted in the right order, messages
are posted holding another lock. This prevents 2 threads trying to post
messages at the same time.

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

gsturi: Remove unnecessary code

gst_uri_handler_set_uri() function has new_uri, location and colon
are not necessary, they can be removed.

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

docs: pwg: fix some links to the API docs

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

filesrc: remove FIXME

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

gst: Fix spelling error

Thank to Adrian Owen for reporting this error.

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

typefindelement: do not leak sticky events in flush_stop

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