queue2: fix possible data corruption in ring buffer mode when seeking

Fix race that could cause data corruption when seeking in ring buffer
mode.

In perform_seek_to_offset(), called from the demuxer's pull_range
request, we drop the lock, tell upstream (usually a http source)
to seek to a different offset, then re-acquire the lock before we
do things to the ranges. However, between us sending the seek event
and re-acquiring the lock, the source thread might already have pushed
some data and moved along the range's writing_pos beyond the seek
offset. In that case we don't want to set the writing position back
to the requested seek position, as it would cause data to be written
to the wrong offset in the file or ring buffer.

Reproducible doing seek-emulated fast-forward/backward on 006653.

Conflicts:
plugins/elements/gstqueue2.c

tests: fix buffer leak in queue2 unit test

check: remove glib deprecation compatibility trickery

check: port to the new GLib thread API

tests: port to new GLib thread API

tests: benchmarks: align error message with code

pad: expose gst_pad_mode_get_name() and use it in baseparse

scripts: update for gst-ffmpeg -> gst-libav

Now that we have a gst-libav git repository (symlinked to gst-ffmpeg).

query: adjust test logic for scheduling mode with flagS

query: add convenience API to query for scheduling mode and flags

events: remove STREAM_CONFIG

We won't be able to implement this so it's better to move it out of the way.

collectpads: clean up header indentation

utils: allow NULL stream_id also when 0 srcpads

We usually first create the stream_id for the stream_start event and then add
the pad to the element. This means that this functions should work when there
are no pads on the element yet.

baseparse, typefind: only activate in pull mode if upstream is seekable

Upstream might support pull mode, but only sequential pulls,
which isn't gonna do much for us.

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

porting-to-0.11.txt: some minor fixes

sample: free info structure with sample if there is one and fix copy with NULL info structure

memory: add padding to GstMapInfo

libs: adjust comment style

gst: remove some defunct commented code

docs: improve porting doc

tests: disable deprecation warnings

define GLIB_DISABLE_DEPRECATION_WARNINGS earlier so that it is defined before
the glib headers are loaded or else we trip over the GValueArray deprecations in
gst-inspect.c.

controller: fix direct control binding double -> int conversion

Round properly to nearest integer. Fixes controller
unit test on PowerPC G4.

examples: fix bus/fd leak in hello world example

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

gst-element-check.m4: fix action-if-found and not-found invocation

Arguments got shifted back by one.

collectpads: handle GAP event

basesink: wait_eos -> wait_event

Fix a FIXME. Now we can also pass the GAP event to the subclass.

examples: update Makefile.am android bits in controller example

Should fix build failure reported on IRC.

pad: check sticky events also after pad block

Recheck for sticky events after doing a pad block because the pad block could
have caused a relink and then we need to resend the events to the newly linked
pad.

Fixes things like switching of visualisations.

baseparse: update for gst_message_new_duration -> _duration_changed()

message: rename GST_MESSAGE_DURATION -> GST_MESSAGE_DURATION_CHANGED

The duration should be re-queried via a query using the
normal path, we don't want applications to use the value
from the message itself, since it might no match what a
duration query done from the sink upstream might yield.

Also disables duration caching in GstBin. It should be
added back again at some point.

configure: add reminder to remove GST_UNSTABLE_API stuff before 1.0.0

gst-element-check.m4: rename AM_GST_ELEMENT_CHECK to GST_ELEMENT_CHECK

And allow passing of a minimum version (if not needed, pass 1.0).

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

tests: add check for gst-inspect --exists functionality

tools: add --exists and --atleast-version option to gst-inspect

For checking if an element exists with a given minimum version.
Will use that in our new GST_ELEMENT_CHECK m4 macro.

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

pluginfeature: disable version mangling for post-1.0.0 release

Just in case we don't grep for FIXME 1.0 before the release.

basesink: Make GAP events actually trigger preroll

Slightly hacky approach needing refinement

gstpad: make some debug statements more verbose

ghostpad: Make some debugging more verbose

Also, remove an unnecessary #include in input-selector

GstSegment: Fix doc description string last_stop->position

inputselector: fix clock leak

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

tools: output gst-inspect errors to stderr

value: fix crash serialising a 0 flags value when there's no name for it

