tracers/stats: log optional fields instead of GST_CLOCK_TIME_NONE

Simplify the traces and avoid trace analyzer to know that ((1<<64) - 1) means
we had no value.

autogen.sh: drop a leftover docbook related bit

value: add structure intersect/union/is_subset/fixate implementations

Allows proper usage of structures in structures in caps.  Subtraction
is not implemented due to complications with empty fields representing
all possible values.

The only implementation that doesn't delegate to the already existing
GstStructure functions is the union function.

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

tests/structure: add some more is_subset checks

Explicitly testing extra/missing fields and name differences

tests: meta: add test for gst_buffer_iterate_meta*()

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

buffer: add gst_buffer_iterate_meta_filtered()

For convenience. Pretty much every user of
gst_buffer_iterate_meta() filters for a specific
api type.

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

buffer: mark gst_buffer_iterate_meta() as 'skip' for bindings

The pointer state arg won't work well, bindings can use
the foreach function instead.

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

g-i: Fix annotations for gst_event_new_select_streams() and gst_event_parse_select_streams()

A gchar is not a string.

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

gstpad: only warn on performance penalty if not using the template caps

After b76ecfd992b0d3a423cc9ace5539ecd2ba509d41 introduced
GST_PAD_FLAG_ACCEPT_TEMPLATE, the performance penalty this
message is refering to (the cascading ACCEPT_CAPS query)
only applies to the cases where !GST_PAD_IS_ACCEPT_TEMPLATE

queue/queue2: Protect against spurious condition variable wakeups

Make sure that we only wake up when we have to flush, or when this
specific query was handled.

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

queue/queue2: Ensure that the streaming thread is unlocked after deactivating the srcpad

It might happen that the srcpad task function is never called at all, in
which case unlocking everything from there will never happen.

Make sure to unlock everything another time after the task function is
definitely stopped.

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

gstvalue: add serialisation for GTypes

We need this in the GstTracerRecord. This will serialize GTypes to the typename
and vice versa.

gst: Fix building with msvc

gst: init new flags type in gst_init()

Fix 'make check' some more.

win32: update .def file for new API

info: Add a 'flags' parametter to gst_debug_get_stack_trace

This is an API break but that API has not been released yet.

We are passing a flag rather than a simple boolean as we can imagine
to implement more features in the future for example to retrieve a
stack trace for all the threads, etc..

Retrieving source file and line numbers is pretty
expensive while getting a stack trace, this new argument
allows the user to decide to retrieve a backtrace
without those infos instead which is much faster.

For example running $ GST_LEAKS_TRACER_STACK_TRACE=1 GST_DEBUG=GST_TRACER:7 \
GST_TRACERS=leaks time gst-launch-1.0 videotestsrc num-buffers=1 ! fakesink:

* With simple stack traces:

    0.04s user 0.02s system 99% cpu 0.060 total

* With full stack traces:

    0.66s user 0.23s system 96% cpu 0.926 total

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

filesrc: Set GError in another error case

When changing the location while open, properly set the GError regarding
the failure.

multiqueue: Fix overflow on get_buffering_level()

guint64 denominator factor for gst_util_uint64_scale_int() can cause overflow

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

meson: Fix build

meson: Support building with Gst debug disabled

check: Fix macro check for OS X

TARGET_OS_MAC is defined on all Apple platforms. You need to check for
!TARGET_OS_IPHONE to detect OS X (now called macOS).

typefind: Use gst_query_has_scheduling_mode_with_flags() convenience function

check: Don't try to include CoreServices.h on iOS

On iOS, we have MobileCoreServices.h but it's not really needed.

check: Don't check for malloc/realloc and try to fallback

When malloc is not available, this will set #define malloc rpl_malloc
which is implemented only inside libcheck, and not everything will link
to libcheck.

We don't really need to care too much about how malloc is implemented
and we don't care about platforms that don't implement malloc.

Add a README.txt with context for libcheck

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

libcheck: Update the compatibility code and checks

This brings us up-to-speed with the latest compatibility code from upstream
check git. For completeness, we do all the checks that upstream check does, but
we skip the snprintf/vsnprintf code because it's not straightforward (involves
running code and that is bad for cross-compilation) and not necessary for the
platforms we support anyway.

If someone really wants this, they can uncomment this and copy the relevant
checks from the check git repository.

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

libcheck: Just move libcompat files to a subdir

Makes it clearer which files are actually used in libcheck and which are used
for cross-platform compatibility. This is going to be especially useful when we
add all the libcompat fallback code that upstream libcheck has which will add
about 6 new files.

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

libcheck: port to latest check git

Upstream seems to have stopped doing releases, but we need to update for better
Windows and Visual Studio support.

This patch only updates the libcheck sources and ignores the compatibility
sources for now.

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

tracers/log: log more detail

Log the objects like we would in GST_TRACE_OBJECT. Add the hook function into
the fucntion field.

tracer/log: fix hook prototype

s/GstElement/GstPad/

tracer: move the PAD_LINK tracer hook to _pad_link_full()

