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

queue2: Clean up after the streaming thread has stopped

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

queue: Clean up after the streaming thread has stopped

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

parse: Add GST_FLAG_NO_SINGLE_ELEMENT_BINS

This makes gst_parse_bin_from_description() return an element instead of
a bin if there's only one element. Also changed gstparse.c to use this,
so gst-launch won't create superfluous bins.

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

query: return NULL when parsing uri redirection that was not set

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

buffer: Update since marker for gst_buffer_extract_dup() to 1.0.10

queue2: don't crash on EOS if queue is empty

Fixes spurious crash in test_simple_shutdown_while_running
unit test.

queue2: don't change global buffering state from within query handler

When a buffering query is handled it uses the get_buffering_percent()
function to get some statitics. Unfortunately this function also
calculates whether the queue should be buffering and adapts the
global queue2 state in case of state transitions from/to buffering
(including whether a buffering message was posted on the bus!).

This means that there is a race which can cause buffering messages
to never posted if the global state changes happen as a result of aa
query instead of resulting from bytes flowing in/out.

Spotted by Sjoerd Simons.

Change to only query state in get_buffering_percent() and update
state only in update_buffering().

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

queue2: update buffering when changing capacity

When the capacity of the queue changes, make sure we post an updated buffering
message because we might suddenly have completed the buffering stage.

Free thread pools in gst_deinit()

check: Call gst_deinit() at exit of all processes

clock: simplify internal gst_clock_return_get_name() helper

basesrc: improve flush-start handling

Use custom code to implement flush-stop, we can't reuse the set_flushing code
because we can't touch the live_playing flag and we need to signal the
streaming thread.

basesrc: stop flushing in flush-stop

basesrc: handle flush better

Unlock the streaming thread when flushing so that we can
insert the flush-stop correctly.

.gitignore: ignore .dirstamp

check: Don't use nodist headers on gir scanner

Just creates noise and bogus symbols

gst: minor docstring fixups to make g-i happy

note: the #ifndef move is actually a move of the "SECTION" docstring

.gitignore: Ignore files from automake test-driver

baseparse: Add a property to disable passthrough

In some specific cases (like transmuxing) we want to force the element
to actually parse all incoming data even if the element deems it is not
necessary.

This property simply ignores requests from the element to enable passthrough
mode which results in processing always being enabled.

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

dataqueue: add gst_data_queue_push_force

Adds a variant of the _push function that doesn't check the queue limits
before adding the new item. It is useful when pushing an element to the
queue shouldn't lock the thread.

One particular scenario is when the queue is used to serialize buffers
and events that are going to be pushed from another thread. The
dataqueue should have a limit on the amount of buffers to be stored to
avoid large memory consumption, but events can be considered to have
negligible impact on memory compared to buffers. So it is useful to be
used to push items into the queue that contain events, even though the
queue is already full, it shouldn't matter inserting an item that has
no significative size.

This scenario happens on adaptive elements (dashdemux / mssdemux) as
there is a single download thread fetching buffers and putting into the
dataqueues for the streams. This same download thread can als generate
events in some situations as caps changes, eos or a internal control
events. There can be a deadlock at preroll if the first buffer fetched
is large enough to fill the dataqueue and the download thread and the
next iteration of the download thread decides to push an event to this
same dataqueue before fetching buffers to other streams, if this push
locks, the pipeline will be stuck in preroll as no more buffers will be
downloaded.
There is a somewhat common practice in dash streams to have a single
very large buffer for audio and one for video, so this will always
happen as the download thread will have to push an EOS right after
fetching the first buffer for any stream.

API: gst_data_queue_push_force

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

sysmem: Only copy the requested part of memory instead of the complete source memory

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

query: add Since markers for new API and add to exports file

query: fix annotation for gst_query_parse_uri

query: add new redirection uri the URI query

query: add some missing 'transfer none' gi annotations

The current documentation is controverse, while it states that the
returned value is valid only while the query is is valid, which presumes
a 'transfer none' policy. But the tooltip for the 'out' annotation
states the default is 'transfer-full'.

Add the missing 'transfer none' annotations to fix this.

bytereader: Accelerate MPEG/H264 start code scanning

Accelerate MPEG/H264 start code scanning using Boyer-Moor bad character
heuristic.

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

pipeline: g-i: allow clock to be NULL in gst_pipeline_use_clock()

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

controller: fixes int overflow with properties that span +-INT_MAX

When the range for a property is defined as -INT_MAX-1 .. INT_MAX, like
the xpos in a videomixer the following expression in the macro
definitions of convert_g_value_to_##type (and the equivalent in
convert_value_to_##type)

v = pspec->minimum + (g##type) ROUNDING_OP ((pspec->maximum - pspec->minimum) * s);

are converted to:

v = -2147483648 + (g##type) ROUNDING_OP ((2147483647 - -2147483648) * s);

(2147483647 - -2147483648) overflows to -1 and the net result is:

v = -2147483648 + (g##type) ROUNDING_OP (-1 * s);

so v only takes the values -2147483648 for s == 0 and 2147483647
for s == 1.

Rewriting the expression as minimum*(1-s) + maximum*s gives the correct
result in this case.

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

build: add subdir-objects to AM_INIT_AUTOMAKE

Fixes warnings with automake 1.14

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

design: fix typo

queue2: Fix backwards seeks into undowloaded ranges

When in download buffering mode queue2 didn't check if a range offset is
in a undownloaded range before the currently in-progress range. Causing
seeks to an earlier offset to, well, take a while.

docs: some small gtk-doc markup fixes

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

gst: register new color mode enum, fixing 'make check'

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

typefindhelper: Avoid using buffer_get_size in tight loops

Calling gst_buffer_get_size represented 2/3 of the cost of helper_find_peek
which was called whenever a typefindfunction wanted to peek at data.

We already know the size (from the GstMapInfo), so just use that.