bin: Use the new group-id field of the stream-start message for stream-start message aggregation

If all stream-start messages had a group id (for backwards compatibility),
we only consider a stream started if all had the same group id.

In 2.0 we should make the group id mandatory.

gst: Add new group-id field to the stream-start event

All streams that have the same group id are supposed to be played
together, i.e. all streams inside a container file should have the
same group id but different stream ids. The group id should change
each time the stream is started, resulting in different group ids
each time a file is played for example.

common: revert accidental change of common submodule

gst: Add some more Since: 1.2

info: Add some Since: 1.2

info: Add debug color mode option

This allows to explicitely set the debug output color
mode to UNIX on every platform, enable it (use platform
default color mode) or enable it.

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

info: Fix black and underline coloring on W32

Fixes #674320

info: Cut down src file names for MinGW too

Fixes #674320

gst-uninstalled: Fix gst-plugins-gl in uninstalled setup

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

adapter: Take account of the skip in gst_adapter_take_buffer_fast()

Include regression test

adapter: Add function to return buffer composed of multiple memories

API: gst_adapter_take_fast()

query: Don't assert if no context is set in the query

benchmarks: ignore new benchmark binary

query: sprinkle some Since 1.2 markers in docs

timeprovider: g-i: allow None as address for gst_net_time_provider_new()

element: Return an empty GstContext if none was set yet

query: Add gst_query_has_context_type()

multiqueue: only block serialized query when it's safe

We must be certain that we don't cause a deadlock when blocking the serialized
queries. One such deadlock can happen when we are buffering and downstream is
blocked in preroll and a serialized query arrives. Downstream will not unblock
(and allow our query to execute) until we complete buffering and buffering will
not complete until we can answer the query..

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

pad: A newly activated pad should be marked as needing reconfiguration

Revert "pad: Don't consider flushing pads as needing reconfiguration"

This reverts commit 948a9d2f2b728f5fb60be45d47a818cebeb60c7d.

This is racy and trying to reconfigure and fail is still better
than not trying to reconfigure at all.

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

basesrc: Leave the loop function faster if we're flushing

Especially don't even try to send stream-start event or try
to negotiate.

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

inputselector: Deactivate and remove pad without the inputselector lock

Otherwise we might get deadlocks caused by lock order inversion:
During the chain function the stream lock is first locked and then the
inputselector lock. During pad release we first locked the inputselector
lock and then deactivating the pad would lock the stream lock.

There's no reason why the inputselector lock should be required while
deactivating and removing the pad, it's only needed before.

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

Back to development

Release 1.1.2

Update .po files

bin: Always forward clock-lost message if we're not a top-level bin

This makes sure that no bin misses the clock-lost messages, independent
of the state, and could return an old, non-working clock from
gst_bin_provide_clock_func().

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

inputselector: Keep previous active sinkpad around until we're done with it

Otherwise we'll send a new segment event downstream for each buffer.

allocator: fix type of gst_memory_alignment to match declaration

Fixes compiler warnings such as
gstallocator.c:61:8: error: conflicting types for 'gst_memory_alignment'
../gst/gstallocator.h:52:18: note: previous declaration of 'gst_memory_alignment' was here

po: update POTFILES.in

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

basesrc: Do not lock a mutex that does not exist

The GST_LIVE_LOCK is on GstBaseSrc, not on its source pad.

baseparse: reset PTS after seek

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

Add few missing allow-none annotation

gst-uninstalled: add new -bad mpegts lib

And remove signalprocessor/video libs from -bad which have gone
away or were merged into -base.

funnel: Re-push all sticky events when buffers come from a different pad

Don't special case segment/caps, just push all sticky events when they are
received on the currently active pad or when the active pad changes.

funnel: Use default pad function for upstream event/queries

The default functions in 1.x already do the right thing

tests: Remove funnel pad_alloc test

check: Change stream_id parameter name to match GtkDoc

check: Added gst_check_setup_events_with_stream_id()