This is ultimately executing the pad_link. In the previous position we missed
some links, notably ghostpads.

tracer/latency: clear qdata

When reading the qdata, clear it to avoid it being read and unreffed again.
Fixes #774332

gstconfig: Fix unaligned access support for arc and nios2 architectures

Fixes buildroot autobuild failures ([1], [2]).

[1] http://autobuild.buildroot.net/results/fbd/fbdcd90635d5ec3a62ad98a7ff93b71b8e5ecde4
[2] http://autobuild.buildroot.net/results/f3c/f3c9b0ed4ffb114221057237ce22c995b673a98b

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

uri: Add new uri API to get media fragments URI as table

As an usecase of URI fragment, it can indicate temporal or spatial
dimension of a media stream. To easily parse key-value pair,
newly added gst_uri_get_media_fragment_table () API will provide
the table of key-value pair likewise URI query.
See also https://www.w3.org/TR/media-frags/

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

helpers/gst: Get bash completion options from gst-launch

It is more likely that gst-launch is installed than ges-launch

Reported-by: Marianna Smidth Buschle <msb@qtec.com>
https://bugzilla.gnome.org/show_bug.cgi?id=775714

configure: update for removed docs/design directory

docs: Remove design doc as they have been moved to gst-docs

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

info: Properly start and end dwfl sessions when getting stack traces

We were creating a new session to retrive each line of a stack trace
and we are supposed to start it once for a whole stack trace.

And pass the whole file to gst-indent.

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

net: set clock name in the constructor

gst_net_client_clock_new() and gst_ntp_clock_new() didn't set the
"name" property.

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

gstconfig: Fix unaligned access support for microblaze and xtensa architectures

Fixes buildroot autobuild failures, for details see:
http://lists.busybox.net/pipermail/buildroot/2016-December/178895.html

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

meta: remove unnecessary padding for GstMetaInfo struct

This structure is always allocated by GStreamer, can't be
subclassed or extended, and is never allocated or used on
the stack, so we don't need any padding and can extend it
as we please.

fakesink, identity: print metas attached to buffer in silent=false mode

elements: Handle GstIterator RESYNC return value correctly in gst_iterator_foreach()

tracers/log: log messages in message category

Automatic update of common submodule

From f980fd9 to 39ac2f5

bin: Make sure to resync iterators and handle RESYNC at all in gst_iterator_foreach() calls

clock: Fix offsetting of times_temp relative to the times array

meson: Set default debug level to ERROR when running from git

meson: Add Autotools changes that weren't mirrored

commits:
a7d282d27256ad1d1a55afc37d1db7f60b040089
6fdb4df0f8c8a9e39f7f7cb73ab65306fb0517f5
1aceebd67f0161806dc3b4b68488d599290f283e

check/pipeline: Make failure message more informative

This will provide maybe a bit more insight the next time it fails

check/memory: Don't leak the custom allocator

gstutils: Fix a pad leak

When requesting a pad from a template and it's already linked, this
means it was a static pad. Since we only want to return an *available*
pad, we must return NULL ... but we must also remove the reference
we got from getting that static pad.

