tests: fix spurious failure in test_collect collectpads test

pop() in collected callback.

There were three threads in the test cases that hanged: the test thread and two
threads that push buffers. Each thread push one buffer on one pad. There are
two pads in the collectpads so the second buffer will trigger the
collect-callback.

This is what happens when the hang occurs:

The first thread pushes a buffer and initializes a cookie to the value of a
counter in the collectpads object and waits on a cond for the counter to change
and for someone to consume the buffer (i.e. _pop() it).

The second thread pushes a buffer and calls the collected callback, which
signals the cond that the test thread is waiting for.

The test thread pops both buffers (without holding any lock). Each call to
_pop() increases the counter broadcasts the condition that the first thread is
now waiting for. It then joins both threads (hangs).

The first thread wakes up and returns, since its buffer has been consumed.

The second thread starts executing again. When the callback, called by the
second thread, has returned it initializes a cookie to the value of a counter,
which has already prematurely been increased by the test thread when it popped
the buffers, and wait's on a cond for the counter to change and for someone to
consume the buffer (i.e. _pop() it). Since the buffer has already been poped
and the counter has already been increased it will be stuck forever.

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

gst-uninstalled: remove ffmpeg libs from dynamic linker paths

We link those libs into the plugin statically, or use external
system libs, but never the internal snapshot dynamically.

gst-uninstalled: add gst-editing-serves to GI_TYPELIB_PATH

So pygi can find it.

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

utils: make sure slow GST_READ_UINT* variants don't have unexpected side effects

Fixes unit test on systems where unaligned memory access is not possible.

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

launch: don't exit the loop on buffering in paused

When we receive a buffering message of 100% in the paused state, we exit
the event_loop and move to the PLAYING state. What should happen is that
we wait for both ASYNC-DONE and 100% buffering before continueing.

design: update controller design and add some thoughs for future stuff

docs: mention clipping of values in control-binding docs

controller: code cleanups

Use a property for accessing the control-source on the binding. Drop base_init
on the test object.

query: gst_query_get_n_allocation_params() returns a new ref to the allocator

controller: remove a bogus test

We previously forgot to initilize the amplitde property to the default and thus it was 0.0. Therefore a default lfo controlsource returned a series of 0.0 and the test was asserting on that.

controlbinding: relax the pspec for the control-source

We can change control sources on controlbindings.

lfo: set a sensible lower boundary for the frequency

Use DBL_MIN, which is a the smalles double greater than zero that is not in
denormal format. This exposes the limit better than the runtime check.

Automatic update of common submodule

From 2de221c to 04c7a1e

lfocontrolsource: init the amplitude to the default and update the docs

gst-uninstalled: Add all the base/bad libraries and remove obsolete ones

basesink: fix use of uninitialized variable

Running suite(s): GstBaseSink
==22023== Conditional jump or move depends on uninitialised value(s)
==22023==    at 0x505FFCE: gst_base_sink_get_sync_times (gstbasesink.c:1936)
==22023==    by 0x5068C80: gst_base_sink_do_sync (gstbasesink.c:2379)
==22023==    by 0x506BCD2: gst_base_sink_default_wait_event (gstbasesink.c:2903)
==22023==    by 0x50633A4: gst_base_sink_default_event (gstbasesink.c:2918)
==22023==    by 0x6F5C216: gst_fake_sink_event (gstfakesink.c:383)
==22023==    by 0x505F164: gst_base_sink_event (gstbasesink.c:3108)
==22023==    by 0x52FA090: gst_pad_send_event_unchecked (gstpad.c:4822)
==22023==    by 0x5303756: gst_pad_send_event (gstpad.c:4984)
==22023==    by 0x40165B: basesink_test_gap (basesink.c:148)

buffer: Fix memory copying logic in copy_into()

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

registrychunks: Use correct print format specifiers to fix compiler warnings

Fix misspellings of 'continuous'

collectpads: take DTS into account

Importantly, this patch converts DTS to running time.  Less importantly,
and possibly a problem for some muxers, is that it orders buffers by
DTS (if it is valid, otherwise PTS).  This is generally correct, but
might be somewhat surprising to muxers.

Also note that once converted to running time, DTS can end up negative.

manual: improve the controller docs a little more

Reword some sections. Explain value mappings better.

gst-uninstalled: add gst-libav to pkg-config path

seeking: add more logging for seeking

Especially add logging to error code paths.

buffer: Gracefully handle gst_memory_copy() returning NULL without crashing

gst_buffer_copy_into() and gst_buffer_resize_range() can now fail.

allocator: small internal cleanups

Rename System memory allocator to GstAllocatorSysmem and the memory to
GstMemorySystem.

memory: add method to check memory type

