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

meta: Deprecate GST_META_TAG_MEMORY

The GQuarks are not exported by any public API

meta: Add a #define for memory metadata

basetransform: implement a default transform_meta.
If a metadata has no dependency as shown by the tags, copy it.

meta: API: Add gst_meta_api_type_get_tags() to get all meta tags.

tests/capsfilter: Fix memory leak and compare caps directly instead of strcmp()

tests/capsfilter: Test caps-related queries and property

Update the buffering state before stalling for more data

In some cases the wait for more data was happening without updating
the buffering state, meaning the API user would not be able to notice
it should pause the pipeline and update UI to indicate that is the
case, the video would likely stutter instead.

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

basesrc: preserve seqnum on segments after seeks

The seqnum of the segment after a seek should be the same of
the seek event. Downstream elements might rely on seqnums to
identify events related to a seek.

This is particularly important when a demuxer maps a TIME seek
into a BYTES seek for upstream and it needs to identify the
corresponding segment event and map it back into TIME to push
downstream, possibly using the values from the original seek
event.

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

collectpads: Don't unref NULL GstCollectData

If a pad is removed while a collectpads element (say adder) is in a chain
function waiting to be collected, there is a possibility that an unref happens
on a NULL pointer.

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

Remove PyXML from spec file, it is not longer needed

typefind: Add missing break after handling the GAP event

Thanks to Edward Hervey for noticing.

tools: move gst-plot-timeline.py into scripts directory

So it's not in PATH in an uninstalled setup (thwarting
gst-play autocompletion).

multiqueue: Don't reduce single queue visible size below its current level

If the multiqueue has automatically grown chances are good that
we will cause the pipeline to starve if the maximum level is reduced
below that automatically grown size.

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

outputselector: Don't adjust segment->start to the current time when switching pads

This does not make any sense at all and breaks timestamp->running_time
calculations in unpredictable ways.

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

capsfilter: Delete link directly in pending_events.

When removing a segment event.

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

basesink: demote log message, don't spam INFO level when handling buffer lists

Back to development

Release 1.1.4

Update .po files

po: update translations

filesink: please gcc (avoid a warn_unused_result warning)

filesink: flush (discard data) on FLUSH_STOP

Reset the write position to 0 and truncate the file on FLUSH_STOP.

tests: filesink: small refactoring

tools: gst-launch: don't print properties being reset when shutting down

It's just noise.

basetransform: Don't push out identical caps

This avoids triggering plenty of extra code/methods/overhead downstream when
we can just quickly check whenever we want to set caps whether they are
identical or not

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

docs: flesh out gst_sample_get_buffer() a little

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

parse: make grammar.y work with Bison 3

YYLEX_PARAM is no longer supported in Bison 3.

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

sample: Add gst_sample_copy()

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

buffer: Fix gst_buffer_memcmp() where the buffer is smaller than size

Also add unit tests for gst_buffer_memcmp

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

docs: flesh out gst_element_query_{duration,position} docs a bit

taglist: handle publisher and interpreted-by tags

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

pluginloader: Don't call memcpy() with NULL src and 0 length

queue: Properly unlock the sinkpad streaming thread when deactivating the pad

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

queue2: Properly unlock the sinkpad streaming thread when deactivating the pad

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

multiqueue: Clean up after the streaming thread has stopped

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