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.

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.