gst_private: MSVC doesn't seem to like #warning

Visual Studio complains about "invalid preprocessor command 'warning'"
even if the ifdef doesn't trigger, so just remove this again.

tests: Fix multiqueue test for latest commits.

The problem lies in the fact that multiqueue will now operate somewhat
similarly to the flow aggregation logic of demuxers and therefore
will stopp whenever all downstream pads return NOT_LINKED and/or
UNEXPECTED and there's no more buffers to push.

The latest commits should not affect any regular use-case, but the bug
report will be kept open so the previous behaviour can be re-established
if needed.

Fixes #609486

multiqueue: Don't stop threads on UNEXPECTED and forward flow returns.

When a downstream element returns GST_FLOW_UNEXPECTED we want to:
* let the dataqueue task running
* forward the flow return upstream.

This allows upstream elements to push EOS, and have that EOS event come
downstream.

Fixes #609274

Revert "multiqueue: handle UNEXPECTED flowreturn better"

This reverts commit fbdf4dcedad8692f1e3d8838551188987e462e74.

Partly fixes #609274

git-update.sh: Fix issues

Revert "docs: fix ASCII art so that iterators are aligned property to the diagram"

This reverts commit ae60d06e9e401d1ed4de5ef25b5c283db0696a31 (fixes: #609166)

0.10.25.3 pre-release

po: translation updates

docs: fix ASCII art so that iterators are aligned property to the diagram

gstbytewriter: Fix different function names in .h and .c

gst_byte_writer_reset_and_get_buffer wasn't declared
in .h, instead there was _reset_and_get_data_as_buffer.

Replace it with the real function name, that is smaller
and matches gst_byte_writer_free_and_get_buffer

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

docs: add some more Since: markers to buffer list docs

filesrc: fix typo in warning message

Spotted by bsreerenj@gmail.com.

Fixes #608442.

Automatic update of common submodule

From 15d47a6 to 96dc793

init: don't spew warning about late g_thread_init()s if GLib >= 2.23.2

Late g_thread_init() is fine with newer GLib versions and done automatically
from g_type_init() there, so don't warn if the application hasn't called
g_thread_init() yet when gst_init() is called with new GLib versions.

Fixes #608398.

pkgconfig: don't put -DG_THREADS_MANDATORY into our pkg-config CFLAGS

If we force -DG_THREADS_MANDATORY onto apps, then g_thread_supported()
will always evaluate to TRUE, so the typical thread initialisation
boilerplate code if (!g_thread_supported()) g_thread_init(NULL); will
no longer work, and the threading system not be initialised and us
printing a warning in gst_init. This may be fine in most cases, since
late initialisation is allowed and automatically done in g_type_init()
since GLib 2.23.2, but let's be cautious and only use this define when
compiling GStreamer itself.

See #608398.

pipeline: Take start_time after chaining up too

Refactor the code to take the current start_time when going to PAUSED.
Make sure we also call the start_time update code after we chained up to the
parent bin.

Fixes #607842

0.10.25.2 pre-release

checks: fix spurious ghost pad check failure

win32: update windows headers to latest version

docs: minor update to release notes

po: update translation files

tests: fix warning in rtpool-test

The stream status message object may be of a non-GObject type, e.g.
G_TYPE_POINTER (see GstAudioSrc), so print that properly instead
of assuming the value holds an object.

multiqueue: handle UNEXPECTED flowreturn better

When we receive an UNEXPECTED flowreturn from downstream, we must not shutdown
the pushing thread because upstream will at some point push an EOS that we still
need to push further downstream.

To achieve this, convert the UNEXPECTED return value to OK. Add a fixme so that
we implement the right logic to propagate the flowreturn upstream at some point.

Also clean up the unit test a little.

Fixes #608136

docs: Fix basics-bus docs

Fix wrong information about bus watch functions in the
application development manual.

Fixes #608127

bin: also remove private function from def file

bin: make a interface vmethod implementation static

This should not cause any troubles - the methods wasn't in any header.

childproxy: remove ; after }

queue2: add some docs to mark new property

queue2: add option to remove the temp-file

Add an option to automatically remove the temp file (TRUE by default). This
should make it possible for the application to keep the temp file by other means
than hardlinking or holding an fd open.

Fixes #607739

typefind: don't leak uri string

gstindex: retab .h file

tools: Run g_thread_init() unconditionally

Since we define G_THREADS_MANDATORY, g_thread_supported() evaluates to
TRUE unconditionally, so calling g_thread_init() never happened.

