collectpads: Don't leak seek events

collectpads: implement flushing seek support

Implement common flushing seek logic in GstCollectPads. Add new
API so that elements can opt-in to using the new logic
(gst_collect_pads_src_event_default) and can extend it
(gst_collect_pads_set_flush_function) to flush any internal
state.

See https://bugzilla.gnome.org/show_bug.cgi?id=706779 and
https://bugzilla.gnome.org/show_bug.cgi?id=706441 for the
background discussion.

API: gst_collect_pads_set_flush_function()
API: gst_collect_pads_src_event_default()

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

tests: collectpads: add flushing seek tests

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

tests: collectpads: tweak stub _collect to push all buffers

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

tests: collectpads: update my email address

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

queue: Don't use gst_buffer_get_size() when possible

Makes qst_queue_locked_dequeue 20% faster

systemclock: Add gst_system_clock_set_default

Used for setting the default system clock that is obtained through
gst_system_clock_obtain(), which is sometimes needed for unit
testing.

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

typefind: use g_get_prgname() for error message

valve: proxy caps and allocation

Proxy the caps queries on the srcpad as well.
Proxy the allocation query on the sinkpad.

Automatic update of common submodule

From 865aa20 to dbedaa0

gst-inspect: Remove some dead code

memory: explicitly cast to GstLockFlags to avoid compiler warnings

segment: resurrect sanitizing start and stop for seeking

basesrc: mind boggling wrap when comparing offsets

baseparse: try first frame pts and dts for a valid start timestamp

baseparse: print proper variable in debug statement

parse: Make the FATAL_ERRORS flag also work without a GError

Also add a unit tests

gst-launch: fix potential uninitialized variable warning

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

docs: fix common typos emited/eachother/...

utils: Add some attributes and reorganize code to fix compiler warnings

gstutils.c:3659:41: error: format string is not a string literal
[-Werror,-Wformat-nonliteral]
    gchar *expanded = g_strdup_vprintf (stream_id, var_args);

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

pwg: rename the "samplerate" variable to make example code compilable

In one of the examples about gst_my_filter_setcaps() there is a variable
declared as "rate", but then the name "samplerate" is used when setting
the caps.

Use the name "rate" everywhere in gst_my_filter_setcaps().

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

doc: fix forward reference about ghost pads

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

docs: design: fix some fixes

docs: flesh out gst-uninstalled entry in faq some more

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

docs: FAQ update to mention create-uninstalled-setup.sh

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

registry: small cleanups and use object log variants more

private: remove left-over comment

The caps are saved in the registry.

registry: use g_slice_free for slice memory

Avoid memory list corruption, but g_free'ing slice memory.

design: flesh out the tracing design a little more

docs: fix typos in gstobject

docs: Gram and nit fixes for part-meta.txt

docs: Gram and nit fixes for part-element-source.txt

docs: Gram and nit fixes for part-sink.txt

docs: Gram and nit fixes for part-conventions.txt

docs: Gram and nit fixes for part-controller.txt

docs: Gram and nit fixes for part-context.txt

docs: Gram and nit fixes for part-clocks.txt

docs: Gram and nit fixes for part-caps.txt

docs: Gram and nit fixes for part-buffer.txt

docs: Gram and nit fixes for part-bufferpool.txt

docs: Gram and nit fixes for part-buffering.txt

docs: Gram and nit fixes for part-messages.txt

docs: Gram and nit fixes for part-memory.txt

net: Constify a parameter to gst_net_client_clock_new()

Even though this parameter is not used, it should be const to fit in with the
coding standards for other similar parameters. Client code already passes in
const strings under the expectation that they won’t be modified.

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

datetime: Make sure to include gst_private.h before glib-compat-private.h

We need to define the GLib log domain before including glib.h, which is
included by glib-compat-private.h.

docs: Add gst_pad_store_sticky_event to sections.txt

So it appears in the generated documentation

tests/filesrc: Set location in wrong state

Also remove incorrect comment about code possibly not being reachable
that is now exercised by the filesrc unit test.

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

parse: Fix transfer annotations for parse_launch functions.

gst_parse_launchv, gst_parse_launchv_full and gst_parse_launch_full
all return floating refs, the same as gst_parse_launch, which just
calls gst_parse_launch_full internally anyway.

Add a unit test assertion to check it's true.

Spotted by nemequ on IRC.

core: Fix max DEBUG_LEVEL incongruence on 5 vs 9

In the docs and the autocompletion logic the maximum
value jumped incongruently between 5 and 9.

caps: Skip test_subset_duplication until the bug is fixed

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

docs: Fix some reference URIs

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

caps: Add a testcase for subset checks on lists with duplicated items

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

collectpads: Call the collected function while it returns FLOW_OK

This allows us to make sure the elements is EOS and does not have
remaining buffers to be drained.

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

docs: fix function name in qos design docs

tests: use tcase_skip_broken_test() to skip broken multiqueue test

So that we get a warning in the output that reminds us that
something needs to be fixed.

check: Disable multiqueue test_output_order check

The check itself is racy.

 (CK_FORK=no GST_CHECK=test_output_order make elements/multiqueue.forever).

The problem is indeed the test and not the actual element behaviour.

The objects to push are being pulled out of the single internal queues in the
right order and at the right time...

But between:
* the moment the global multiqueue lock is released (which was used to detect
if we should pop and push downstream the next buffer)
* and the moment it is received by the source pad (which does the check)

