typefindhelper: Remove obsolete FIXME

It's not necessary (and not a good idea) to cache the typefind factory
list anymore.

registry: private is a C++ keyword, don't use it

...otherwise C++ compilers will complain when including gstregistry.h

docs: add Since tag to docs for new API

And tell gtk-doc that GstRegistryPrivate is private.

typefind: Keep typefind factories sorted in the registry. Fixes #599147

This avoids having to do the sorting everytime we use typefind

The behaviour of gst_type_find_factory_get_list has subtlely changed
in the sense that the order was previously undefined, whereas now
it returns them sorted by rank and then by name.

registry: Cache element and typefind factories. Fixes 598896

This avoids unneeded list/filtering if the registry hasn't changed

gstpluginfeature: API : new gst_plugin_feature_list_copy() method

This allows copying AND incrementing the refcount at the same time,
avoiding a double iteratio of the GList

gstregistry: Add a cookie for detecting feature list changes

We also create a private structure, since we will need to add more
data there in following patches.

message: don't use typechecking cast macros

Simply use casting macros for accessing the message fields like we do for
buffers and events. Avoids some costly typechecking that does not really buy us
much.

Revert "gstmessage: Avoid expensive src/type/timestamp fetch."

This reverts commit 61cf93a334b79a2d8493e531cc44ba45a4209805.

gstmessage: Avoid expensive src/type/timestamp fetch.

If we've already checked that we have a valid message, use the entries
directly.

gstcaps: Use inlined version of _is_any()/_is_empty()

CAPS_IS_ANY and CAPS_IS_EMPTY are the equivalent of their gst_caps_*
counterpart except that they avoid the typechecking and are inlined.

CAPS_IS_EMPTY_SIMPLE only checks for empty caps (without checking if
the caps is ANY).

TODO: add item to TODO list

We currently do a little too much work when we push the first buffer around
resulting in excessive caps checking. We can probably make this a little less
expensive.

gstpipeline: Simplify base time checking slightly

Simplify checking and distribution of the base time - don't re-check
the value of a local variable that was set 3 lines earlier.

gsturi: Optimisation: Avoid type-checking in sorting method.

We already know the list only contains plugin features

gsturi: Optimisation : Cast when we're sure of the type.

Also directly access GstElementFactory->uri_type instead of going
through a function that will (once again) check whether it's a
GstElementFactory

tests/fakesink: Add some debugging

tests/faeksink: Lower the number of threads to avoid timeouts

We just end up with way too much contention in glib otherwise.

gstregistry: Use hash table when finding a feature. Fixes #598895

tee: implement custom acceptcaps function

Implement a custom acceptcaps function on the sinkpad. We can accept any caps as
long as it is accepted by all downstream peer elements.

multiqueue: avoid lock for taking the counter

The counter for incomming data is already protected with the STREAM_LOCK so we
don't need to add another lock around it.

registry: hash the plugin basename

Maintain a hashtable of the plugin basename. We can then use this
hashtable to speedup the search for an existing plugin and avoid
a whole lot of strcmp calls.

registry: speed up _strlen

Make the _strlen function a little tighter

registry: name is never NULL

When looking up a feature by name, we never call this internal
function with NULL so we don't have to check for it.

registry: refactor plugin lookup

We keep lookup plugins by their basename. Avoid creating a basename
from a filename if we can.

registry: do quick check for . files

Do a quick check for . files before calling the strcmp functions

Remove executable bits on xml unit test.

docs: Fix docs for gst_caps_set_simple()

docs: Modify docs string slightly.

Apparently starting the last line of a docs string with 'returns' both
confuses and enrages gtk-doc. Use a slightly different wording instead.

pluginloader: When a plugin is blacklisted, output a GST_ERROR line.

tools/gst-inspect: Check we're not handling NULL pointers.

tools/gst-inspect: Remove dead assignment

trace: Do not poison gst_trace_add_entry()

