gst.h: Don't spew warnings if GST_USE_UNSTABLE_API is not defined

Only hide GstTracer and GstTracerRecord API behind GST_USE_UNSTABLE_API,
but don't spew any warnings, otherwise everyone has to define this
to avoid compiler warnings.

This reverts parts of commit 89ee5d948dff560204e6edd210c44ed2b8654b8e.

benchmarks: update the tracer benchmark and add a shell benchmark

docs/design: update tracerspec examples

docs/design: spell checking

tracer: add a GFlag for the tracer scope

Port all tracers. Add the GST_USE_UNSTABLE_API flag to the internal CFLAGS so
that we don't have to specify this for gir, docs, mkenum, ...

output-selector: Make access to the active pad and last buffer thread-safe

Both can be modified from different threads at the same time.

output-selector: Notify when the active-pad property is changing

tracerrecord: add a log record class

We use this class to register tracer log entry metadata and build a log
template. With the log template we can serialize log data very efficiently.
This also simplifies the logging code, since that is now a simple varargs
function that is not exposing the implementation details.

Add docs for the new class and basic tests.

Remove the previous log handler.

Fixes #760267

netclientclock: Check return value of g_socket_close()

CID 1348452

netclientclock: Free data after removing it from the list

Does not matter here but makes Coverity more happy. It can't
know that g_list_remove() only looks at the pointer value but
does not dereference it.

CID 1348454

GstDeviceMonitor: Clarify the behaviour of gst_device_monitor_add_filter

GstDeviceMonitor: Don't remove unmatched class filters

If no providers for a particular class could be found, then removing unmatched
filters would cause all devices to be returned instead which is not at all what
the user intended. We still return 0 for unmatched filters.

netclientclock: Fix GError memory leak in handling NTP response

Error was not released if gst_ntp_packet_receive failed.

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

win32: update exports for API additions

info: expose debugging printf functions

Other gst libraries and/or elements may want to add some debug logging to an
external debug system or implement delayed debugging for performance reasons.

Exposes the internal __gst_vasprintf as gst_info_vasprintf which has a fallback
to g_vasprintf if the debug system is disabled.

API: gst_info_vasprintf
API: gst_info_strdup_vprintf
API: gst_info_strdup_printf

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

basesrc: Only set duration/position query values in case of query success

Currently, the query values are being set even if the query itself was
determined to have failed. Fix this to ensure the values are only set in
case of a query success.

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

tracers: code clean ups

Drop some trailing whilespace. Make field order consistent.

benchmark: fix copy'n'past of the file-description comment

benchmark: add a benchmark for bgo/760267

Big suprise - GstStructure is faster than GVariant.

funnel: improve debug message

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

info: add buffer list support to GST_PTR_FORMAT

controlsource: fix GetValue and GetValueArray documentation

GstControlSourceGetValue() value paramater is a gdouble, not a GValue
and GstControlSourceGetValueArray doesn't return a GstValueArray but
an array of double.

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

tracer: harmonize the query hooks

In post hooks always pass the return value as the last param. Pass the query
also to post hooks since it is still alive.

tracers/stats: add missing parameters to callback functions

tracerutils: update #endif comment

We changed the define, but left the comment inconsistent.

tracerutils: document the tracer hook functions

Document all tracer hook function pointer together with the detail string that
one needs to use with gst_tracing_register_hook().

tracer: Use GST_DISABLE_GST_TRACER_HOOKS instead of GST_DISABLE_GST_DEBUG everywhere

Previously we used the latter one still for the tracer utility code, causing
undefined references in the resulting binary if the debugging system was
disabled but the tracer system not.

gst: Rename _priv_gst_info_start_time to _priv_gst_start_time and initialize it centrally

It's used by the debugging and tracer subsystem and in various files, make it
a central thing that is initialized independ of the existence of those
subsystems.

win32: update win32 exports

Remove the _hook_id() methods we made internal in the prev commit.

structure: log a warning if we can't serialize a field

The function always returns TRUE right now, so atleast log something.

tests: queue2: add test for fill level arithmetic overflow

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

queue2: avoid calculating fill levels multiple times

Macro expansion means we might calculate the fill level once
for the check and then possibly again for the return value.

queue2: fix fill level arithmetic overflow with large values

Based on patch by: Aleksander Wabik <awabik@opera.com>

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

tracer: make gst_tracing_register_hook_id static

We don't need to expose this as public API. Change the only plugin that was
using it.

docs: design: update list of disted files after file rename

docs: Hide NTP packet API and add GST_PTP_STATISTICS_* defines to the docs

docs: remove parent docs for GstPtpClock

Instance docs don't need to docuemnt the parent (first member).

docs: cleanup -unused.txt report for libs

The IGNORE_H_FILES can only contain files or dirs.

docs: add more core api to the right sections

Add new and documented api reported in -unused.txt to -section.txt.

docs: exclude more header from doc-scan