Added a new function gst_check_setup_events_with_stream_id(), since
gst_check_setup_events() does not work with multiple pads.

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

pad: Don't consider flushing pads as needing reconfiguration

Renegotiation and reconfiguration will fail because all queries
and events won't be accepted by the pad if it's flushing. In the
best case this just causes unneeded work and spurious warnings in
the debug logs, in the worst case it causes elements to fail completely.

queue2: only block serialized query when it's safe

We must be certain that we don't cause a deadlock when blocking the serialized
queries. One such deadlock can happen when we are buffering and downstream is
blocked in preroll and a serialized query arrives. Downstream will not unblock
(and allow our query to execute) until we complete buffering and buffering will
not complete until we can answer the query..

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

pad: Add a filter to the caps_query done by acceptcaps

Use the caps that the pad is asked to accept as filter for the query

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

basetransform: optimize default acceptcaps implementation

Pass the fixed caps we're asked to accept as a filter for the caps
query, so we don't get a fully-expanded set of caps back (which we don't
need and can take a lot of time for intersection).

This reduces the time for camerabin to produce a second frame on a
logitech C910 camera from around 52 seconds to a bit less then 16
seconds on my system.

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

taglist: Avoid combinatorial explosion when merging tags

When appending/prepending tags, avoid re-creating (and copying) lists if we already
have one and instead just append/prepend the GValue to the list.

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

queue: Don't hold the queue mutex while doing serialized queries downstream

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

buffer: Add unit test for map_range()

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

buffer: Fix wrong size/index handling when merging memory

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

docs: add missing file for doc-link check

tests: add stress test for buffers and pools

basesink: call state change in all cases

When we asynchronously go from READY to PLAYING, also call the
state change function so that subclasses can update their state for PLAYING.
Because the PREROLL lock is not recursive, we can't make this without
races and we must assume for now that the subclass can handle concurrent calls
to PAUSED->PLAYING and PLAYING->PAUSED. We can make this assumption because not
many elements actually do something in those state changes and the ones that
did would be broken even more without this change.

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

docs: fix some external links

docs: check for broken links in docs

The check is done using curl (if available). It lists the curl exit code + http
status code (for those > 399) together with the use of the url in the code. The
check is not fatal.

docs: change https to http urls

Thank you browser for needlessly changing to https for static doc pages.

docs: update links to developer.gnome.org

The URL layout has changed. Fix the links and comment out one paragraph where
the doc is gone.
Fixes #702135

structure: Add gst_structure_new_from_string()

Convenience API for bindings, gst_structure_from_string() returns
a tuple (structure, end_ptr) in bindings and is unintuitive to use
because of that.

gst: Don't intercept --help in gst_init()

Before this patch gst_init would intercept --help, causing for example
cheese's --help to look like this:

[hans@shalem cheese]$ cheese --help
Usage:
  cheese [OPTION...] - GStreamer initialization

Help Options:
  -h, --help                        Show help options
  --help-all                        Show all help options
  --help-gst                        Show GStreamer Options

gst_init is the only gfoo_init function which does this.

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

gst-uninstalled: add uridownloader lib in -bad to search paths

Even if it might not be around for long.

gst-launch: Remove unref that should not be there

We keep a reference to the context around all the time.

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

gst-launch: Improve GstContext handling

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

win32: Don't include gstcollectpads.c twice

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

input-selector: send notify::active signal for input-selector pads.

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

basesrc: Only force-update the duration for dynamic sources when doing the DURATION query

Doing it after every single create() is not very efficient and not necessary.
Especially on network file systems fstat() is not cached and causes network
traffic, making the source possibly unusable slow.

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

Back to development

Release 1.1.1

Update .po files

Automatic update of common submodule

From 098c0d7 to 01a7a46

gstvalue: Add _append_and_take_value() public variants

API: gst_value_array_append_and_take_value
API: gst_value_list_append_and_take_value

We were already using this internally, this makes it public for code
which frequently appends values which are expensive to copy (like
structures, arrays, caps, ...).