Since gst_trace_add_entry() is a macro, gcc will barf when it is
defined in case it has been poisoned due to trace support being
disabled.

controller: just cast in internal API where we have checked parameters already

controller: use g_slice for controlled property structures

Use g_slide instead of nomal g_new, Also don't init struct with 0 as we need to
init it anyway with the real values.
Also join the 3 flags checks into one.

libs/base/typefindhelper: Remove useless typechecking in tight loop

The list against which we run the comparefunc will only contain
GstPluginFeature, therefore remove the 6 expensive type checks we do
for every single comparision.

caps: fix typo in docs

win32: Add new API symbol

Automatic update of common submodule

From 85d1530 to 0702fe1

Improve caps setters API

This patch adds gst_caps_set_value() and allows gst_caps_set_simple() to
work on non-simple caps. See the API documentation for the functions
about what they do.

The intention of these changes is to ease working with caps in caps
transform functions. An example for this would be ffmpegcolorspace,
where the caps transform function could be changed to look roughly like
this (pseudocode ahead):
  result = gst_caps_copy (template_caps);
  value = gst_structure_get_value (gst_caps_get_structure (caps, 0),
    "widh");
  gst_caps_set_value (result, value);
  /* same for height, framerate and par */
  return caps;
which is much cleaner and easier to understand than the current code.

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

Add XML_LIBS when building tests that use xml-specific functions

po: Don't create backup .po files

As well as preventing creation of useless backup files, it works
around a bug in gettext 0.17 on OS/X

basesrc: fix race in PLAYING->PAUSED->PLAYING

When we quickly switch from PLAYING to PAUSED and back to PLAYING it's possible
in some cases that the task refuses to start, This is because when we go to
PAUSED, we unschedule the clock timeout, which could return UNSCHEDULED when
we're back to PLAYING, causing the task to PAUSE again with a wrong-state.

This patch checks if we are running when we return with an UNSCHEDULED return
value and if we are, try to create a new buffer.

Fixes #597550

docs: clarify preset api docs

docs: fix controller sections docs

Automatic update of common submodule

From 6380d4b to 85d1530

iterator: Fix a documentation typo

build: ...and add missing endif

build: Also don't run make check-exports if debugging is disabled

build: Only run make check-exports if no public API was disabled

Fixes bug #598297.

gstobject: Replace recursive gst_object_has_ancestor() with an iterative version

This is slightly more efficient because the compiler can't do tail
recursion here and has to keep all stack frames.

Not that efficiency is that important here but I already had
the iterative version somewhere else and both are easy to read.

gstobject: Add simple unit test for gst_object_has_ancestor()

netclientclock: fix timestamp comparission, Fixes #597407

check: Disable the test_fail_abstract_new() test entirely on OS/X

Fixes a compiler warning from the function being compiled but not
used.

debug: Mark the GST_POLL symbol as extern to avoid multiply-defined error

Update common to 6380d4b370f078f0cca7240428ea9f6639571ff5

gstpoll: Make the new GST_POLL debug completely private

Make the GST_POLL debug category symbol private to libgstreamer, as
there should be no external users of it.

checks: Disable a fairly silly gstobject test on OS/X

This test used to SIGBUS on OS/X but now SIGSEGV's instead on
Snow Leopard. It's not worth the effort to figure out which platform
should produce which error for what is fundamentally a pretty silly
test, so just disable it on OS/X

libs/gst/check: Make writing threadsafe. Backported from libcheck trunk

libs/gst/check: Run gst-indent on libcheck.

gstpluginloader: Don't wait forever on gst_poll_wait.

This allows the macosx versions to properly error out when fds are closed.

This is only a temporary fix until the pluginloader is switched to not
use GstPoll but GIOChannels.

gstpoll: Only take into account active fds

This is needed so that select properly errors out on macosx (sigh)

gstpoll: Add some debugging statements

gstpoll: Use the error fdset when using select/pselect.

This is needed to properly detect fds that are closed or that got
an error