The "No need to unref" message (which wasn't true for quite some time)
dates back from the very very very first commit introducing the 0.10
features.

check: Fix leak in queue2 test

utils: Export linear regression calculation as public function

It is useful outside the GstClock code too.

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

Remove generated gstreamer.spec file

Likely extremely bitrotten, and we should not ship this anyway.

docs: add dataurisrc to docs and update

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

tests: filesrc: init and clear GCond and mutex

Might otherwise leak on non-Linux systems.

tests: rewrite and enable dataurisrc test

Can't use playbin for core unit tests.

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

elements: add dataurisrc to build

Moved from -bad.

dataurisrc: fix string leak in property getter

Move dataurisrc element from -bad

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

message: Ensure that the "debug" field of error/warning/info messages is valid UTF-8

The caller might pass arbitrary data here that caused the error, and
trying to set invalid UTF-8 in a GstStructure causes it to be not set at
all. Later when trying to parse it, the field will not exist and the
return value will point to invalid memory. Prevent this by storing NULL
instead.

Also print a g_warning(), the caller should never ever do this to begin
with.

common: use https protocol for common submodule

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

scripts: create-uninstalled-setup: use https protocol to clone repos

The git:// protocol is problematic from a security perspective, as
it provides no authenticity of data. https:// also works better in
environments with restricted network connectivity.

Also add CLONE_OPTS to do shallow checkouts more easily.

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

meson: Support execinfo.h on FreeBSD by using -lexecinfo

FreeBSD supports execinfo.h and backtrace* functions, but
using them requires linking with -lexecinfo.

Requires sufficiently-new meson with #1053 fixed (post-0.36).

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

tools: Remove files to be cleaned

manpages are no longer auto-generated
cov-related files should not be there (if needed we could use gitignore)

basesink: Document the interaction between unlock() and wait_preroll()

This was totally non-obvious, the kind of big problem is that subclasses must
be able to unblock their streaming thread and continue exactly where they left off
on unpause!

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

fdsink: Block in preroll_wait on unlock

The correct behaviour of anything stuck in the ->render() function
between ->unlock() and ->unlock_stop() is to call
gst_base_sink_wait_preroll() and only return an error if this returns an
error, otherwise, it must continue where it left off!

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

element: Don't increment NULL pointers

Trivial workaround for coverity false warning.

CID 1394488, 1394487.

tools: fix distcheck and .gitignore

meson: tools: install the man pages

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

tools: ship the final man pages directly, no more man pages templates

Don't use templates for the man pages, the API version change is a rare
event, so it's not really worth keeping in place the "sed" boilerplate
to have it set at build time.

Shipping the final man pages directly also makes it easer to install the
man pages with meson (in a future commit).

Note that now all the occurrences of the programs names have the API
version as a suffix.

Traditionally the example command lines looked like:

  gst-launch ...

Now they look like:

  gst-launch-1.0 ...

This reflects the actual programs names and makes it easier to copy and
paste the example commands.

Also, the .gitignore file is adjusted not to ignore the final man pages
anymore.

You may need to clean your src/build directory before pulling in this
patch.

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

controllers: add new proxy control binding

Allows proxying the control interface from one property on one GstObject
to another property (of the same type) in another GstObject.

E.g. in a parent-child relationship, one may need to
gst_object_sync_values() on the child and have a binding (set elsewhere)
on the parent update the value.

Note: that this doesn't solve GObject property forwarding and must be
taken care of by the implementation manually or using GBinding.

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

memory: log with GST_INFO instead GST_ERROR when subclass map failed.

Add unit test to ensure that.

It can be a normal execution path to do some map trials and there is
no need to worry the user in that case.

The application has to check the return value of gst_memory_map.

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

basetransform: Ensure to set the RECONFIGURE flag again if reconfiguration failed

It might've failed just because of flushing or other things, and we
should retry again on the next possibility if something ever calls in
here again.

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

meson: add_global_arguments -> add_project_arguments

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

multiqueue: Make sure not-linked streams get woken up

When running in sync-by-running-time mode, pad groups
that have exactly 1 pad and it's not-linked might never
wake up after computing a high time, as the per-pad-group
high time was only recomputed when a pad in the group
advances.

Wake those up using the global multiqueue high-time across
all other groups instead.

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

docs: misc fixes

utils: use temp var in fallback GST_WRITE_*() macros

To make sure the value is only expanded/used once, in case
there are side effects to it, and to avoid calculating it
or looking it up multiple times if there is a calculation
or lookup involved.

baseparse: Fix previous commit

Check the correct segment format value.

parse->segment.format is the format we're outputting in,
not the upstream format. Use parse->priv->upstream_format instead,
and make sure it's set in pull mode.

baseparse: Restrict query/convert responses when demuxing

If the parser is not parsing a raw elementary stream, restrict
the position, duration and conversion query replies to
things we can sensibly answer about - especially don't do
random conversions to/from bytes.

queues: Don't return negative position queries.

When subtracting queued data sizes from upstream queries
in queue, queue2, downloadbuffer and typefind, clamp the
result to not go negative, in case upstream returned
a nonsense value that's too small (as could happen if
upstream is estimating, or just broken)

Cast away const from GstMetaInfo in *_get_meta_info() functions

MSVC warns about the const in the implicit argument conversion in the
calls to g_once_init_{enter,leave}. It's OK so explicitly cast it.

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

typefindhelper: Update prototype of helper_find_suggest()

forward declaration prototype is updated to match the change in:

 5a72c23 Change some types to match their prototypes

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

win32: remove copies of generated headers

docs: remove more docbook build cruft that's no longer needed

docs/README: remove more outdated pieces of info/advice

scripts/gst-plot-traces.sh: make log parsing a bit more solid

Use grep -o to grab the log message only. This makes it work with colored log
files too. Prefilter the log to not catch tracer classes.

Update the commandline for the script in the docs.

meson: require meson 0.36 and use new `pic` arg on static libs

Removes a meson warning and some special casing we had.

Change some types to match their prototypes

Particularly note that the underlying integer type of the enum
GstTypeFindProbability is implementation dependent and may not match
guint.

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

po: update for new translatable strings

docs: remove FAQ which was moved into gst-docs module

gst-uninstalled: add GIO_EXTRA_MODULES

In case glib is installed into local prefix dir.

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

utils: faster GST_WRITE_* macros if unaligned access is possible

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

parse: better error message when linking two elements with capsfilter fails

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

Add gst_print(), gst_println(), gst_printerr(), gst_printerrln()

Useful for debugging.

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

taglist: remove `return void` in gst_tag_register

MSVC warns on this and the documentation about the warning says:

> The compiler assumes the function returns a value of type int

which is a little scary, so lets just remove the unnecessary 'return'

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

tests: add unit test for gst_adapter_prev_pts_at_offset()

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

adapter: fix distance when getting prev pts/dts at offset

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

meson: don't add_global_arguments when being built as a subproject

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