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.

po: update translations

common: revert accidental re-winding of common submodule

query: Clarify the estimated-total documentation

Tweak the documentation slightly to clarify that the estimated-total in
a a Buffering query the total remaining time of a download, not the
total time for the complete download. Also indicate the unit used.

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

queue2: Forward the schedule query upstream

When asked about the scheduling flags first check with upstream and
simply add the _SEEKABLE flag when using a temporary file as storage.

This enables the forwarding of _SEQUENTIAL and _BANDWIDTH_LIMITED from
sources if needed.

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

Back to development

Release 1.1.3

Update .po files

base: Fix handling of SEGMENT query

The values should be in stream-time, and start/stop should not
be swapped for negative rates.

typefind: Only advance offset by the number of bytes we actually read

There might be a short read at EOS.

baseparse: Implement SEGMENT query

buffer: fix Since: marker for new gst_buffer_extract_dup()

clock: debug the clock return values

baseparse: fix seqnum handling for seeks

Use the same seqnum as the seek for flushes/segments that are
caused by the seek. Also do the same for segment events

Fixes #676242

info: parse debug levels > 9

value: Fix copy&paste mistakes in the bitmask function docs

basesink: Don't shadow variables that are set inside our scope and then used outside our scope

Fixes uninitialized use of these variables.

tests: Remove other interface structs from the ABI tests too

tests: Remove GstTagSetter from ABI checks

Interfaces can have new members added without breaking ABI, so
remove it from the check.

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

libcheck: Escape strings in the generated xml files

This is copy pasted from upstream libcheck

basesink: Print some debug output if a stream-start event without group-id arrives

Ideally all elements would implement handling of that to get proper
stream-start message handling and other things.

input-selector: Fix missing pad activation notification

A new active pad might not be notified in some cases, which results
in the current track number not being set in playbin.

The active-pad notification is only sent in the chain and sink_event
functions, and only when the buffer or event that triggered the active
pad selection is from the newly activated pad. So in the other case
the notification will never be sent.

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

value: handle deserialisation of nonexistant enum value more gracefully

inputselector: Don't push new stream-start events on stream change unless they all have group ids

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

typefind: Use new group-id in stream-start event

base: Use new group-id field in stream-start event and message

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.