Avoids copies of the values for users. The passed GValue will also
be 0-memset'ed for re-use.

New users can replace this kind of code:
gst_value_*_append_value(mycontainer, &myvalue);
g_value_unset(&myvalue);

by:

gst_value_*_append_and_take_value(mycontainer, &myvalue);

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

gstbuffer: Use internal function for buffer_new_wrapped

Shaves ~10% instruction calls from the total cost

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

input-selector: return FALSE for "active" property if selector is NULL

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

manual: update elements to match the rest of "Boost priority of a thread" section

manual: fix comment in effectswitch example

manual: fix a typo in "Inserting data with appsrc" section

pwg: fix a few typos

docs: remove double "the"

scripts: improve git-update.sh status message

By default when the script is about to exit (normally or due to an error),
it checks whether $ERROR_LOG file exists.  If the log file exists, the
script prints a "Failures: " message prefix and dumps the log file to the
output.

Apparently the log file is always created and if the update/build is
successful, the script finishes with a bit misleading "Failures: " message.

An improvement provided with this change lets the log file to be created as
needed, i.e. if there's an error message to be printed.  If the file
doesn't exists, the script prints a "Update done" message which clearly
indicates success.

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

check: fix position unit test

basesink: improve position reporting without clock

When no base time or when sync is disabled, use the same logic as
in paused to report position. The logic in PLAYING assumes we use the
clock.

pad: Fix memory leak in the unit test

elementfactory: Add support for checking subtitle/metadata factory types

elementfactory: Add support for checking only the media type of a factory

And while at it also add Metadata and Subtitle media types.

(multi)queue: Don't access query items during flushing

multiqueue: Don't do serialized queries when we're flushing

Just immediately fail the query, otherwise we would wait forever
for the query to be answered.

queue2: First set query result, then signal GCond

queue: Fix handling of serialized queries

During FLUSH_START the query needs to be unblocked already, otherwise
it can lead to deadlocks if the FLUSH_START is the result of something
done from the streaming thread of the srcpad (the queue will never be
emptied!).

queue2: Unblock any waiting serialize queries on FLUSH_START

Fixes some deadlocks during flushing.

And store queue items differently to not accidentially read
already unreffed queries when flushing. Queries are owned by
upstream and not us.

queue/queue2/multiqueue: When flushing, make sure to not lose any sticky events

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

pad: Store sticky events even if the pad is flushing

But do this only for events that are not dropped by flushing,
i.e. do it only for everything except SEGMENT and EOS.

Without this we might drop a CAPS event if flushing happens
at an unfortunate time and nobody is resending the CAPS event.

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

valve: Don't read sticky flag from unrefed event

tee: fix property description for now-unused "alloc-pad" property

Should probably proxy ALLOCATION queries on that though, if set.
But what else? CAPS and ACCEPT_CAPS too?

basetransform: remove 0.10-ism from docs

gst_buffer_pad_alloc() never existed, and gst_pad_alloc_buffer()
doesn't exist any more either, so don't mention it in the docs.

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

queue2: Add support for serialized queries if using a memory queue

queue: Set the last serialized query result to FALSE when flushing

multiqueue: Initialize all GstMultiQueueItem fields in both code paths

multiqueue: Don't access the query after signalling the waiting thread

It might've free'd the query already.

multiqueue: Make sure to always signal any possible pending serialized queries

And don't unref them when flushing the queue, they're owned by the caller!

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

basetransform: Return GST_FLOW_ERROR if the allocator did not allow to allocate a buffer

docs: Remove mention of gconf* elements

Instead recommend pulsesrc/sink for audio, there is nothing GNOME
specific for video.

typefind: Handle the force-caps property more similar to all typefinding code flow

This makes sure that events happen in order and simplifies the code a bit.

check: Fix event handling in gst_check_element_push_buffer_list()

Automatic update of common submodule

From 5edcd85 to 098c0d7

pad: Only check if we get buffers before stream-start/segment if compiling without G_DISABLE_ASSERT

In releases this is set usually.