Make code safe for -Wredundant-decls

Adds that warning to configure.ac

Includes a tiny change of the GST_BOILERPLATE_FULL() macro:
The get_type() function is no longer declared before being defined.

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

Make sure generated code doesn't run with -Werror

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

Update to common/ changes to ERROR_CFLAGS

Revert "registry: remove unused function"

Turns out  the function is not unused, but was in an #ifdef WIN32
section.
Whoops.

This reverts commit 57d5db424c68ab5a61f33ce36ce0179eb30251ac.

Automatic update of common submodule

From 7cc5eb4 to 0b6e072

registry: remove unused function

Actually, there was two functions with the same name, but only one was
used.

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

remove unused gst_element_default_error()

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

utils: Use mulq instead of mul as some assemblers can't guess the size of the operands

Fixes bug #612370.

Automatic update of common submodule

From 7aa65b5 to 7cc5eb4

Automatic update of common submodule

From 44ecce7 to 7aa65b5

build: Make some more rules silent if requested

configure: Use automake 1.11 silent rules instead of shave if available

This makes sure that we use something that is still maintained and
also brings back libtool 1.5 support.

lfocontrolsource: Optimize get_value_array()

Don't convert from GValue to the actual type for every single
value.

interpolationcontrolsource: Optimize get_value_array()

This makes it >10x faster if more than a single value is requested
by not searching in the GSequence for every value and converting
the value from GValue to the real value type.

controller: Add benchmark for getting a value array of the control points

Fix typos in documentation

caps: Fail when fractions are followed by random text

Previous code treated "1/1yourmom" the same as "1/1" and "1wimsmom" the
same as "1". Now the code is stricter and will fail to convert a
fraction when followed by garbage text.

Back to development

Release 0.10.28

Update .po files

caps: Allow 1/max as the minimal fraction value > 0

This is useful for formats that require a valid framerate (like
theoraenc).

element: fix typo in comments

Release 0.10.27

Update .po files

gstpoll: don't pass non-objects as first argument to GST_DEBUG_OBJECT()

This may cause crashes when logging is enabled, especially on windows.
It's not safe to pass random pointers to g_type_check_instance_is_a().

Fixes #611719.

0.10.26.4 pre-release

po: update translations

docs: fix up bytewriter doc chunks for float functions as well

bytewriter: fix headers for float/double writing functions

The functions are called gst_byte_writer_put_{float32|float64}_*() and not
gst_byte_writer_put_{float|double}_*().

Spotted by: Benjamin Otte <otte@redhat.com>

tags: try to make comment for translators more helpful

basesink: fix emergency rendering timestamp tracking

Specifically, if all (including initial) buffers turn up late,
emergency rendering should also kick in appropriately.

Fixes #611087.

0.10.26.3 pre-release

po: update translations

interpolationcontrolsource: Don't pass NULL pointers to GSequence API

This causes assertion failures. Fixes bug #610444.

interpolationcontrolsource: Add const qualifiers to values in the _set functions

The values are not modified and are copied, a const before the parameter
should make this even more obvious.

controller: Add some FIXME 0.11 comments

corelements: Combine redundant code

Fix compilation of fdsink and fdsrc with MSVC

0.10.26.2 pre-release

.gitignore: ignore some more temporary docs cruft

build: fix indenting in win32-update target

No idea why we need to run gst-indent twice on that file, but it
only seems to settle on a final format with minimal diff to the
one in git after two runs.

gstcheck: more debug logging for gst_check_element_push_buffer_list()

collectpads: Improve docs about 'data' attribute

Adds a reminder to 'data' attribute doc

Fixes #610366

memindex: avoid busy loop when doing EXACT lookup

Fixes #610367.

introspection: add annotation for gst_element_get_state

state and pending are "out" arguments.

Fixes #605189.

filesrc: Don't use expensive cast checks in _create

_create() is a pad function set by ourselves, therefore we're sure basesrc
is a GstFileSrc.

