gstutils: Add new scaling functions to the docs

gstutils: Add (very) minimal unit test for the new rounding scaling functions

gstutils: API: Add rounding to nearest and next integer versions of the 64 bit integer scaling functions

The new functions are
gst_util_uint64_scale_int_round()
gst_util_uint64_scale_int_ceil()
gst_util_uint64_scale_round()
gst_util_uint64_scale_ceil()

Fixes bug #590919.

gstutils: Revert parts of last change to optimize the scaling functions again

Partially fixes bug #590919.

gstutils: Fix violations of strict-aliasing rules in gst_util_uint64_scale()

gstutils: Refactor gst_util_uint64_scale()

This will later make it possible to provide rounding versions
of it without much code duplication.

Partially fixes bug #590919.

bufferlist: update doc string

gstsegment: Actually start==stop==segment_start is inside the segment

Still the old code was wrong as it claimed that start==stop<segment_start
would be inside the segment and returned insane clipping differences.

gstsegment: Fix unit test and add an additional test

The previous test assumed that start=stop=segment_start will
be inside the segment but this is wrong.

gstsegment: Clipping should detect start=stop<segment_start as outside the segment

Before it returned that [start,stop] is inside the segment and that the
difference between segment_start and start needs to be clipped. If the
clipping is done on a buffer (like in baseaudiosink) this will result
in the data pointer being at a invalid memory position.

Fixes bug #589849.

gstbus: Unref pipeline after usage in test_custom_main_context unit test

This makes the core unit tests valgrind clean again.

docs: add Edward's git plugin moving howto to moving-plugins document

checks: don't forget to include config.h in the GstObject unit test

checks: try to fix GstObject unit test on OSX

Seems like we get SIGBUS instead of SIGSEGV here when GLib crashes
where it shouldn't crash (and we even have a unit test for that!).

checks: set pipelines to NULL state in parse-launch unit test

Fixes timeouts in gst_task_cleanup_all().

checks: set pipeline back to NULL state in GstBus unit test

Fixes timeout in gst_task_cleanup_all().

check: add some logging before calling gst_task_cleanup_all()

check: Call gst_task_cleanup_all() in GST_END_TEST

This fixes many unit tests under valgrind that shows
leaking GstTasks that are not really leaked but just
not unreffed by the task thread before the unit test
stopped.

Fixes bug #591045.

basesink: Remove dead assignments

gst: Remove dead assignments

gitignore: ignore new queue-error test

check: add internal-check.h to BUILT_SOURCES in attempt to fix the build

For some people the build of libgstcheck was broken because the make
target that creates the internal-check.h file wasn't executed for
some reason. This should hopefully fix this.

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.