Revert "preset: Do not save deprecated properties"

This reverts commit 81e10f61231ad56ca4aa07278993b87c6ec0f058.

A mistake lead to committing it twice in a weird way.

preset: Do not save deprecated properties

It will g_warn upon deserialization and we should not use
those anyway.

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

taglist: Avoid assertions when getting tag nick from unregister tag

With serialized GstDiscovererInfos we might end up trying to use
tags that have not been registered.

preset: Do not save deprecated properties

It will g_warn upon desarialization and we should not use
those anyway.

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

tracers: log: no need to link to our internal printf implementation

The call to __gst_vasprintf() was removed in commit 1a3e218b8.

tools: gst-inspect: don't print element flags whch are always 'none'

We print the interesting flags like clocking capabilities separately
later, this function just always prints 'none', so remove it.

tools: gst-inspect: print pad properties where we know the subclass type

tools: gst-inspect: refactor way indentation is done during printing

gstinfo: Use free instead of g_free

Because

gstinfo: Don't leak array of strings

The array provided by backtrace_symbols needs to be freed.

check/buffer: Remove usless memcmp with empty size

1) checking nothing against nothing is pointless
2) memcmp needs to be provided non-NULL arguments

gstutils: Fix linear regression comparision

The check for dropping precision was wrong when sxx and syy were negative.

if they are negative then "G_MAXINT64 - val" would always overflow

The check was meant to use G_MININT64 (like in the loop contained just
after).

check: add missing harness function to symbol export list

Fixes in user code:

undefined reference to `gst_harness_add_element_sink_pad'

Also reorder harness function list to be strictly in alphabetical order and
double check the list with:

awk '{ if ($1 !~ /#define/) if ($2 ~ /gst_harness_/) { print $2 }; if ($3 ~ /gst_harness_/) { print $3} }' libs/gst/check/gstharness.h | sort

tests: include config.h and don't include unix headers

In many cases the unistd.h includes weren't actually needed.

Preparation for making tests work on Windows with MSVC.

tests: add missing license header to test_transform.c

gsttraceutils: actually disable tracing system hooks if configured

`./configure --disable-gst-tracer-hooks` didn't do anything, hooks were
always enabled regardless of the option. It works correctly in the
Meson build though.

utils: add gst_utils_dump_buffer()

Useful for debugging.

meson.build: use join_paths() on prefix

So that "/" are correct on Windows and the paths in
the .pc files are like C:/some/where and not
C:\some\where.

libs: fix indentation

pad: gst_pad_activate_mode() always succeed if same mode

Checking that the pad is in the correct mode before the parent is
checked makes the call always succeed if the mode is ok.

This fixes a race with ghostpad where gst_pad_activate_mode() could
trigger a g_critical() if the ghostpad is unparented while the
proxypad is deactivating, for instance if the ghostpad is released.
More specifically, gst_ghost_pad_internal_activate_push_default()'s
call to gst_pad_activate_mode() would fail if ghostpad doesn't have a
parent. With this patch it will return true of mode is already
correct.

gstbasetranform: replace GST_BASE_TRANSFORM with GST_BASE_TRANSFORM_CAST

To avoid a global type-lock on chain etc.

ghostpad: return TRUE if target pad was already set

The state is as it should be, so no reason to return
FALSE really, everything's good.

ghostpad: access internal pad with lock held

ghostpad: fix race-condition while tearing down

An upstream query will take a ref on the internal proxypad, and can
hence end up owning the last reference to that pad, causing a crash.

check: Add test suite time elapsed output

queue: Only calculate level if we have valid levels

Doing calculations with GST_CLOCK_STIME_NONE would result in
completely bogus levels

gstvalue: allocate slightly larger than needed tables at startup

If we pre-allocate only *exactly* as many nodes as we need for the
core types, we are practically guaranteed a re-alloc when external
code like GstVideoTimeCode or GstEncodingProfile register their
own GstValue things. So allocate a bit more than strictly needed.

pad templates: Allow specifying GType

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

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

win32: update for latest promise api changes

gst: add a promise object

An object that can be waited on and asked for asynchronous values.
In much the same way as promise/futures in js/java/etc

A callback can be installed for when the promise changes state.

Original idea by
Jan Schmidt <jan@centricular.com>

With contributions from
Nirbheek Chauhan <nirbheek@centricular.com>
Mathieu Duponchelle <mathieu@centricular.com>

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

registrychunks: Make sure we use aligned memory

This is in the same vein as for all other features. Some systems
might not allow unaligned read.

tools: gst-inspect: stop printing element state_change function

This is really not interesting at all, not sure why we print this.

tools: gst-inspect: fix double empty line after pad templates

gstpad: Make pad (de)activation atomic

The following could happen previously:
* T1: calls gst_pad_set_active()
* T2: currently (de)activating it
* T1: gst_pad_set_active() returns, caller assumes that the pad has
  completed the requested (de)activation ... whereas it is not
  the case since the actual (de)activation in T2 might still be
  going on.

To ensure atomicity of pad (de)activation, we use a internal
variable (and cond) to ensure only one thread at a time goes through
the actual (de)activation block

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

gstpad: Make calls to GstPadActivateFunction MT-safe

checking whether we already were in the target GstPadMode was being
done too early and there was the risk that we *would* end up
(de)activating a pad more than once.

Instead, re-do the check for pad mode when entering the final pad
(de)activation block.

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

paramspec: Move condition check inside the g_return

It's mostly a debug check and crash avoidance, it's better to
keep all the condition inside the macro.

paramspec: Add missing since for _spec_array()

concat: Handle single-pad use-cases

When EOS reaches concat, it will switch to the next candidate as its
activate pad.

The problem arises when there is only one sinkpad, the "active" pad
becomes NULL. This results in concat becoming unusable after it receives
a *single* EOS on its single sinkpad.

If we detect there is a single sinkpad and there is no current active pad:
* If we are waiting (from selected sink event/buffer), become the current
  active pad.
* If there is a seek request, send it upstream. We don't switch the
  active_sinkpad property at that point in time, since the seek could
  fail. If the seek succeeds, the following SEGMENT (or STREAM_START)
  will cause the pad_wait() to elect that pad as the new active one.
* Flush events get forwarded

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

concat: Make QoS forward MT-safe

In the same way it's done for other event forwarding.

latency tracer: add timestamp to tracer records

Include the timestamp of the recorded log as in the 'stats' tracer.
This can be useful, for example, to plot a graph showing the latency
over time.

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

typefindhelper: Fix overflow some more

Nothing guaranteed that off+size wouldn't exceed a 2**64 value.

Instead we reverse the operation and use a subtraction.

typefindhelper: Fix signed integer overflow

Make sure the whole calculation is done with 64bit unsigned values

(To be ready for people want to typefind exabyte files).

element: add gst_element_foreach_*pad()

Add convenience API that iterates over all pads, sink pads or
source pads and makes sure that the foreach function is called
exactly once for each pad.

This is a KISS implementation. It doesn't use GstIterator and
doesn't try to do clever things like resync if pads are added
or removed while the function is executing. We can still do that
in future if we think it's needed, but in practice it will
likely make absolutely no difference whatsoever, since these
things will have to be handled properly elsewhere by the element
anyway if they're important.

After all, it's always possible that a pad is added or removed
just after the iterator finishes iterating, but before the
function returns.

This is also a replacement for gst_aggregator_iterate_sink_pads().

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

gstleaks.c: always log leaks listing

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

net: Add new file to the meson.build

examples/netclock-replay: Fix build

netutils: Add util for setting socket DSCP

Util function for setting QoS DSCP added, to remove duplicated code in
netclientclock and nettimeprovider. Fix build error if missing IP_TOS.

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

gstvalue: Cast GST_MAKE_FOURCC arguments

To make it explicit that we are dealing with uint32 targets

Avoids erroneous  runtime error: left shift of negative value -1

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

latency-tracer: Exclude synchronization time

The goal of this tracer is to measure the processing latency between a
src and a sink. In push mode, the time was read after the chain function
have returned. As the amount of time we wait to get synched is reverse
to the amount of latency the source introduced, the result was quite
surprising.

This patch moves the latency calculation in the pre-push hook. When
there is no processing in a a pipeline (e.g. fakesrc ! fakesink), the
latency will now be 0 as it's supposed to. For pull mode, the code was
already correct. When GstBaseSink operate in pull mode, the processing
time is done durring the pull, so pull-post is the right hook. The
synchronization will happen after the pull has ended. Note that
GstBaseSink rarely operate in pull mode.

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

event: Don't allow invalid SELECT_STREAMS event creation

Asking to select no streams makes no sense and can create various
issues.

If one doesn't one any stream it should deactivate (or not use) the
element in question.

gst: Fix build with option parsing disabled

segment: update the tests

Boy scout rule. Make is a little less painful to debug the tests by using
fail_unless_equals_{uint64,int64,float} where appropriate. Ideally the large
tests would be splitted to avoid guessing data dependencies.

segment: clarify the segment docs for the duration

Revert "segment: also intialize the duration"

This reverts commit f1baaae17557fa75a9bcd940b994597714be2f74.

segment: also intialize the duration

If start and stop are set, calculate the duration and set it too.

gst-inspect: print more details for typefind and tracer features

Print full details for typefind features. Print some of the available features
for tracers and add some todos for the ones we'd like to see.

gst-inspect: reduce casting back and forth

Refactor the print_element_info() to take a GstPluginFeature. Reduces the need
to cast to and from GstElementFactory.

gst-inspect: simplify the code for printing feature info

Rename print_element_features() to print_feature_info() and move the code that
handles the ElementFactory there. This simplifies the calling code and improves
readability.

Also don't leak the features for other factories.

netclientclock: Add possibility to set QoS DSCP value

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

nettimeprovider: Add possibility to set QoS DSCP value

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

seqnum: Never return a seqnum of 0, reset GST_SEQNUM_INVALID

Various plugins use the value of '0' as an invalid seqnum value
(qtdemux for matching duplicated seek events, for example). Make
that behaviour explicit, create a GST_SEQNUM_INVALID value,
and ensure gst_util_seqnum_next never returns it.

queuearray: Fix for possible crashes due to null pointer dereferencing

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

collectpads: mention the query function in the docs as well

gstbuffer: fix meta removal in gst_buffer_foreach_meta

When updating the linked list, prev->next = next is correct
if prev is actually updated after being set to the head
of the list at the start.

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

flowcombiner: Fix version for ref and unref functions

The functions were introduced in version 1.12.1, GstFlowCombiner was
introduced in 1.4.

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

basesink: use new gst_buffer_list_calculate_size() utility function

device-provider: gst_device_provider_unhide_provider() always fails

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

meson: remove vs_module_defs

The GST_EXPORT should handle it.

Use proper GtkDoc notation for NULL/FALSE/TRUE

bin: iterate_sorted: Ensure sources are always returned last

For linked elements, the resulting gst_bin_iterate_sorted() will
properly return elements from sink to sources.

If we have some elements that are not linked, we *still* want to
ensure that we return:
* In priority any sinks
* Last of all any sources
* And in between any element which is neither source nor sink

For this to work, when looking for the next candidate element,
not only check the degree order, but if there are two candidates
with the same degree order, prefer the non-source one.

Amongst other things, this fixes the case where we activating a
bin containing unlinked sources and other elements. Without this
we could end up activating sources (which might start adding pads
to be linked) before other (to which those new source element pads
might be linked) are not activated

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

tee-test: Test a real use case

The real use case is when downstream didn't set a pool or
allocation params, in which case we expect the tee to not
create a pool or param from thin air. Dowstream setting
an pool with size=0 was in fact testing a downstream element
bug. The fact we handle that is accidental.

tee: don't create a pool if none is needed

If the aggregated size is 0 and we create a pool, the pool would provide
buffers with no memory assigned. Handle that case and skip the pool.
This was the behaviour before cf803ea9f4e3fde92c1da86ecc47444035f7c0a7.

Add a test for this scenario.

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

tee: Re-enabled alloc query + allow-not-linked test

In the unit test refactoring, the unlinked pad required to test
the different behaviour induced by "allow-not-linked" property
was removed.

  Commit e364d7944ecbab86dea73c0ee3e639e766938d36

Move all the code for this test in the proper function, and re-add
the missing unlinked pad. This makes the test useful again.

tee: split the allocation query test

Split the large allocation_query test into seperate tests. Add a setup helper
to reduce code duplication. Fix the original test that used fail_unless instead
of ck_assert_int_eq and had it accidentially working.

structure: add a todo comment

Printing NULL is confusing when the type is e.g. a GArray that is not empty.

query: doc consistency

Mention that it is the 'buffer size', like we do elsewhere.

win32: update exports file

gst: Modify behaviour of gst_get_main_executable_path

To actually return the path of the executable, not its
directory.

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

gst: API: gst_get_main_executable_path()

This is useful for plugins that need to inspect the
folder of the main executable in order to determine the
set of features they will expose, for example:

https://github.com/centricular/gstreamer-vst3

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

gst: Do not try to determine executable path on iOS.

The method used relies on "libproc.h", which is only available
on OSX.

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

gst: Fix typo in windows function name.

GetModuleFilename -> GetModuleFileName

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

win32: Update export file

plugin dependencies: fix 6cddce7663cb4b6ee061950d20365f42cb755851

There were a few errors:

* The plugin scanner now accepts executable path as an argument.
  In case it is NULL, argc == 2

* We find the executable path in init_pre instead of gst_init,
  allowing this to work when gst is initialized through the
  option group (eg gst-inspect)

* There was a semi-colon missing in the __APPLE__ #ifdef

plugin: API: GST_PLUGIN_DEPENDENCY_FLAG_PATHS_ARE_RELATIVE_TO_EXE

When a plugin declares a dependency using this flag, all the
relative paths are considered to be relative to the path of
the main executable.

We try to determine the path of the executable portably,
with implementations provided for Linux, Windows and Mac.

If retrieval of the path fails, we will not detect changes.

In order for the main executable path to be the same when
scanning a plugin in a child process, a new variable is
exposed in gst_private.h, _gst_executable_path

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

git-update: Also build $EXTRA_MODULES

Doing a git pull but not autogen.sh / make is not consistent behaviour.

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

gst-inspect: Print GstValueArray properties nicely

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

Multiqueue: don't allow dropping SEGMENT_DONE events

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

info: GstStackTraceFlags were added in 1.12

plugins: use new gst_buffer_list_calculate_size()

bufferlist: add gst_buffer_list_calculate_size()

Returns size in bytes.

bufferlist: add gst_buffer_list_get_writable()

Ensures buffer is writable. Useful if we want to change
metadata on it such as timestamps.

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

gstconfig.h.in: initial e2k arch support

This makes gstreamer buildable on Elbrus 2000.

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

info: GstDebugColorFlags are flags, not an enum

Annotate as such.

identity: Add a drop-allocation property

When enabled, this property will make the allocation query fail. This is
the same as one could have done using a tee before the tee started
implementing the allocation query.

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

tee: Allocate one more buffer when multi-plexing

This extra buffer ensure that the downstream threads are not starved
when multiplexing a stream.

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

tee: Add test for the allocation query

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

tee: Implement allocation query aggregation

This will aggregate allocation params, pool and will keep all
meta that has no parameters.

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

tee: Deprecate alloc-pad property

It has no effect, not implemented, and would lead to bad rendering.

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

pad: add test to check handled and drop probes

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

pad: Don't call remaining probes after they return DROPPED|HANDLED

If multiple probes are set on a pad and one probe returns either
GST_PAD_PROBE_HANDLED or GST_PAD_PROBE_DROPPED we need to stop
calling the remaining probes.

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