Fixes segfault when doing gst-launch-1.0 -v -m camerabin
(encodebin notifies a 0 value for its "flag" property).

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

gst: log performance warning debug message if glib emulates atomic ops

gst: use configure-detected or externally provided glib-mkenums

To ease cross-compilation.

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

Automatic update of common submodule

From 668acee to 4f962f7

configure: bump gtk-doc req to 1.12 (mar-2009)

This allows us to e.g. unconditionally use gtkdoc-rebase.

memory: add _make_writable

docs: mention some media type changes in porting-to-0.11.txt doc

docs: minor update to porting doc for child proxy lookup method

And a typo fix.

allocator: make a copy with the same alignment

When making a copy of the memory allocated from the default memory allocator,
make sure the new copy has the same alignment as the original memory.

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

baseparse: make seeking in DEFAULT format work if the subclass can convert for us

We only deal in TIME format ourselves, but if the subclass can handle
converting other formats into TIME format, we can support that too.

Fixes seeking in DEFAULT (sample) format with flacparse,
and the flacdec unit test.

tools: minor fixes to gst-launch man page

preset: implement child_proxy support

Elements such as the GstIirEqualizerNBands would so far not store the properties
of their children. Now we also grab the properties of child elements and try to
restore them.

inputselector: Wait for other streams to advance on unselected pads

Otherwise we end up dropping a lot of data in the case where data starts
arriving on the non-selected pad, resulting in big gaps in stream switching

inputselector: More debug statements

inputselector: Don't forward stream-start sticky events

Only one STREAM_START event should be let through, else it will
confuse downstream elements that think a new stream is starting
whereas in fact we are just switching to a different input.

In the future we might want to let them through but with the same
sequence number.

docs: Add new basesrc/basetransform API to the docs

basetransform: getters for pool and allocator

Sometimes a transform filter would need the buffer pool or the memory
allocator negotiated by the base class, for example, for querying different
parameters, such as a bigger number of buffers to allocate by the buffer pool.

This patch expose a two getters accessors: one for the buffer pool and the
other for the memory allocator.

basesrc: getters for pool and allocator

Sometimes the sources would use the buffer pool or the memory allocator for
something else than just allocating output buffers; for example, querying for
different parameters, such as a bigger number of buffers to allocate by the
pool.

This patch expose a two getters accessors: one for the buffer pool and the
other for the memory allocator.

registry: remove some unused and in their current form pointless API

Not so useful: just adds/reads stuff from an internal GList without
actually doing anything with those paths, so remove for now:

 gst_registry_add_path
 gst_registry_get_path_list

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

parse: fix up for gst_child_proxy_lookup() only working on child proxy interfaces

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

childproxy: make gst_child_proxy_lookup() a proper GstChildProxy method

No longer accept any old GObjects. This makes things nicer for
bindings. If a utility function that handles both nicely
is deemed worthwhile, we can still add one to gstutils.

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

value: when serialising arrays or lists, handle types we can't serialise more gracefully

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

consistencychecker: add some more details to failure messages

Mention pad where the problem occured, and the event name.

tests: fix collectpads test

After an EOS we must send a FLUSH_STOP event if
we want to send data again.

event: fix leak in gst_event_parse_stream_start()

gst_structure_id_get() will make a copy of the string
extracted, but we're assigning it to a const gchar *.

tests: make pipeline test valgrind clean

tests: fix pipeline unit test

Which was disabled because it failed.

scripts: fix unterminated quoted string in create-uninstalled-setup.sh

docs: mention gst_video_format_parse_caps() in porting guide

docs: fix up docs a bit

childproxy: fix up g-i annotation for _lookup() paramspec return value

No ref is returned here.

win32: update .def file for new buffer functions

baseparse: fix reverse playback with upstream demuxers that support it

Don't just return FALSE for seek events with negative rates when
operating in push mode. An upstream demuxer may support this just
fine, so if we're not operating in pull mode always check upstream
first if it can handle the seek event. This fixes reverse playback
where the upstream demuxer supports it (e.g. with qtdemux). The
same code would work fine in 0.10, because baseparse will just
call the default pad event handler if FALSE was returned from the
baseparse event handler, and the pad event handler will just
forward it upstream. In 0.11 the baseclass or subclass is
responsible for chaining up to the parent class or forwarding the
event upstream in any case.

