tools: remove outdated completion script

+ Remove the associated test

https://bugzilla.gnome.org/show_bug.cgi?id=744877#c21

segment: add helper to get negative running-time

Add a helper method to get a running-time with a little more features
such as detecting if the value was before or after the segment and
negative running-time.

API: gst_segment_to_running_time_full()

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

segment: fix offset handling with non 0 start

The position in the segment is relative to the start but the offset
isn't, so subtract the start from the position when setting the offset.
Add unit test for this as well.

funnel: Add support for buffer lists

baseparse: remove duplicate code

These are already freed by gst_base_parse_clear_queues()

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

pluginloader: Fix indention

baseparse: reset skip on segments and discontinuities

Large scale skip is an optimization, and thus it is safer to
stop skipping than to continue. Clear skip on segments and
discontinuities, as these are points where it is possible that
the original idea of "bytes to skip" changes.

multiqueue: Don't grow queue infinitely if only one pad is linked

This was introduced by
https://bugzilla.gnome.org/show_bug.cgi?id=719893
https://bugzilla.gnome.org/show_bug.cgi?id=722891
but it doesn't make any sense at all and causes huge memory leaks.

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

basesink: handle empty buffer list more gracefully

Don't abort, just ignore it. It's like a buffer
without memories.

adapter: minor optimisation for gst_adapter_take_buffer_list()

Try to allocate buffer list with a suitable size from the
beginning to avoid having to re-alloc the buffer list array.

tests: add unit test for gst_adapter_take_buffer_list()

adapter: add gst_adapter_take_buffer_list()

API: gst_adapter_take_buffer_list()

Add new streamiddemux binaries to .gitignore

collectpads: avoid multiple calls to gst_buffer_get_size() in macro

adapter: avoid multiple calls to gst_buffer_get_size() in macro

element: properly escape percent sign in documentation

buffer: Use the correct enum type to fix a compiler warning

gstbuffer.c:522:58: error: implicit conversion from enumeration type 'GstBufferFlags' to
      different enumeration type 'GstBufferCopyFlags' [-Werror,-Wenum-conversion]
  if (!gst_buffer_copy_into (copy, (GstBuffer *) buffer, flags, 0, -1))
       ~~~~~~~~~~~~~~~~~~~~                              ^~~~~
gstbuffer.c:534:46: error: implicit conversion from enumeration type 'GstBufferCopyFlags' to
      different enumeration type 'GstBufferFlags' [-Werror,-Wenum-conversion]
  return gst_buffer_copy_with_flags (buffer, GST_BUFFER_COPY_ALL);
         ~~~~~~~~~~~~~~~~~~~~~~~~~~          ^~~~~~~~~~~~~~~~~~~
./gstbuffer.h:433:31: note: expanded from macro 'GST_BUFFER_COPY_ALL'
  ...((GstBufferCopyFlags)(GST_BUFFER_COPY_METADATA | GST_BUFFER_COPY_MEMORY))
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

defs: update defs

meta: Add `GstNetControlMessageMeta`

GstNetAddress can be used to store ancillary data which was received with
or is to be sent alongside the buffer data.  When used with socket sinks
and sources which understand this meta it allows sending and receiving
ancillary data such as unix credentials (See `GUnixCredentialsMessage`)
and Unix file descriptions (See `GUnixFDMessage`).

This will be useful for implementing protocols which use file-descriptor
passing in payloaders/depayloaders without having to re-implement all the
socket handling code already present in elements such as multisocketsink,
etc.  This, in turn, will be useful for implementing zero-copy video IPC.

This meta uses the platform independent `GSocketControlMessage` API
provided by GLib as a part of GIO.  As a result this new meta does not
require any new dependencies or any conditional compliation for
portablility, although it is unlikely to do anything useful on non-UNIX
platforms.

allocation: Allow allocation pool without size

This allow proposing a number of buffers required even if the size
of buffer is unfixed. This is often the case for encoded formats.

bufferpool: Don't stop the pool in set_config()

Don't stop the pool in set_config(). Instead, let the controlling
element manage it. Most of the time, when an active pool is being
configured is because the caps didn't change.

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

basesink: drain on allocation query

Allows buffers to be reclaimed when caps is to be renegotiated so
that bufferpools can be stopped. As the allocation query is
serialized all buffers have been already drained from the pipeline,
except this last_sample one.

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

basesink: when draining, deep copy the last buffer to unref old memory

Use gst_buffer_copy_deep() to force the copy of the underlying
memory instead of possibly doing a shallow copy of the buffer
and just referencing the memory

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

gstbuffer: add gst_buffer_copy_deep

A variant of gst_buffer_copy that forces the underlying memory
to be copied.

This is added to avoid adding an extra reference to a GstMemory
that might belong to a bufferpool that is trying to be drained.
The use case is when the buffer copying is done to release the
old buffer and all its resources.

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

bus: Use g_list_free_full() instead of manually unreffing and freeing

Also unref the messages, not the GList nodes.