pluginloader: fix compiler warning on win32

Move variable that's only used on unix into the unix block so that
the compiler doesn't complain about the unused variable on win32
(see #597662).

pluginloader: try scanner set via env var before using the installed one

If the GST_PLUGIN_SCANNER environment variable is set, we should try
the scanner specified there first, to make sure the right scanner binary
is used for uninstalled setups and builds from source when there's
already an installed version.

build: Define G_THREADS_MANDATORY everywhere

We require threads to be supported in any case and defining this
will simplify the mutex, condition variable, etc. macros from gthread
to not always check if threads are really supported.

Fixes bug #607481.

pluginloader: disable external plugin loader on Windows until it is ported properly

See #597662.

gst_private.h: make sure gst_private.h is included before glib.h

For the reason outlined at the beginning of gst_private.h (inline
functions in glib may need the g_log_domain variable). Also include
gst_private.h before using any G_OS_* defines, esp. in plugin loader.

docs: minor gtk-doc markup fixes

Automatic update of common submodule

From 14cec89 to 15d47a6

docs: small docs updates

gstpad: directly set the caps when pushing buffer with different caps.

This check is not necesarry as we are not negotiating anymore. And it can
be wrong if upstream can't produce this caps anymore, but downstream can
process them fine.

miniobject: The GValue collection function can not assume that the destination is initialized

...and it will usually be either filled by zeroes or random values.

Fixes bug #607283.

basetransform: Only use suggested caps in buffer allocation if a size was suggested too

structure: remove superfluous guard against NULL

All callers of this static function already check for NULL-ness
themselves, so no need to do it again (and if we do it, we should
probably do so before dereferencing the pointer for the first time).

structure: micro-optimise some getters

Avoid checking the GType of the value twice (once on our side and
once in g_value_get_*()) by by-passing g_value_get() and accessing
the GValue structure directly.

message: update docs a little

docs: minor release docs update

basetransform: Handle buffers with NULL caps correctly

This means that the caps didn't change so don't try to handle
the NULL caps as the new caps.

docs: Move field specific Since markers at the same line

Fixes gobject-introspection warnings about Since being defined multiple times.

faq: remove revision history that no one updates or cares about anyway

faq: fix link to gst-uninstalled on cgit

Fix link to gst-uninstalled now that it's been moved, and fix a typo
while we're at it. Also add a new section to 'Building GStreamer from
git' that points to the 'How do I develop against an uninstalled copy
of GStreamer' section.

docs: we're in git since a while

scripts: move gst-uninstalled from docs/faq to scripts

Don't include the long gst-uninstalled script in verbatim in the faq anymore
(there is a link to cgit). Dist the script under its new location.

registry: avoid some more unnecessary malloc/frees

registry: avoid some unnecessary strdup/free when reading the binary registry

Strings in the binary registry are NUL-terminated, so we can just use them
directly if we only need them temporarily, and avoid unnecessary mallocs
and frees.

typefindelement: use new typefind function

Refactor a little.
Use the new typefind helper function that uses the extension to speed up
typefinding.

typefind: add a new method that also uses the file extension

Add a method to perform get_range typefinding that also uses the
uri/location extension as an extra hint. It will first try to call the
typefind functions of the factories that handle the given extension. The result
is that in the common case, we only call one typefind function, which speeds up
the typefinding a lot.

docs: update QoS documeent

Add some ideas about a new QoS message.

See also #322947

tee: make release_pad threadsafe

Protect the ->removed field with the object lock as well. Take the DYN lock
earlier so that we can mark the pad removed and avoid a race in pad_alloc.

Fixes #606435

bus: whitespace fixes

utils: defer getting the classes until we actualy need them

This function has a lot of early returns. Give them soem more benefit.

utils: avoid extra hop in gst_element_link

No need to call gst_element_link_pads_filtered with filter=NULL, which would
call gst_element_link_pads() in that way. Call it directly to save a call and
expensive gobject type checks.

check: remove some cruft from header file

Remove some cruft from the gstcheck header file that's not needed
any longer now that we ship with our own copy of libcheck.

pwg: remove empty midi section

Merge branch 'master' of ssh://git.freedesktop.org/git/gstreamer/gstreamer

Update spec file

po: update for new translated strings

docs: minor documentation fixes for recently-added tags

Mention the type of the tag in the gtk-doc blurb, so people know
which accessor API to use, and fix up the doc blurbs to match the
actual tag define.

tags: fix up translated strings for some new tags

Fix up translated strings for some recently-added tags to match the
existing strings: we want short mnemonic-like strings here that start
with a lower case letter.

registry: deprecate useless gst_registry_xml_{read|write}_cache()

The only reason these two functions are still around is that at some
point in the past they were in a public header, so we can't really
remove them now even though they should have been private all along
(and aren't really particularly useful). Since these are just empty
stubs now that do nothing but return FALSE and will be removed in
0.11 anyway, we may just as well deprecate them formally.

docs: add Since markers to task pool docs and document task function

configure: move SHAVE_INIT behind all checks

Move SHAVE_INIT behind all other checks, in particular AG_GST_CHECK_CHECKS.
This should fix problems with header checking and checking for localtime_r,
which causes compilation errors with clean checkouts where common/shave has
not been created yet when those checks are run. It seems like SHAVE_INIT
changes the environment so that checks depending on a compiler need shave
to exist at that point, which will fail if AC_OUTPUT hasn't created it yet.

Fixes #605930.

check: patch internal check copy some more so that failures actually fail

Include unistd.h so that _POSIX_VERSION is actually defined when
it should be defined. Without that, stuff like fail_if(1) doesn't
actually fail, presumably because other parts of the code do include
unistd.h and then have _POSIX_VERSION defined.

Fixes #604565 even more.

docs: add missing returns: tag

multiqueue: set iterate_interal_links function on source pad

buffer: remove unneeded casts

buffer: remove subbuffer subclass

Move the parent buffer pointer into the GstBuffer struct so that we can
remove the subbuffer class and type. This is interesting because it allows us to
more naturally implement methods to get the real type and parent
of a subbuffer (See #545501).
It should also be slightly faster because there is no extra object hierarchy to
initialize and free.

collectpads: don't keep buffers reffed longer than needed

Make sure we take ownership of the buffer early without increasing its refcount
when we go in the collect function. This reduces the amount of copies needed in
order to make the buffer writable in most cases.

miniobject: avoid unneeded casts

collectpads: avoid doing subbuffers when we can

In some cases we can avoid allocating a subbuffer and instead simply ref
the buffer. Callers should perform _make_metadata_writable() in all
cases now.

collectpads: add ability to install clipping functions

Add a method to install a clipping function that is called when a buffer is
received. Users of collectpads can then perform clipping on the incomming
buffers.

Also retab the header file a little.

See #590265

docs: add some more buffer2 ideas

avoid some more type checks

pipeline: avoid some type checks

Avoid type checks when we can
Don't need to peek the parent_class, the boilerplate does that for us.

launch: also print leaked objects

Make the -T option also print the leaked objects

trace: include type name in leaked objects

When we are dealing with a GObject, print the type name along with
the pointer for easier debugging.

pad: Fix problem with destroy callback not being called

When we unblock a pad with the same user_data, the destroy callback is not
called. This leads to refcounting leaks that cannot be avoided. Instead always
call the destroy notify whenever we install a new pad block.
In particular, this fixes a nasty pad leak in decodebin2.

Also update the unit test to have more accurate comments and test the required
behaviour.

tee: small cleanups, use some G_LIKELY

tee: Don't crash if there is no source pad

Automatic update of common submodule

From 47cb23a to 14cec89

gsttaglist: Adds new tags

Adds the following new tags:
GST_TAG_SHOW_NAME
GST_TAG_SHOW_SORTNAME
GST_TAG_SHOW_EPISODE_NUMBER
GST_TAG_SHOW_SEASON_NUMBER
GST_TAG_LYRICS
GST_TAG_COMPOSER_SORTNAME
GST_TAG_GROUPING

Fixes #599759

configure: always call our check checks for the SUBUNIT conditional

The SUBUNIT conditional needs to be set even if check is disabled. Also
remove a FIXME that is not needed any longer / after all.

check: patch internal check copy so it works with our build system

Fixes #604565.

check: update autotools and docs stuff for new check version

check: update internal libcheck to 0.9.8

filesrc: printf format fixes

docs: link bus and tasks

Add a link from bus section docs to the task docs. Add a paragraph to task docs
to tell about messages and the bus.

docs: add more docs around GstState and GstStateChange

Take reviewed docs from docs/design/part-state to have that more prominent
inside the api docs. Add a few sentences to link things better together.

docs: review and fix spelling

gstelementfactory: set object name earlier if applicable

Setting an object name is nice for proper debug logging. Ideally this would
still happens earlier (.e.g when pads are added to an element, its not yet set).