0.10.24.5 pre-release

collectpads: Get the flushing state with the object lock taken.

Fixes #590056

collectpads: Make sure the CollectData list is up-to-date when reading/setting it

Without this, we risked:
* Checking the flushing state on an unexisting list
* Not setting the flushing state on pads that had just been added

Partially fixes #590056

collectpads: Split out _check_pads into a version without lock taking.

This is so we can use _check_pads in places where we've already taken
the lock in question.

Partially fixes #590056

check: make new GstStreamConsistency structure private

There's no need to have GstStreamConsistency in a public header for
the time being, so make it private. While we're at it, add a gtk-doc
blurb for it though. Re-fixes #588744.

0.10.23.4 pre-release

basesrc: don't handle SEEKING queries for formats that don't match the one the source operates in

Return FALSE in basesrc's default query handler when we get a SEEKING query for
a format that's not the one the source operates in. Previously (ie. before, in
the git version) we would return TRUE in that case and seekable=FALSE, which
is more correct, but causes backwards compatibility problems. (Before that
we would change the format of the query when answering, which was completely
broken since callers don't expect that or check for it). Since the SEEKING
query is a fairly recent addition, not all demuxers, parsers and decoders
implement it yet, in which case any SEEKING query by an application will
just be passed upstream where it will then be handled by basesrc. Now, if
e.g. totem does a SEEKING query for TIME format and we have a demuxer that
doesn't implement the query, basesrc would answer it with seekable=FALSE in
most cases, and totem can only take that as authoritative answer, not knowing
that the demuxer doesn't implement the SEEKING query. To avoid this, we make
basesrc return FALSE to SEEKING queries in unhandled formats. That way
applications like totem can fall back on assuming seekability depending on
whether a duration is available, or somesuch. Downstream elements doing
such queries are likely to equate an unhandled query with a non-seekable
response as well, so this should be an acceptable fix for the time being.

See #584838, #588944, #589423 and #589424.

Automatic update of common submodule

From fedaaee to 94f95e3

gstregistrybinary: add +1 after error checking

The current code made the error checking pointless by changing -1 to 0 in error
cases. Also don't leak a pad template on error.

0.10.23.3 pre-release

tests: make sure the tasks are joined

Call _clean_all() on the task to make sure everything is joined and stopped.

See #589127

task: fix taskpool leak

GstTaks does not always unref the taskpool it was created from because it
depends on when the pool provided an ID for joining the task.
Rework some code so that we always unref the pool and optionally join when the
pool provided an id.

Fixes #589127

basesrc: make tag queuing threadsafe

See #588745

gstcheck: Add a stream consistency checking helper routine. Fixes #588744

binaryregistry: don't unref NULL if we have an early read error

basesrc: Serialize tags into the dataflow. Fixes #588745

docs: fix API docs for gst_{adapter|byte_reader}_masked_scan_uint32

Clarify byte reader docs a bit: offset is relative to the current
position of the reader, not to the start of the data. Also, the
examples in both the adapter docs and the byte reader docs have
the mask and pattern arguments swapped (see #587561). Spotted
by Carl-Anton Ingmarsson.

tags: only emit a g_warning() for empty tag strings for git versions

For now, don't show a g_warning() for empty tag strings and NULL
tags with non-git versions; we should wait for the fixes in our
plugin modules to make it into a release before we enable this
unconditionally.

0.10.23.2 pre-release

value: add explanation for shortcut

basetransform: take size once

value: fix can_intersect to behave like intersect

Add a quick return if two types are the same. Change the check for the
intersection function to be the same as the one used in intersect(). The
later tries both directions.

gstinfo: maintain ABI compatibility even if debugging is disabled

structure: Change NULL and empty string handling

Don't forbid the empty string "" in generic structures, only in taglists.
Properly allow the NULL string by adding special cases for serialising
and deserialising it. prop1=(string)NULL is the NULL string,
prop1=(string)"NULL" is the actual string with the value "NULL"

Automatic update of common submodule

From 5845b63 to fedaaee

filesink: Fix segfault with MSVC

Don't use deprecated fileno on MSVC but replace with _fileno

Fixes #587052

docs/design: Update Makefile.am for changed framestep document name.

tools: the plugin features listed by gst-inspect are typefinders, not types

docs: add draft for arbitrary buffer metadata idea

docs: more framestep docs out of draft

docs: update framestep document

Remove experimental status from the framestep draft.

tools: Fix compilation if option parsing is disabled

Fixes bug #587976.

registry: Use g_build_filename() instead of g_strjoin() with /

This makes sure that the generated filenames use the platform
specific directory separator instead of /.

Fixes bug #587973.

docs: add 'Since' tag for new GST_DEBUG_CATEGORY_GET macro

collectpads: make it the best of wims and edwards patch.

Check the right flushing flag, but still add it to the pad-list.

info: allow getting other log categories. Fixes #587417

Add a new macro GST_DEBUG_CATEGORY_GET to get a log category by name. This
allows plugins to use e.g. core categories like PERFORMANCE or CLOCK.
API: GST_DEBUG_CATEGORY_GET

basetransform: make comment a FIXME comment

logging: log object type in message

logging: use perf category for dropped buffers

collectpads: Don't forward FLUSH_STOP if some input streams are still flushing.

This guarantees that only one FLUSH_STOP event (the last one) will be sent
downstream when a flushing seek is being done through collectpads.

collectpads: Update the cookie when setting ourselves as flushing.

This forces the pad status to be re-evaluated on the next _check_pads().

docs: fix gtk-doc /*< private >*/ marker

typefindelement: log probability in debug message

message: fix parsing of the step done message

Parse the duration field too.

binaryregistry: Use local values in while/for loops, use branch prediction macros

Spread branch prediction macros.

These are based on profiling several playback scenarios using playbin2.

Use local variables in for/while loops.

This makes the generated code faster since:
* It won't have to read an undirect value (which will most likely be
 outside of the L1/L2 cache)
* We know that value never changes (the compiler has no clue that it doesn't).

libs/controller: Set default gst debugging category.

tests: fix example

bufferlist: use faster gst_buffer_list_get()

Use the faster gst_buffer_list_get() to get the first buffer of a list.

bufferlist: fix example

The _do function now takes user_data in all cases.

basesink: take timestamp later

Make sure we don't accidentally cast a bufferlist of a buffer and try to take
the timestamp of it.

Refixes #585960

docs: fix some typos

logging: add a performace log category

This category can be used to log slow code path and help auditing the
performance. Add FIXME-0.11 to some questionable categories.

structure: fix int->gint to be in sync with the *.h  and usage

autogen.sh: Use printf instead of 'echo -n'. Check for automake-1.1[01]

Check for more automake command variants. Use printf instead of 'echo -n'
for portability

Automatic update of common submodule

From f810030 to 5845b63

request-pad: tell about ref counts in release_request_pad docs.

It is not too obvious that getting and releasing request pads is not entierly
symetrical regarding to the pad refcount. Add a note about that to the docs.
This might deserve a FIXME-0.11 too.

basesink: don't do things with side effects within a g_assert()

Make the bufferlist stuff work properly when things are compiled
with -DG_DISABLE_ASSERT.

caps: avoid doing logic in g_assert

Make sure we still do the right thing when glib is compiled without
assertions.

multiqueue: Fire the overrun signal on EOS

Fixes startup of some short MPEG files with decodebin2/playbin2
where all the data fits in the multiqueue and EOS arrives before
the group is exposed.

Automatic update of common submodule

From f3bb51b to f810030

GstStructure: Use direct values for repetitive conditionals (for/while).

miniobjects: Don't chain up to empty finalize method.

If ever we do anything in mini_object_finalize, we should make sure the 4
core miniobject finalize methods chain back up again.

gstcaps: Use direct values for repetitive conditionals (for/while).

make check: add check for enum type class unrefs in gst_deinit() too

Just because we can really.

trace: use proper locking in GstTrace

Protect the allocated list of objects with a lock so that trace actually works
reliably.
Shortcut the alloc trace sooner when disabled.

object: also add pointers to debug

Add the object pointers in the debug info for _replace.

capsfilter: Add GAP flag support

capsfilter doesn't actually touch the data so we don't want the GAP flag to
be unset by basetransform.

Fixes bug #586566.

defs: add new byte reader methods

bytereader: add a bunch of utility functions for strings and a data dup function

API: gst_byte_reader_dup_data
API: gst_byte_reader_dup_string
API: gst_byte_reader_dup_string_utf8
API: gst_byte_reader_dup_string_utf16
API: gst_byte_reader_dup_string_utf32
API: gst_byte_reader_skip_string
API: gst_byte_reader_skip_string_utf8
API: gst_byte_reader_skip_string_utf16
API: gst_byte_reader_skip_string_utf32
API: gst_byte_reader_peek_string
API: gst_byte_reader_peek_string_utf8
API: gst_byte_reader_get_string
API: gst_byte_reader_get_string_utf8

And some basic unit tests. Fixes #586568.

taglist: fix typo in tag description

tests: fix crash and leak in bufferlists unit test

Don't access already-freed iterator, makes check-valgrind work and fixes
crash on PPC; unref buffer we're going to steal to make valgrind happy.

init: Fix indent, and ref the gst_buffer_list_item_get_type() class

Fix the check tests by reffing the GstBufferList class. Run gst-indent
to make git happy about some existing stuff

gst-inspect: fix broken flags to flag string serialisation

e.g. cdparnoiasrc would show fragment|full for a flags value of 2.

tee: add buffer-list support

bufferlist: remove old enum from docs

gstinfo: define __gst_debug_min to LOG_LEVEL_NONE if debugging is disabled

Just in case someone who clearly can't be deterred by any number of leading
underscores uses this very private but still somewhat documented symbol
directly in their code (*cough* qtdemux *cough*).

bufferlist: Various cleanups

Add new method to iterate a bufferlist without having to allocate an iterator.
Add convenience method for getting an item from the list based on the group and
index.
Remove redundant _do_data callback and method.
Update unit-tests and add some more for the new methods.

docs: make gtk-doc happy

po: update .po files after string changes

fdsink: clean up some more error and debug messages

taskpool: fix unused variable warning in case debugging is disabled

gstinfo: fix export of GST_CAT_BUFFER_LIST when --gst-disable-debug is used

Move all the categories to export to one single place, so we don't
accidentally update or add vars in one place but not the other.

collectpads: use the right flushing flag.

We need to use the pad private flag because the other pad flag is protected with
the pad lock instead.

collectpads: Properly handle flushing pads.

If a pad is flushing, it should not be considered as either eos or
containing data.

fdsink: fix error message

Users should never see the term 'file descriptor', much less a file
descriptor number, in an error message. Put that into the debug
string instead and use the default error message.

fdsink: add the new field in the header

fdsink: make fdsink seekable

Implement the same logic as filesink to implement seeking.

Fixes #578908

gstelement: moved the clock unref to the right place

gstelement: unref the clock when the element changes to null state

Replaced deprecated win32-compatibility function with undeprecated one.

Fixes #560442.

gstbin: swap the lines of my previous commit

Fixes a bug introduced in my previous commit that released the
clock provider and after used it to create the clock lost message.

gstbin: remove clock references when clock lost happens

Remove reference to clock and clock provider stored in the bin
when the clockprovider element is removed from the bin.

basesink: add Since tag for new method

basesink: add support for buffer list

Fixes #585960

ghostpad: Add support for GstBufferLists

Fixes #585834

iterator: Explicitly mention refcounting in docs

Fixes #585938

gstxml: fix (de)serialisation of properties of type GstStructure

souphttpsrc has a property of type GstStructure, which causes an
assertion when serialising it to xml. Fixes #585137.

queue: fix compiler warning

The compiler suggests to add some () to indicate if the && or the || takes
priority, so reflow code a bit so we don't have to add yet another layer
of (). Hopefully this was the intended meaning of the code.