bus: Fix another case where we hold the object lock while unreffing a message

bus: Unreferencing messages outside the lock

Shouldn't take the lock while unreferencing messages, because that may cause
more messages to be sent, which will try to take the lock and cause the app to
hang.

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

utils: Add gst_bin_sync_children_states()

gst_bin_sync_children_states() will iterate over all the elements of a bin and
sync their states with the state of the bin. This is useful when adding many
elements to a bin and would otherwise have to call
gst_element_sync_state_with_parent() on each and every one of them.

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

printf: handle unsigned modifier for long long

Otherwise, an unsigned integer will be displayed as a signed one if we
use internal print, ie HAVE_LONG_LONG_FORMAT is not defined.

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

streamiddemux: Reset pad counter after removing all pads

streamiddemux: Add streamiddemux element

Demultiplex a stream to multiple source pads based on the stream ids from the
stream-start events. This basically reverses the behaviour of funnel.

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

win32: update

tests: add some basic unit tests for our printf stuff

To test new %I32 support.

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

printf: add support for %I32

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

info: move category level threshold check into log function dispatcher

Minor optimisation: check category log level earlier in the
log function dispatcher and not only in the default log
function.

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

typefind: Reset segment when deactivating pull mode or not running in pull mode

We use the segment format to detect if we run the streaming thread or not.
Without resetting we might believe we do so, although we only did in the past
and are now running in e.g. push mode.

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

check: Use mkstemp instead of tempnam if possible

Using tempnam() is deprecated, this gives warning and fails the build
with -Werror.

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

basesink: clean up the need_preroll variable

Based on patch from Song Bing <b06498@freescale.com>

Don't just set the need_preroll flag to TRUE in all cases. When we
are already prerolled it needs to be set to FALSE and when we go to
READY we should not touch it. We should only set it to TRUE in other
cases, like what the code above does.

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

funnel: handle GAP event to forwards sticky events into downstream

If no data is coming and funnel receive GAP event, need to forwards sticky events
into downstream if it needs.

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

check: duplicate code branches

CID #1226446

Fix double semicolons

win32: update exports

flowcombiner: add a gst_flow_combiner_update_pad_flow() method

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

API: gst_flow_combiner_update_pad_flow()

flowcombiner: add a gst_flow_combiner_reset() method

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

API: gst_flow_combiner_reset()

basesrc: Fix typo in debug message

info: avoid malloc/free if log object is NULL

info: move __FILE__ path shortening into default log handler

Instead of always shortening the __FILE__ path, even if the
log message is not actually printed, which might happen if
the log level is activated but the category is not, only
shorten the path if we're actually going to output it and
if it looks like it needs shortening. Log handlers had no
guarantee that they would get a name instead of a path
anyway on any architecture, so it shouldn't be a problem.

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

info: shorten __FILE__ on all platforms

This is useful not only for MSVC, but also with gcc/Linux
when doing cross-compilation builds and out-of-tree builds.

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

docs: clarify min-latency wording in part-latency.txt

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

win32/common/gstconfig.h: removed libxml include directive

This is a leftover from 0.10 and not needed anymore.

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

queue2: Signal the sinkpad thread if a flow error happened

It might still be waiting for a query to be handled, or the queue to become
empty again for the next item. Also if downstream returns FLUSHING, flush the
queue like we do in queue and multiqueue.

queue: Wake up the query function on errors from the loop function

Otherwise we might wait forever for serialized queries to be handled as the
loop function is stopped and as such we will never ever dequeue the query and
handle it.

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

utils: improve warning when linking  elements without common ancestor

This comes up quite a lot and it's a common mistake, so let's
try to improve the warning message a little.

input-selector: Drop custom latency query handling

The default latency query handler now implements this logic

gst-unsinstalled: Add ges-launch manuals path to MANPATH.

pad: Don't fail latency query on unlinked pads

A single unlinked pad can make the latency query fail across the
pipeline, which is probably not desirable. Instead, we return a default
anything goes value.

Perhaps we should also be emitting a gst_message_new_latency() when a
PLAYING element has one of its pads linked.

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

baseparse: Don't emit errors on EOS if we saw GAP events

If we saw GAP events (meaning the streams is advancing) before we get
EOS, we should not post an ERROR, since it is not fatal.

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

gstvalue: Make sure GST_FOURCC_ARGS produces printable characters

Some systems will crash if we use non-printable characters in print/debug
statements.

Make sure that GST_FOURCC_ARGS never does that

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

gstutils: remove incorrect Fixme comment

If the checks were changed to using g_return_if_fail() the GST_DEBUG lines
about the specific failure would be lost.

gstutils: remove obsolete Fixme comment

gst_pad_link_filtered() is very long gone and current
gst_element_link_pads_filtered() doesn't apply to this Fixme comment.

gst-uninstalled: add adaptivedemux paths from -bad

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

docs: remove dead link

Remove dead link to wiki page for SubmittingPatches

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

helpers: Fix install of completion-helper.

