Revert "queue2: add overrun signal"

This reverts commit 8ae8b2723d0cf179a4f09b2f6c5f797e2d97034d.

It's not used anymore by anything and was considered a bad idea in general.

docs: annotate C examples as such

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

benchmark: tracerserialize: add missing return statement

tracerserialize.c:117:1: error: control reaches end of
non-void function [-Werror=return-type]

protection/harness/systemclock: move declaration out of for loop initialization

C90 compilers complain about it
error: ‘for’ loop initial declarations are only allowed in C99 or C11 mode

Also run gst-indent on systemclock tests.

harness: always set our test-clock on the harnessed element

The integration is already so tight, there is no reason to
not further formalize it!

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

harness: fix compilation

Was supposed to be squashed with 336c7bb6

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

harness: fix indentation

harness: Fix MT issues when forwarding event/query to sink harness

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

gst-uninstalled: add new -bad libraries audio, player and wayland to paths

And remove egl which no longer exists.

pushsrc: fix minor typos in header

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

harness: Add event stress test functions with callback

Similar to the stress test functions for buffers that has a callback to
create the buffer to be pushed, it's useful to have functions that use a
callback to create the event to be pushed.

API: gst_harness_stress_push_event_with_cb_start()
API: gst_harness_stress_push_event_with_cb_start_full()
API: gst_harness_stress_send_upstream_event_with_cb_start()
API: gst_harness_stress_push_upstream_event_with_cb_start_full()

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

testclock: add crank method

And use it inside GstHarness

API: gst_test_clock_crank()

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

harness: enable empty harness creation and refactor around this

Also make the testclock a member of the harness, allowing some
more interactions with the clock prior to adding elements.

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

check: fix unused parameter compiler warning

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

harness: fix the race in blocking push mode

Depending on when gst_harness_pull was called - before the buffer reached
gst_harness_chain or after we can get different behaviors of the test
with enabled blocking push mode. The fix makes the behavior always the
same. In pull function we get the buffer first, thus making sure
gst_harness_chain waits for the signal, and emitting the signal after.

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

check: Add tcase_skip_broken_loop_test

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

harness: Fix docs for stress test functions

notify is not called per buffer, but when the thread is freed.

Comment about serialized events and OOB does not make sense for upstream
events.

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

harness: Unset sink_forward_pad before tearing down sink_harness

Set the sink_forward_pad to NULL before tearing down sink_harness to
avoid that the harness tries to forward events/queries to it while it's
tearing down.

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

harness: fix up docs to reference functions properly

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

bufferpool: pass acquire params to alloc_buffer

When allocating a new buffer in the pool, both the do_alloc_buffer() and the
vmethod, alloc_buffer(), receive the parameter GstBufferPoolAcquireParams.

Nonetheless, when default_acquire_buffer() calls the do_alloc_buffer() it does
not pass the received GstBufferPoolAcquireParams, so when the user pass those
parameters they are ignored by alloc_buffer() vmethod.

This one-liner patch pass the received acquire params to do_alloc_buffer().

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

systemclock: Fix wait/unschedule race

Fixes a race where an entry is set to BUSY in
gst_system_clock_id_wait_jitter() and is UNSCHEDULED before
gst_system_clock_id_wait_jitter_unlocked() starts processing it. The
wakeup added by gst_system_clock_id_unschedule() must be cleaned up.

Two stress tests are added. One test that triggers the specific issue
described above. The second stresses the code path where a wait is
rescheduled because the poll returned early.

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

systemclock: handle unschedule of late entries

If the clockentry is too late and is unscheduled before it gets
a change to detect its lateness the wakeup count and the poll are
used but never properly cleaned up. This leaves it in a dirty state
that is going to mess with the next clock entry waiting requests.

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

tests: extend the AM_TESTS_ENVIRONMENT from check.mak

To get the CK_DEFAULT_TIMEOUT defined for all tests

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

Automatic update of common submodule

From 86e4663 to b64f03f

baseparse: fix stray discont flag set on outgoing buffers in push mode

We have no guarantees about what flags are set on buffers we take
out of the GstAdapter. If we push out multiple buffers from the
first input buffer (which will have discont set), only the first
buffer we push out should be flagged as discont, not all of the
buffers produced from that first initial input buffer.

Fixes issue where the first few mp3 frames/seconds of data in push
mode were skipped or garbled in some cases, and the discont flags
would also trip up decoders which were getting drained/flushed for
every buffer. This was a regression introduced in 1.6 apparently.

controller: Do not unset uninitiallized GValue

In case the property was not interpollable we might never initialize
the GValue, we should thus never unset it.

docs: pwg: fix missing end of line semicolon in custom meta example

uri: add guard to make sure gstreamer is initialized

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

gst-uninstalled: add new rtsp server plugin location to plugins path