Disable reverse playback in pull mode for now, there seems to
be something going wrong with the segment configuration in that
case.

basetransform: do not error on not-negotiated

Don't error out too early and let upstream decide if it can
workaround a not-negotiated problem

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

basesrc: retry on not-negotiate if a reconfigure is pending

Before erroring out on not-negotiated returns, check if the pad
has the reconfigure flag set and retry.

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

pad: add gst_pad_needs_reconfigure

Add an alternative version of gst_pad_check_reconfigure that doesn't
clear the reconfigure flag.

Useful for increasing error resilience without duplicating the
reconfigure code in pad task functions.

API: gst_pad_needs_reconfigure

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

pad: add GST_PAD_LINK_CHECK_DEFAULT to GstPadLinkCheck

This allows introspection-based bindings to access
Gst.PadLinkCheck.DEFAULT instead of
Gst.PAD_LINK_CHECK_DEFAULT.

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

buffer: mark gst_buffer_wrapped* data as array

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

introspection: fix some warnings generated by g-ir-scanner.

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

buffer: convert gst_buffer_* macros to functions

GObject Introspection does not support macros.
This is needed for bindings. We can still add back
macros or inline functions again later if we think
it's worth it.

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

netclientclock: fix printf format in debug message

bufferpool: fix max_buffers handling

When max_buffers > 0 and the pool is empty, actually try to allocate more
buffers up to the max_buffers limit.
We need to add a counter for this to count how many buffers we allocated and
check this against the max_buffers limit.
Reorganise and clean up some code.

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

netclientclock: simplify by using g_socket_condition_timed_wait()

No need to use a custom main context and custom timeout sources,
just use g_socket_condition_timed_wait() instead, which was added
for exactly this case.

Also seems to help with the unit test deadlocking with glib 2.33.x

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

gstobject: fix double string escaping in gst_object_default_deep_notify()

Make output of gst-launch -v readable again.

last-message = "event\ \ \ \*\*\*\*\*\*\*\ \(fakesink0:sink\)\ E\ \(type:\ tag\ \(20510\)\,\ GstTagList-stream\,\ taglist\=\(taglist\)\"taglist\\\,\\\ video-codec\\\=\\\(string\\\)H264\\\,\\\
 minimum-bitrate\\\=\\\(uint\\\)636611\\\,\\\ bitrate\\\=\\\(uint\\\)980729\\\,\\\ maximum-bitrate\\\=\\\(uint\\\)1116707\\\;\"\;\)\ 0x15bc760"

vs.

last-message = event   ******* (fakesink0:sink) E (type: tag (20510), GstTagList-stream, taglist=(taglist)"taglist\,\ video-codec\=\(string\)H264\,\ minimum-bitrate\=\(uint\)856039\,\ bitrate
\=\(uint\)1019748\,\ maximum-bitrate\=\(uint\)1116707\;";) 0x11149e0

miniobject: check writability

fix the writability check for miniobjects. We should check the shared counter.

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

allocator: Set the alignment at the correct place in GstAllocationParams

Back to development

Release 0.11.93

tests: remove silly test_fail_abstract_new check

Our check would make sure that GLib segfaults when
someone tries to instantiate an abstract type, which
is an extremely useful thing to check for.

In newer GLibs this is fixed and we get an abort with
a g_error() now it seems, so let's just remove this
check entirely.

examples: don't put things with side effects inside g_assert()

They will be defined away to NOOPs otherwise in release builds.

win32: update for stream-id API additions

parse: fix for new GstChildProxy::child-added signal callback signature

Fixes crash with gst-launch-1.0 uridecodebin uri=... suburi=... ! ..

bus: Add allow-none to the function argument of gst_bus_set_sync_handler()

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

docs: Make sure scanner gets required libraries

consistencychecker: print which event we received before stream-start

basesrc: don't try to answer URI queries with NULL URIs

Should make unit tests in -base that use appsrc a bit happier.

event: Update for stream-start event API changes

event: Add new stream-id field to the stream-start event

This is supposed to allow uniquely identifying a single stream.

inputselector: Use the first created pad by default

This guarantees a bit more consistency in which input stream will
be selected by default. It would previously be the first pad on which
an event/buffer/query was received ... which was racy and non-predictable.