Exclude gst/printf/*.h and a few generated .h files from the api-scan. This
makes -unused.txt report file from gtkdoc useful again.

clock: adjust/unadjust_with_calibration() have a clock parameter but it's useless

Document this, for 2.0 we should just remove that parameter.

clock: Add gst_clock_unadjust_with_calibration()

We already have gst_clock_adjust_with_calibration() and
gst_clock_unadjust_unlocked(), having the other variant is useful.

docs: add new pad-event function to the docs

docs: hide internal tracer api from docs

Also address warnigns regarding the unstable tracer api.

docs: add the tracer to the docs

Add GstTracer and GstTracerFactory to the core docs.

docs: rename the tracer doc to part since it is now merged

tracerutils: move header to noinst section

This is internal code, that is only to be used in core.

configure: add a new option to disable the tracer hooks

This was previously done via {enable,disable}-gst-debug. Since both subsystems
are independent having separate options is better.

clock: Fix typo

clocked -> clock

netclientclock: Disconnect the "synced" signal handler from the internal clock

Not from the external one.

controller: rename new cubic interpolation mode

Don't abbreviate to 'mono' and use 'monotonic' instead.

clock: Don't allow setting an unsynced clock as master

clock: Don't try to slave unsynced clocks

They will return useless values from get_time().

ntp: The clock inherits from GstNetClientClock, not just GstSystemClock

netclientclock: Destroy a cached clock 60 seconds after its last use

There's not much lost by having the clock idle around a bit longer but it will
potentially allow anybody wanting to use the same clock server again to sync
much faster.

netclientclock: Only ever run one clock against a specific server

If multiple net/NTP clocks are created for the same server, reuse the same
internal clock for all of them. This makes sure that we don't flood the server
with too many requests and also possibly allows faster synchronization if
there already was an earlier synchronized clock when creating a new one.

nettimeprovider: Use GInitable instead of having a new() function that can return NULL

Bindings don't like that much and as we're using GIO here anyway we can as
well use GInitable for possibly failing initialization.

nettimeprovider: Mark address, port and clock properties CONSTRUCT_ONLY

They can't sensibly be changed after construction.

parse_launch: make nicer log messages

Add two macros to build nicer element/pad name strings. The macros avoid
printing "(NULL)" and print the element type in addition to the name.

parse-launch: warn when still waiting to plug sub-pipelines after no-more-pads

The parse-launch API automagically handles dynamic pads and performs delayed
linking as needed, without any feedback about whether the linking succeeded or
not however. If a delayed dynamic link can't be completed for whatever reason,
parse-launch will simply wait in case a suitable pad appears later. This may
never happen though, in which case the pipeline may just hang forever.

Try to improve this by connecting to the "no-more-pads" signal of any element
with dynamic pads and posting a warning message for the related outstanding
dynamic links when "no-more-pads" is emitted.

Fixes #760003

docs: fix some warnings and add some since markers

tracing: add some pointers about memory tracing

grammar.y: remove trailing whitespace

tests: value: test buffer serialisation/deserialisation more thoroughly

Tests data/strings as well, not just that we received
something non-NULL back.

gst: fix typo in comment

netclientclock: Implement resolval of hostnames

Just allowing IPs here is not ideal and implementing DNS resolval is easy.

ptpclock: Add read-only properties to get the master and grandmaster clock ids

caps: Add (transfer full) annotation to simplify() and subtract() return value

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

caps: add 'transfer full' annotation to caps returned by interserction functions

To make clear caller is responsible to unref them.

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

net: Add NTP and PTP clock types to the docs

This gives us the property documentation for example.

check: don't memcmp twice

Simply call fail() in the condition after the first memcmp.

Back to development

Release 1.7.1

Update .po files

po: Update translations

configure: Make -Bsymbolic check work with clang.

Update the -Bsymbolic check with the version glib has. This version
works with clang.

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

docs: update to git

core: Add g_autoptr() support to all types

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

tee: Check if parsing the name template with sscanf() was successful

If not, go back to the automatic pad numbering.

CID 1195129

queue/multiqueue: Don't special-case CAPS events in the event handlers

For CAPS events we will never ever have a FALSE return value here, so just
remove the dead code instead of causing future confusion.

Revert "queue: Illegal memory access of sink event"

This reverts commit 78614c505a2a761cb4dcb7f4e5f3e9f97c9a8e88.

The code it was fixing does not have any effect anyway and will be removed in
the next commit.

queue: Illegal memory access of sink event

Once event is pushed to pad, then queue should not access the event.
This is leading to invalid read valgrind errors

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

element: Unref event if GstElement::send_event() is not implemented

Otherwise we'll take ownership of the event only if it's implemented, and
leak the event in all other situations.

manual: Fix dynamic pipeline example

Use GST_PAD_PROBE_PASS to pass through all events other than EOS instead of
blocking on the first non-EOS event forever. Also fix a typo in a comment in
that function.

Thanks to David Jaggard for reporting this on the mailing list.

docs:  typo in the location of make_element tool

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

downloadbuffer: drop unneeded macros for G_OS_WIN32

fdsrc: drop unneeded macros for G_OS_WIN32

fdsrc: enable large file support in Android

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

fdsink: enable large file support in Android

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

queue2: fix some typos

queue2: enable large file support on Android

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

downloadbuffer: fix some typos

downloadbuffer: enable large file support on Android

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

sparsefile: drop bogus reference to file descriptor

+fix typo on return value comment

sparsefile: enable large file support on Android

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

element: unref message in _post_message when there is no implementation

'gst_element_post_message' takes the ownership of the message, so it
shall unref it when there is no post_message implementation. Otherwise
message is leaked.

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

element: Don't hold state lock all the time while sending an event

This lock seems to exist only to prevent elements from changing states while
events are being processed. However events are going to be processed
nonetheless in those elements if sent directly via pads, so protection must
already be implemented inside the elements for event handling if it is needed.

As such having the lock here is not very useful and is actually causing
various deadlocks in different situations as described in
https://bugzilla.gnome.org/show_bug.cgi?id=744040

filesrc: enable large file support in Android

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

filesink: enable large file support on Android

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

filesink: indentation fixes