baseparse: Try to generate caps on the srcpad before forwarding GAP event

To configure downstream elements and complete initial pre-rolling,
ensure we have default output caps before forwarding GAP event.

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

typefindelement: Improve English grammar

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

docs: fix an other typo in clock chapter

Shockingly I missed this bigger typo in the previos fix

docs: fix typo in clock chapter

tests:gstinfo: Fix string memory leak

info_fourcc test leaks string.

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

tests: info: make work without registry

tracer: rename GST_TRACER_PLUGINS env var to GST_TRACERS

tracerrecord: don't leak the spec structures

Change the gst_tracer_record_new() api to take the parameters the make the
spec structure directly. This allows us to own the top-level structure and
also collect the args so that we can take ownership of the sub-structures.

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

device: fix comparison in _has_classesv()

We're comparing a pointer type with '\0' here, which
probably isn't right, and the loop condition made sure
that classes[0] is != NULL already, so it's pointless.
Was probaby meant to check if the string pointed to is
not empty, so make it do that instead.

test/tracerrecord: unref objects and free string

systemclock: tests: added stress test for async order

Keep inserting alarms at the beginning of the list. Due to
https://bugzilla.gnome.org/show_bug.cgi?id=760757
alarm thread will get confused and not serve them in order.

systemclock: fixed race condition in handling alarms

When choosing the first entry from the list, gst_system_clock_async_thread
must set the entry state to busy before releasing the clock lock. Otherwise
a new entry could be added to the beginning of the list and
gst_system_clock_async_thread will be unaware and keep waiting on the entry
it has already chosen.

Also improved messages about expected state and bumped them to ERROR level
to detect unexpected state changes.

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

utils: remove duplicate check

multiqueue: two small fixes for when an existing pad is requested

Unlock when returning NULL from gst_single_queue_new(), and don't
crash with debug logging enabled if NULL is returned.

Spotted by Steven Hoving.

gstvalue: remove a half finishesh sentence in the docs

No idea what was the idea here. SO lets just drop it.

tracerrecord: Fix self->spec structure invalid free

self->spec is got using g_value_get_boxed(), which is a transfer none function.
So the same should not be freed, which is resulting in wrong behavior.

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

tracerrecord: Initialise flags to avoid wrong comparision

GstTracerValueFlags is not being initialized and the same could result in wrong
comparision and behavior. Hence initializing it to GST_TRACER_VALUE_FLAGS_NONE.

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

tests/gst/info: Fix messages glist memory leak

tests: tracerrecord: Fix messages glist memory leak

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

tracer: add an internal ptr format for tracer serialisation

We need to apply the string wrapping that value serialisation does also in the
tracer logging, otherwise we can't parse nested structures.

tracer/gststats: fix mismatch between '.class' and tracer args

Clean up from the recent changes. The logging descriptiors did not match what we logged.

queue2: Add use-tags-bitrate property

The use-tags-bitrate property makes queue2 look at
tag events in the stream and extract a bitrate for the
stream to use when calculating a duration for buffers
that don't have one explicitly set.

This lets queue2 sensibly buffer to a time threshold
for any bytestream for which the general bitrate is known.

tracerrecord: avoid overwriting value

res value is overwritten, remove the assignment.
priv__gst_structure_append_template_to_gstring () always returns TRUE
anyway.

CID 1349645

benchmarks: Disable tracerserialize benchmark on GST_DISABLE_GST_DEBUG

no gst-debugging => no tracer logging (and no pony either)

tracerrecord: Disable logging if GST_DISABLE_GST_DEBUG

Make the gst_tracer_record_log() a no-op if the gst-debug subsystem
is disabled.

value: fail flag deserialization on invalid flag names

tests: structure: fix wrong flag name in deserialization test

There is no GST_SEEK_FLAGS_NONE only GST_SEEK_FLAG_NONE (but
the deserializer silently skips bad flag names currently).

tracers: fix thread-id casts to 64-bit ints on 32-bit systems

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

gst: ref/unref new enum types in gst_init/deinit

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

win32: update exports for new flags get_type()

tracer.h: don't include private noinst header gsttracerutils.h in a public header

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

benchmark: improve script

Use a temp file for the log and fix one env-var.

tracer/stats: use the right log template

When porting we used the wrong record (copy and paste).

tracer: update the docs

Add the new enum and flags. Remove the GstTracerRecordPrivate.

gst-stats: update to latest tarcer api

The thread-ids are serialized as uint64. The 'elem-ix' got changed to
'element-ix'. Make the code a bit more robust.

tracer: use the new flags to create the optional field in the format string

This spares us explicitly listing the field in the spec. and thus hide this
implementation detail.

tracer: add a GstTracerValueFlags and replace strings

This allows us to document the flags and makes the logs a bit smaller.

message: add function guard to gst_message_set_buffering_stats()

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

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