Add a method to check if a memory was allocated from an allocator of
a given type.

API: gst_memory_is_type()

docs: improve docs a little

use GST_MESSAGE_DURATION_CHANGED in docs and code

memory: don't unref allocator too soon

Unref the allocator *after* we have freed the memory. We also need to keep
a ref to the allocator around because following the now freed memory would
lead to crashes.

buffer: Fix inverted logic for deciding if memory should be shared or copied

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

porting: mention segment accumulation in the porting guide

This needs more detail, but at least gives people a hint on the issue.

docs: s/start/done/ copy'n'paste mistake

query: Add new API to remove allocation params and pools from the allocation query

query: Document that the first allocator in the allocation query should allow mapping to system memory

memory: Keep a reference to the allocator

Otherwise the allocator might get freed while it's still used
by the memory

buffer: If sharing a GstMemory fails, fall back to copying it

docs: porting-to-1.0.txt: some element names have changed

memory: It's contiguous, not continous

memory: Add new memory flag to specify that memory can't be mapped

memory: Add memory flag to mark physically continous memory

basetransform: don't pass NULL outcaps to transform_size on shutdown

gst_pad_get_current_caps() on the source pad might yield NULL caps
if we're being shut down and the source pad has already been
deactivated by the other thread that's changing state. Just bail
out in that case, instead of passing NULL caps to the transform_size
function, which it might not expect.

Fixes spurious warnings in audioresample shutdown unit test.

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

gst-uninstalled: Add ORC

utils: avoid unexpected side-effects of GST_WRITE_* macros

Make sure the data argument is only evaluated once.

check: add some more fail_unless_*() macros for convenience

API: fail_unless_equals_int_hex
API: assert_equals_int_hex
API: fail_unless_equals_int64_hex
API: assert_equals_int64_hex
API: fail_unless_equals_uint64_hex
API: assert_equals_uint64_hex
API: fail_unless_equals_pointer
API: assert_equals_pointer

scripts: add new -base allocators library to gst-uninstalled search paths

collectpads: add two more tests using collectpads within an element

Add a static plugin with a rudimentary element using collectpads and do some
pipeline based tests.

docs: Fix some ambiguous wording

triggercontrolsource: add missing end_iter check for sequence

Avoid accessing the end-iter, this is a marker without a data field.

docs: link to the appropriate messages from gst_elements_set_state() docs

For an async state change return one would wait for ASYNC_DONE or STATE_CHANGED.

buffer: add since marker for new COPY_DEEP buffer flag

buffer: add option to deep copy a buffer

Add a buffer copy flag to force a memory copy in all cases.

Revert "utils: Use gst_pad_get_pad_template() in gst_element_get_compatible_pad_template()"

This reverts commit 1a1a9e143fb0e155d7627aa8e489cd5d04bc093c.

This breaks the pipelines/tagschecking unit test for some reason
(fakesrc ! capsfilter ! qtmux linking fails now). It might be
a bug in the unit test of course, but someone will need to
investigate this. Reverting for now.

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

allocator: improve fallback copy function

Only use the allocator of the copied memory when we can use the default
_alloc function on it. Otherwise we will have to use the default
allocator for the copy.

docs: fix location in user's home directory where GStreamer looks for plugins

It's based on the xdg user data dir now in 1.0.

utils: Use gst_pad_get_pad_template() in gst_element_get_compatible_pad_template()

motivation comes from: /* FIXME: why not gst_pad_get_pad_template (pad); */
this code path is quite nicer, we now only revert to creating the template
if gst_pad_get_pad_template fails.

with this fork, we gain a non-allocation of GstCaps *templcaps

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

gst-launch: Use g_unix_signal_add() to handle keyboard interruption

Current implementation uses a traditional signal handler and a 250ms
timeout callback in the event loop.  Adding a GSource with
g_unix_signal_add() to the GMainLoop is a much more elegant solution.
The signal handler with this approach can send a message to the bus
directly rather than set a flag as all dispatching intricacies are handled
by GLib.

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

docs: flesh our 'Running GStreamer' bits a bit

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

docs: flesh out gst-launch-1.0 man page a little

Fix up default location of the registry.

Mention more options for GST_DEBUG (wildcards and
named debug levels).

Explain what to do with the dot files that can be
produced by setting GST_DEBUG_DUMP_DOT_DIR.

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

docs: fix advanced-metadata code example in manual

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

message: accept NULL error argument in gst_message_parse_{error,warning,info}

And simplify code a bit while at it.

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

value: Remove set-style bitmask intersection/union/subtraction functions

Set operations on the bitmasks don't make much sense and result
in invalid caps when used as a channel-mask. They are now handled
exactly like integers.

This functionality was not used anywhere except for tests.

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