Speeds up _create() by 17% and the total call by 8% (instruction calls measurements
done with valgrind).

Fixes #610246

basesrc: Don't use expensive cast checks in get_range.

_get_range() is a pad function set by ourselves, therefore we're certain that
the parent is a GstBaseSrc.

Speeds up _get_range by 38%, and the total call by 30%. (valgrind instruction
calls measurements).

Fixes #610246

fdsrc: cleanup parameter initialisation and add comemnt+logging

Initialize new_fd with DEFAULT_FD and fd with -1. Setting the property will set
new_fd and in _update_fd() we cehck fd against -1. Also add a coment about the
warning we get in the log from gst_poll_remove_fd(). We could get rid of the
warning if we want by tracking if fd has been added to fdset.

design: write about the current state of tag-handling

Document the taghandling in gstreamer. List gaps and propose new mechanisms to
deal with them.

taglist: remove blank lines in variable declarations

tools: call g_set_prgname() before doing the option parsing

g_setprgname is implicitly called by g_option_context_new() with a check
to see if it's been set already, so set it before g_option_context_new()

Move version printing back until after the options have been parsed,
otherwise it won't work, since it evaluates a flag set by the
option parser.

Revert "tools: Move gst_tools_print_version call to avoid warning from new GLib."

This reverts commit 93dd95f02ef3fa530f54ce81e8ffba96f3b679cb.

This commit made --version not work any longer. The g_setprgname()
warning is fixed in recent GLib versions.

build: make sure gst-plugin-scanner gets installed where we expect it

Add check to make sure gst-plugin-scanner really gets installed where
we will look for it later, ie. paths and prefixes are set at configure
time and not specified via make.

Fixes #609941.

docs: prefer short desc from GstElementDetails

docs: fix gtk-doc chunk for gst_check_element_push_buffer_list()

po: update po files for new comments

tags: wrap long string constants

And fix indenting issue

tags: add some comments for translators so tag mnemonics get translated correctly

We want 'preview image' translated as a noun, not as 'preview [the] image'.

pad: don't print WARN debug statements for normal things like EOS

Automatic update of common submodule

From 96dc793 to 44ecce7

typefind: Reset the working mode when going to READY/NULL

This allows properly re-using typefind (else it would think it's
already done the typefinding when being re-used with another
stream).

bytewriter: Adds a test for _fill

bytewriter: add _fill function

Adds a new function to GstByteWriter that writes
a constant value to a memory area (aka memset).
Useful for adding padding to buffers.

Also updates .def file and docs.

API: gst_byte_writer_fill()

typefind: Avoid messing pads activation

Typefind might mess up pads modes (pull/push) if a
downstream element is plugged and its pads activated
in 'step 2' of typefind pads activation.

This happens because the following steps don't check
if we already emitted typefound due to upstream setting
caps on buffers being pulled in the typefind helpers.

Avoid that by checking if typefound is already emmited.

Fixes #608036

basesrc: Make locking of the segment a bit more strict and update documentation

Updating the segment values must only be done while holding the
STREAM_LOCK and OBJECT_LOCK. This means, reading can be done as
long as one of them is held, not both, which removes some lock-unlock
blocks from performance critical code paths.

Also document, that gst_base_src_set_format() *must* be called in
states <= READY and add an assertion for this. Changing the format
later will completely mess up the segment information.

pwg: several typo fixes

Fixes #609286.

basesrc: Protect segment values from concurrent access from different threads

This could happen easily in the query functions or when the size is set
on appsrc from some non-streaming thread.

typefindelement: Protect internal fields from concurrent changes from different threads

Fixes bug #608877.

gst-launch: don't leak timeout GSource

docs: flesh out release doc some more

Update MAINTAINERS, add myself

configure: back to development

Slushy freeze remains in effect.

Release 0.10.26

Update .po files

configure: define GST_PLUGIN_SCANNER_INSTALLED in win32 config.h

Even if it's not used, it still needs to be defined for things to
compile.

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