=> another single queue (like the unlinked pad) might pop and push a buffer
downstream

What should we do ? Putting a bigger margin of error (say 5 buffers) doesn't
help, it'll eventually fail.

I can't see how we can detect this reliably.

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

value: fix caps serialization when there are caps inside caps

Wrap caps strings so that it can handle serialization and deserialization
of caps inside caps. Otherwise the values from the internal caps are parsed
as if they were from the upper one

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

pluginloader: Check errors on the proper fd

Most likely a copy-paste error from the block before.

If we're going to check for error/closed on the write fd... do it
on the write fd

docs: fix spelling of "generic" in GstBaseSrc's documentation.

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

pad: only check event order when something changed

Check the event order in dataflow only when something changed instead
of for each buffer.

Automatic update of common submodule

From 6b03ba7 to 865aa20

configure: Actually use 1.3.0.1 as version to make configure happy

Back to development

Release 1.2.0

Update .po files

context: Add test for the context caching in GstBin

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

Revert "Potential GstContext regression"

This reverts commit e658379534eb4a90b654d90f1d0bdf86f37c6e31.

This test commit should've never been pushed. Oops.

bin: Make sure to cache context types that we did not store yet

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

Potential GstContext regression

Since the refactoring of GstContext (commits
qc9fa2771b508e9aaeecc700e66e958190476f,
a7f5dc8b8af837f01782d1572379948ff62daab7,
690326f906dc82e41ea58b81cdb2e3e88b754,
d367dc1b0d4ecb37f4d27267e03d7bf0c6c06a6, and
82d158aed3f2e8545e1e7d35085085ff58f18) I am no longer able to get
a shared context for an element that is used twice in a pipeline.

I used the documentation and eglglessink as my reference for
implementing the GstContext logic.

As the code was tied to a hardware decoder, I have ported the
GstContext code to fakesink to show the problem. Using the old
API a single ExampleMgr instance is created, but using the new
API each element is creating its own instance.

collectpads: Make sure that the object lock is always taken when accessing the private pad list

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

collectpads: Use private pad list in set_flushing_unlocked

pads->data is the public list. It is dynamically rebuilt at each call to
check_collected, in check_pads to be specific. When you add a pad and
collectpads have been started, it is not added to the public list.

Thus there exists a possible race where :

1) You would add a pad to collectpads while running.
2) You set collectpads to flushing before check_collected has been called again
-> the pad is not set to flushing
3) the pad starts pushing data as downstream might not be prepared, in the case
of adder it then returns FLOW_FLUSHING.
4) elements like demuxers, when they get a FLOW_FLUSHING, stop their tasks,
never to be seen again.

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

tests: handle unscheduled entries correctly

Make the testclock return GST_CLOCK_UNSCHEDULED when an unscheduled entry is
used for gst_clock_wait() or gst_clock_wait_async().

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

gst-uninstalled: Allow specifying the checkout directory by env variable

For some rare cases, one might not be able to use the hardcoded $HOME/gst
location yet would still want to use the gst-uninstalled script as-is (which
has the benefit of being constantly updated).

For these cases, the checkout directory can be specified with the
GST_UNINSTALLED_ROOT environment variable.

Ex:
export GST_UNINSTALLED_ROOT=$HOME/somewhere/with/checkouts

And then just call gst-uninstalled directly:
$GST_UNINSTALLED_ROOT/gstreamer/gst-uninstalled

Automatic update of common submodule

From b613661 to 6b03ba7

Automatic update of common submodule

From 74a6857 to b613661

Automatic update of common submodule

From 12af105 to 74a6857

check: testclock: fix function guards

Should be g_return_*() not g_assert(), even if it's for tests only.

check: testclock: don't put code with side-effects in g_assert()

Fixes unit test failures when -DG_DISABLE_ASSERT is used.

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

gstcontext: Fix return values some more

Return value is a boolean not a pointer

context: Fix return values for gst_context_has_context_type() in assertions

Back to development

Release 1.1.90

Update .po files

context: Add convenience function gst_context_has_context_type()

po: Update translations

message: Implement getting the name of the context message types

context: Fix unit test for GstContext changes

gst-launch: Update for GstContext changes

context: Update docs

bin: Implement context caching and propagation again

message/query: Simplify CONTEXT messages/queries to only contain a single type

context: Update documentation

context: Change GstContext to contain only a single context

It was unintuitive that GstContext was actually a list of different
contexts. GstContext now is only a type string and a structure to
contain the actual context.

element: Remove GstContext caching

context: Add persistent qualifier for a context

Non-persistent contexts are removed when elements go back
to NULL state, persistent contexts are not. Applications
most likely want to set persistent contexts.

query: Make CONTEXT query upstream and downstream

event: Remove CONTEXT downstream event

This is going to be implemented with an upstream query instead
for consistency and simplicity.

gst: Stop all unused threads in GThreadPool in gst_deinit()

Since the default number of max unused threads in GThreadPool has been
changed from 0 to 2 it needs to be set to 0 to stop all threads or
valgrind will report them as memory leaks.

controlbindings: fix pspec relaxation for control source properties

The change should have been from PARAM_CONSTRUCT_ONLY to
PARAM_CONSTRUCT, otherwise bindings are affected, since
they look for the CONSTRUCT flag.

See ec55363d

queue2: Only update current level if we already downloaded a range

Otherwise queue->level is NULL and dereferencing that is not a good
idea in general.

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