bin: The latency query should return TRUE by default, different to other queries

Fixes unit test failures caused by f3d268de7f7fb1161778a9a95e0d54d8c89ef626

structure: Make sure that subsets have all fields of the superset

"video/x-h264,parsed=(boolean)true" is not a superset of
"video/x-h264,stream-format=(string)byte-stream,alignment=(string)nal"
for example.

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

update exports for baseparse API changes

baseparse: Fix doc typo

baseparse: add gst_base_parse_set_ts_at_offset()

Sets the buffer timestamps based on last seen timestamps at a
particular offset into the frame.

API: gst_base_parse_set_ts_at_offset()

adapter: Add gst_adapter_prev_[pd]ts_at_offset()

Original patch written by Michael Smith <msmith@rdio.com>.

API: gst_adapter_prev_pts_at_offset()
API: gst_adapter_prev_dts_at_offset()

bin: query sink elements and source pads of the bin

gst_bin_query() now forwards the query to the source pads as well if
none of the sinks of the bin satisfied the query. This helps in the
case of DURATION queries done a bin containing a source element.

Fixes bug 638749

bin: Let gst_bin_send_event() send events to ghost pads as well

compat: kill more uses of gst_pad_set_caps()

gstvalue: Adding offset to GstSegment serialize/deserialize.

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

baseparse: improve PTS interpolating

... and tracking of DTS.  Fixes cases where PTS is locked on to the
DTS of an incoming buffer with no PTS with invalid data, leading to
no outgoing PTS (since it is not allowed smaller than DTS).

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

docs: add more porting details

controlbinding: error handling for binding controlsources to wrong properties

Add warning if property is not suitable for controlling. When adding a control-
binding check that pspec!=NULL.

element: remove old docs about iterators

bin: remove old comment

The iterators now return a GValue and not the object directly anymore.

bin: reset GValue from iterator after usage

tests: add basesink test

basesink: handle sync of EOS after item without duration

After a buffer or GAP without duration, an EOS event should be rendered
immediately instead of waiting for the end of the segment.

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

gstpipeline: fix failed assertion caused by seeking pipeline with NULL clock

Under certain GST_STATE_CHANGED_PAUSED_TO_PLAYING transitions, a pipeline with
a NULL clock will fail an assertion due to an unchecked call to gst_object_ref().
This is fixed by simply adding a check and only ref-ing if the clock is not NULL.

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

structure: change argument name for docs

debugutils: fix order of caps on an unnegotiated link

headlabel is the sink_pad (where the link points to) and not the other way around.

docs: update the controller docs

Add the control bindings to the docs. Add a little more detail.

porting: a few updates for the porting guide

dataqueue: can't pass a GType through GINT_TO_POINTER

Use GSIZE_TO_POINTER instead. sizeof(GType) may be larger
than sizeof(gulong) and sizeof(int), so the casts may
chop off some bits from the GType value on some architectures.

tests: unit test to trigger the queue/flushing race condition bug for allocation queries

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

queue: remove query from queue if queue is flushing

When querying a queue that is flushing we end up adding
a query to the queuearray without taking a reference to
that query (because the normal functionality is to block
until that query is done and discarded from the queue).
This later causes problem if the query is unreffed outside
of the queue before we discard the queue. There is a check
to avoid unreffing any lingering query-objects, but since
the query has been deleted that check fails.

This commit depends on other fixes done to gst_queue_array_find()
and gst_queue_array_drop_element().

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

queuearray: make _find() find the value if no compare function is provided

Allow NULL as compare function for direct value lookup.

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

tests: check return value of gst_queue_array_drop_element() too

Was added when the API was made public in git master.

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

tests: one more test for gst_queue_array_drop_element()

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

Conflicts:
tests/check/libs/queuearray.c

queuearray: fix gst_queue_array_find()

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

queuearray: fix gst_queue_array_drop_element()

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

Conflicts:
libs/gst/base/gstqueuearray.c

docs: align the comments correctly with the declaration in bitreader docs

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

basesrc: handle renegotiation correctly

Don't retry to negotiate when we fail to negotiate but instead produce a
NOT_NEGOTIATED error. We only want to retry negotiation if the result from
gst_pad_push() returned NOT_NEGOTIATED.

Automatic update of common submodule

From a942293 to 2de221c

example: fix FIXME in example

Use setpriority to raise priority

g-i: add built enumtypes headers and sources to gir creation

Add gstenumtypes.h/c for inclusion with g-ir-scanner. This fixes
problems where introspection based bindings think GstState is
typeless due to the GType not being included as an annotation.

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

tests: use g_timeout_add_seconds in pipeline stress test

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

docs: Put the right path for the gstttestclock include file

pwg: rename variable

The filter variable was used twice for different things.

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