By applying the supplied transformation to the program name,
for example --program-prefix.

completion-helper: Add missing DESTDIR

Otherwise doing "make install DESTDIR" will try to write to
/usr/share/...

completion-helper: Add filtering by klass and sink caps.

multiqueue: avoid returning downstream GST_FLOW_EOS from previous segment to current upstream segment

flowcombiner: fix documentation comment typo

baseparse: drain segment upon SEGMENT_DONE to ensure proper event order

baseparse: clean up some bogus commented code

completion-helper: Append $(EXEEXT) to the name of the moved file.

Fixes the build on Windows
(https://ci.gstreamer.net/job/cerbero-cross-mingw32/1742/console)

Fix distcheck

Disable bash completion during distchecking otherwise
it may try to install into a system path and fail.

Dist new data directory

Fixes 'make dist'

bash-completion: Implement in a different way.

+ Gets installed
+ Uses a helper tool, gst-completion-helper, installed in
  bash-completions/helpers.
+ Adds a common script that other tools can source.

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

GstDeviceMonitor: keep alphabetical order

Revert "element: set pads need-parent flag to false when removing"

This reverts commit 1911554cff2c4a11772b541a8215a80c728b1097.

This breaks the functionality of GST_PAD_FLAG_NEED_PARENT, the reason for this
flag is that if a pad is removed from a running element, you don't want
functions (such as chain or event) to be called on the pad without a parent set.
This can happen if you remove a request or sometimes pad from a running element.

I don't see the code that caused this in tsdemux, but if it needs to unset
the flag on remove, it should do it itself and then make sure that the parent
exists in any pad function.

check: cast element in ASSERT_SET_STATE.

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

inputselector: Use a separate query for upstream pads and let it fail if one upstream query fails

pad: If the latency query fails for one of the pads, it fails overall

queue: Remove unused boolean parameter from internal functions

queue: Add unit test for buffer list and time level handling

queue: Add support for buffer lists

queue2: Count the number of buffers in a buffer list for updating the current levels

instead of just assuming one buffer.

message: revive async delivery message before bus thread can run unref

Revive message in dispose handler before we signal the bus thread,
otherwise the bus thread might be woken up and unref the message
before we had a chance to revive it yet.

tests: bus: add unit test for async message delivery

message, bus: fix async message delivery

Async message delivery (where the posting thread gets blocked
until the message has been processed and/or freed) was pretty
much completely broken.

For one, don't use GMutex implementation details to check
whether a mutex has been initialized or not, esp. not
implementation details that don't hold true any more with
newer GLib versions where atomic ops and futexes are used
(spotted by Josep Torras). This led to async message
delivery no longer blocking with newer GLib versions on
Linux.

Secondly, after async delivery don't free mutex/GCond
embedded inside the just-freed message structure.

Use a new (private) mini object flag to signal GstMessage
that the message being freed is part of an async delivery
on the bus so that the dispose handler can keep the message
alive and the bus can free it once it's done cleaning up
stuff.

info: nicer buffer offset printing when offsets are not set

Print unset offsets as 'none' instead of humongous numbers,
for better readability.

pad: Only initialize GValue to a type once, not on every retry

Otherwise we'll get warnings like this:
cannot initialize GValue with type 'gboolean', the value has already been initialized as 'gboolean'

gstutils: check uri before using it in gst_pad_create_stream_id_internal

If an element implements wrongly the URI query and set the uri to NULL and if
the element calls gst_pad_create_stream_id at some point, it will lead to crash
as the uri is not supposed to be NULL in the gst_pad_create_stream_id_internal
function.

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

timedvaluecontrolsource: Do not wrongly send value-removed

And avoid freeing something we do not own

gst-uninstalled: Set GST_VALIDATE_PLUGIN_PATH

basesrc: fix documentation and debug message after latency updates

Changes docs and message according to latency handling fix

fdsrc: use g_ascii_strtoull() to convert size string in uri

sscanf() doesn't handle G_GUINT64_FORMAT well on mingw64 it
appears, leading to compiler warnings.

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

pad: gst_pad_iterate_internal_links() can return NULL if there are none

pad: Return NULL instead of FALSE for pointers

pad: Implement more useful default handling for the LATENCY query

Before we just took the values from the first pad that succeded the query,
now we accumulate the results of every sinkpad properly and return that
result.

design/part-latency: Minor logic fix

The maximum latency will be the element's minimum latency or bigger,
not bigger than the element's minimum latency or bigger.

Improve and fix LATENCY query handling

This now follows the design docs everywhere.

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

design/part-latency: Add more details about min/max latency handling

These docs missed many details that were not obvious and because of that
handled in a few different, incompatible ways in different elements and base
classes.

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

tests: Fix clock regression test

Fix up the values the test is checking for now that
the clock regression returns parameters starting from
the end of the regression range.

netclock: Don't update the clock when it desynch

Add a hold off when the clock calibration suddenly loses synch,
as it may be a glitch, but also make sure we update if it stays
desynched for more than a few seconds