gstpoll: Don't use poll on systems with broken poll

gst: Add debugging category GST_POLL for gstpoll

configure.ac: Detect broken poll()

basesink: lets keep -1 for segmenst as they are guint64 and not GstClockTime

basesink: use GST_CLOCK_TIME_NONE and GST_CLOCK_TIME_IS_VALID more

pluginloader: Fix valgrind warnings by zeroing padding bytes.

check: Hopefully fix an 'may be used uninitialized' warning on OS/X

registrychunks: Fix a debug format string harder to satisfy OS/X's gcc.

registrychunks: Fix format string for debug error message.

pluginloader: Move stdin and stdout out of harm's way

In the plugin loader subprocess, move stdin and stdout to new fd's
so that plugins printing things during plugin init or (*gasp*)
possibly reading from stdin don't interfere with the data sent to
and from the parent.

plugin docs: Add GST_PLUGIN_SCANNER env var to the inspect environment

Automatic update of common submodule

From 37f898b to a3e3ce4

gitignores: Ignore the bytewriter check binary

registry: Fix error handling in the registry loader

When the plugin-scanner load fails (because the helper can't be
spawned), make sure to load the plugin that failed in-process, so
that all plugins do get loaded.

check: Fix test run in tests/examples/manual

Add the GST_PLUGIN_SCANNER env var to the check environment here too
so that it doesn't fail when no installed scanner is available.

gstinfo: add back fix that shouldn't have been reverted

queue: more queue optimizations

Split gst_queue_locked_enqueue() into variant for buffer and event to get rid of
the if() and make the code more readable (constant boolean parameters are never
nice). Removes the if (item) checks as we dereference the pointer before anyway.

Also apply the same idea of reusing the previous knowledge in
gst_queue_locked_dequeue to remove more type checks.

multiqueue: split gst_multi_queue_item_new

Split gst_multi_queue_item_new into buffer and event variant to make save an if
and make code more readable.

plugins/gstfilesrc: Make a fast-path for length == 0 buffer creation.

If the requested length is 0, we don't need to read anything from the file.

gst/check/consistencychecker: Check type of miniobject in probe

gst/base/basesink: Remove dead assignment.

The code was previously:
* checking if ret was != OK
* .. but if it was FLOW_STEP, swith it to OK
* .. and then not using ret

Instead we just make it more compact by checking if it's OK or STEP.

gstobject: Remove dead assignment.

object is no longer used after that line

gstindex: Make sure writer is non-NULL.

Fixes the NULL dereference a few lines lower (where it gets the object type).

benchmarks: Clean up gstbufferstress.

benchmarks: Fix the complexity and mass-elements benchmarks

checks: Fix string leaks in the new childproxy test

plugin: Ignore an empty dependency list.

If a plugin registers an empty dependency set, just ignore it rather
than serialising and checking an empty set.

registrychunks: Fix off-by-one error. Improve debug.

Fix an off-by-one error in the size guard for unpack_element, and
improve various debug statements in the failure paths.

Also, swap some g_new0 to g_malloc0 for the fun of it.

filesink: Use _wfopen on win32 to open files with non-ascii filenames correctly.

docs: fix Since: tags in docs for newly-added API

info: revert two of the changes

It only needed for the non constant string.

info: use a "%s" format string when printing the memory dump line

We know that the content is save, but the compiler does not.

parse: don't format the string twice

We were formatting the string once and then passing the string as a format
string to the log functions.

pad: add variants of gst_pad_get_caps() that don't copy caps. Fixes #590941

In most places in core and baseclasses we just need the caps to do caps-
intersections. In that case ref'ed caps are enough (no need to copy).
This patch also switches the code to use the new functions.
API: gst_pad_get_caps_refed(), gst_pad_peer_get_caps_refed()

tests: add ghostpad test for setting target again after pad is linked

tests: remove empty lines from wrong indent run

check: Fix compilation of the bytewriter test