autogen.sh: older aclocals don't like -I. so use -I . instead

gstbuffer: add additional checking for writability

Check for metadata writability when setting caps on buffer or when copying
metadata flags. Only enable these extra assertions in git versions.
This should help us find bad elements.

check: disable unit test support on win32 for now

Until we make the internal libcheck work on windows.

check: fix symbol exporting

check: fix issues with 'make distcheck'

Seems to work now, at least on *nix. One of the configure checks
caused these weird issues - but which one?

check: use private copy of check for libgstcheck

See #577275. Seems to work fine, but doesn't distcheck yet.

check: add internal copy of check-0.9.6

Not hooked up yet. See #577275.

docs: fix Since: tag for new gst_caps_can_intersect() function

utils: use new _caps_can_intersect()

pad: use new _caps_can_intersect()

basetransform: use new _caps_can_intersect()

caps: add gst_caps_can_intersect()

Often we don't need the result of the intersection. Add a variant that only
tries to intersect. It can break out earlier and does less GValue copying.
API: gst_caps_can_intersect()

basetransform: only check caps_is_fixed() if they changed

The previous code could call gst_caps_is_fixed() for the same caps many times.

caps: split callback for structure intersect into two functions

We call this separately. there is no much benefit in reusing the callback.
Splitting is let us remove a branch also.

logging: log if we copy caps to be able to track it

caps: add comments about g_ptr_array size behaviour

Just explain the behaviour to avoid that someone else is wasting time trying to
improve this too.

example: unref the clock id

pad: use correct variable in test

registry: add filename to debug message, like elsewhere

bin: fix compiler warning about unused var when disabling debug logging

queue: post error message when pausing task

If downstream returns error and upstream has already delivered
everything (including EOS) and will no longer be around to find
out that we paused (and why), post error message.  Fixes #589991.

queue: add unit test

Make a downstream element return an error after upstream has already
put all data into queue (including EOS).  As such, upstream
will not be around to pick up the error, so it is up to queue to
act appropriately.  See #589991.

Note there may be downstream fatal errors (e.g. negotiation) that do
not warrant an error message already having been posted.

basetransform: clarify _caps_is_equal()

basetransform: refactor metadata modifications

Check when we need to touch the metadata of the output buffer after selecting
the output buffer so that we have everything in one place.
Also take flags and timestamp modifications into account.

capsfilter: only set caps when different

When we have an input buffer with caps and when those caps are different from
the caps we want, only then make a writable copy of the input buffer as the
output buffer and set the caps on that output buffer. This avoids some cases
where we took a subbuffer for setting caps that were the same.

basetransform: enable optimisation

When we have the same input as output caps, reuse the input caps object. After
the caps refcounting has been sorted out now, we can finally enable this
optimisation.

tests: don't set caps on unwritable buffers

Take the ref after setting the caps on a buffer because else the buffer is
techinically not writable.

queue: get caps after making writable

Get the caps of the buffer after we made the buffer writable. This did not
cause any problems but it's nicer this way.

capsfilter: fix refcounting problem

Make sure the metadata is writable before setting the caps on a buffer.

basetransform: fix refcounting problem

Add some more debug info.
Make sure that the output buffer has writable metadata before we attempt to set
caps on it.

fixes #583999

caps: add some more debugging in _replace

pad: Add some more debugging

ghostpad: small improvements

Unref the target pad after we used it for debugging.
Add some more debug.
Only replace caps when they changed.

basesink: cleanups in position queries

Use existing boolean flag to pass position queries upstream. Also add upstream
queries for the last position queries.

configure.ac: fix libxml2 check, which is only needed for xml load/save now

Since the registry doesn't use libxml2 any longer, it's no longer necessary
to disable both xml load/save *and* the registry to get rid of the libxml2
dependency, disabling just xml loading/saving is enough. Fixes #590841.

gst-uninstalled: rename uninstalled registry file to registry.dat

We're not using the xml registry any longer after all.

gst-uninstalled: refine search paths for uninstalled plugin modules

Use more refined search paths for our plugin modules. Not only does
this make things much faster in an uninstalled setup, it also makes
sure we're not accidentally using out-of-date plugins built ages
ago as part of a (failed) 'make distcheck' when we forget to clean
up the distcheck build directory.

docs: dist GStreamer-1.0 buffer design draft

taglist: Add new ALBUM_ARTIST tag to the docs

taglist: Add support for ALBUM_ARTIST tag

The "album artist" tag is used when the artist of an entire
album differs from the artist of an individual track; for example,
when a "guest artist" appears on an album, or on compilations.

Fixes bug #590430.

basesink: Query upstream for the position if conversion in PAUSED failed

Fixes bug #590045.

basetransform: Improve debug output in gst_base_transform_acceptcaps()

Fixes bug #589524.

basetransform: Don't unset GAP flag if working in passthrough mode

Fixes bug #589314.

back to development -> 0.10.24.1

Release 0.10.24

Update .po files

bytereader: avoid wrap-around in buffer size checks.  Fixes #590622.

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