basesink: small cleanups

framestep: implement backwards framestep

Update framestep document, we want to pass the flush flag in the step-done
message.

Add flush flag to the gstmessage.

Update examples to use the new step-done message api.

Implement framestep with playback rates < 0.0 too.

basesink: add framestepping in time

examples: step in time as well

example: print step_done message and sync

Dump the step_done message contents.
Sync against the clock when going to PLAYING.

basesink: keep track of stepped time

Pass running_time around so that the stepping code can calculate the elapsed
time correctly.

basesink: move stuff around, more stepping

Make start and stop_stepping methods and move their invocation in the right
places.

Perform the atual stepping operation where we have full context about the
timestamps.

Add frame stepping in PAUSED example

basesink: first stab at frame stepping in PAUSED

Unlock the prerolled frame and recheck if we need to step.
Keep a simple counter for the frames we're about to skip while stepping and
preroll/post step_done when stepping finished.

add new API for framestepping

Add new STEP event and methods for creating/parsing the event
Update design docs.
Add new STEP_DONE message and method to create/parse.

API: GstEvent::gst_event_new_step()
API: GstEvent::gst_event_parse_step()
API: GstMessage::gst_message_new_step_done()
API: GstMessage::gst_message_parse_step_done()

structures: don't leak invalid or empty strings when we warn

Fixes minor memory leak in unit tests caused by the recent changes.
Since we're expected to take ownership of the GValue in the structure
field struct here, we need to unset it if we don't use it.

controller: add test for cubic int. and too few control points

Added another tests to check some worries in Bug #582564.

fakesrc: add a FIXME comment for blocksize vs. size-max property issue

identity: hack around g_object_notify() bug by protecting it with a lock

Out-of-band events might lead to us calling g_object_notify() from a
non-streaming thread, which can cause crashes if g_object_notify() is
being called from the streaming thread at the same time. See #554460.

controller: use real world number in benchmark

registry: fix comment formatting

fakesink: hack around crasher bug in g_object_notify() for out-of-band events

GObject may crash if two threads do concurrent g_object_notify() on the same
object. This may happen if fakesink receives an out-of-band event such as
FLUSH_START while processing a buffer or serialised event in the streaming
thread. Since this may happen with the default settings during a common
operation like a seek, and there seems to be little chance of a timely fix
in GObject (see #166020), we should hack around this issue by protecting all
of fakesink's direct g_object_notify() calls with a lock.

Also add unit test for the above.

Fixes #554460.

taglists: make _get_{string|pointer} return FALSE for NULL values

Make gst_tag_list_get_string() return FALSE for NULL strings and
empty strings, and gst_tag_list_get_pointer() return FALSE for
NULL pointers, like we do with dates and buffers.

Fixes #560345.

taglists: warn if someone tries to add empty or NULL string tags to a taglist

Also warn if an element or application tries to add a field with an
empty string to a structure (NULL strings are still needed and
allowed though) and do all those checks in the right function.

Fixes #559643.

structure: add gst_structure_id_new() convenience function

Add convenience wrapper for gst_structure_id_empty_new() plus
gst_structure_id_set() and use it in a few places.

API: gst_structure_id_new()

micro-optimisation: use GST_QUARK in more places

Use gst_structure_id_empty_new() in combination with GST_QUARK
rather than gst_structure_id_new() when creating message, event,
query and taglist structures. Mostly just because we can.

element: reset start_time in lost state

docs: update element an pipeline docs

docs: remove a TODO item that is fixed now

pipeline: deprecate old methods, fix test

Deprecate the old _set_stream_time and _get_last_stream_time methods because
they are now equivalent to the better named _set/_get_start_time.

pipeline: use START_TIME to keep track of time

Use the element START_TIME to keep track of the running time when the pipeline
paused so that it can be used to restore the base_time.
Take the start_time before setting the children to PAUSED so that we can
distribute the start_time to the children.

bin: set the base_time and start_time better

Simply set the start_time and base_time on the element instead of calling the
setters.

bin: make the bin set the start_time on elements

Set the start_time of the bin on the elements when they are added to the
pipeline and when a state change happens.

element: add start_time field an methods

Add a start_time field and some methods. The start_time will contain the
running_time of when the element last went to paused. This time can be user to
report the position in PAUSED but also to do more correct clipping and
stepping later.

adapter: fix _masked_scan_uint32() at boundaries

gst_adapter_masked_scan_uint32 could return values smaller than offset
if the first byte(s) of the mask are 0 and the pattern matches the
beginning of the adapter.
Added examples to documentation of gst_adapter_masked_scan_uint32().
Also added some more masked boundary tests.
Fixes #584118

pad: add pad private structure

Add pad private structure and move the new chainlistfunc into the private
struct. This avoids ABI breakage and allows us to expand in the future.

Add missing symbol to the win32 exports

This was accidentially removed by my last commit.

buffer: avoid memory leaks

Avoid leaking the caps of the dest buffer and avoid doing needless caps
refs.
When the source and target buffers are the same, return immediatly.

API: Add gst_message_{new,parse}_tag_full() to get/set the source pad

Fixes bug #582588.

Revert "element: Set the originating pad as message source in gst_element_found_tags_for_pad ()"

This reverts commit bebfde75027e975b7e7c74c6358c5be83ea4ac9f.

This change shouldn't be done in a stable release series as
applications are actually expecting the sender to be an
GstElement. One example is totem.

Update common

Fix 'make distcheck'

The check-enum-gettypes rule didn't work for 'make distcheck' since
it makes assumptions about the location of the source files from the
current working directory which isn't true during distchecking.

manuals.mak: attempt to make 'make distcheck' work with -jN

Attempt to fix the 'cannot create regular file build/image.entitites:
file exists' error I got.

docs: fix cdparanoia example pipeline in gst-launch man page

element: fix typo in comments

dist: Fix the name of the header to dist: testrtpool.h, not rtpool-test.h

Update common

clock: remove assertion

Remove an assertion, this is not really an error in all cases.
Fixes #582010

clock: enable monotonic clock when we can

Enable the monotonic clock by default when we can.
Fixes #583554

docs: add Image to draft klass documentation

pad: keep task ref before releasing the lock

Keep a ref to the task on the pad so that a concurrent stop can stop and join
the task.

gsttask: avoid join to return early

Unset the running flag after we released the lock for posting the stream-status
message. If we set the running flag to FALSE too early, the join method will
just continue without waiting for the message to be posted, leading to potential
crashes.

preset: fix update rule

Only update the preset from system, if we had a preset before and system
version is newer.

controller: add a benchmark to verify the switch to gsequence

controller: add more error handling to example

registry: don't free node-date and deref again. Fixes #580579

When writing a cache chunk fails, we were freeing the node and jump to a final
cleanup which dereferenced a null pointer. Leve freeing the node to the cleanup
code in fail_free_list. (sorry for committing wrong fix before).

registry: don't free node-date and deref again. Fixes #580579

When writing a cache chunk fails, we were freeing the node and jump to a final
cleanup which dereferenced a null pointer. Leve freeing the node to the cleanup
code in fail_free_list.

controller: add G_LIKELY and join two if for same condition

A G_LIKELY for the sequence!=NULL checks. Join two ifs to an if-else. Add
indent guides to keep indent form breaking the function declaration

gsttypefindhelper: Fix indentation

docs: fix gtk-doc warnings

Move MT safety to main description (it does not belong to Return: or Since:
statement). Add a few missing return docs. Downgrade a normal comment froma doc
comment. Fix a doc header to only contain symbol name.

Automatic update of common submodule

From d3a8fab to 888e0a2

dist: Add rtpool-test.h to the sources list so it gets disted.

Fixes the distcheck

gitignores: Ignore the clockstress benchmark binary

controller: Silence a warning from the GSequence being NULL.

Fix a warning that occurs when the self->priv->values is NULL and
the code tries to retrieve an iterator from it. The warning was showing
up in the checks for the volume element.

docs: Fix up some documentation warnings.

Since: tags should always be the last thing in a doc block, apparently.
Add some Returns: descriptions to some recent functions.

docs: update docs for stream_time->running_time

Change some instances where we wrongly refer to stream time where it should have
been running time.

gst-launch: don't use G_GUINT32_FORMAT in translatable string

xgettext doesn't handle this very well. Fixes #583419.

autogen.sh: can remove the -Wno-portability from here now

since we added it to configure.ac.

adapter: improve the flush function

Remove a compare and branch from flush.

controller: fix assertion when freeing the control source

adapter: potentially save a memcpy in _take

Directly use the assembled_data in _take() functions when we can instead of
copying it out.

adapter: micro optimisations

adapter: avoid comparisions in fast path

Small tweaks to reduce the number of useless compares in loops.

tests: one more adapter test

adapter: avoid branch in copy code

loadsave: fix requestpad handling and serialisation order.

Support request pads when loading. Reverse pad serialisation order to
preserve it when recreating the pipeline.

defs: add new symbol

docs: add new symbol to docs

adapter: add _masked_scan_uint32

Add a reasonably optimized new gst_adapter_masked_scan_uint32() function
to scan the adapter for a pattern after applying a mask.

Add some unit tests.

API: GstAdapter::gst_adapter_masked_scan_uint32()

Fixes #583187

adapter: more optimisations

Remove duplicate copy code (_peek_into and _copy) and make a unified
optimized copy function.

configure: pass -Wno-portability to automake to suppress warnings

GNU make is required, no point pretending otherwise.

docs: mention that GST_FORMAT_{PERCENT|BUFFERS} are not implemented

gstclock: Fix ABI breakage on 32 bit architectures

The padding of GstClock is a GstClockTime and not a
gpointer, so adding a pointer requires the padding
size to be changed depending on the pointer size.
Use an union instead.

Fixes bug #582878.

[gstvalue] adds safety parenthesis to macros missing them.

[gstutils] Adds more safety to GST_WRITE_* and GST_READ_ macros.

Adds safety ( ) to parameters in _GST_PUT and _GST_GET macros.
Fixes #582708.

clock: use seqlocks to parallellize readers

stress: add a clock stresstest

Add a stresstest for gst_clock_get_time().

Makefile.am: update for added/moved/removed files that weren't dist-ed.

docs: Release script modifications

controller: Use ordered GSequence instead of GList

This makes lookups and insertions O(log n) instead of
always O(n) for insertions and O(n) in worst case for
lookups.

Fixes bug #582564.

docs: rename and delete some design docs

gst-launch: Print the path string for message sources

This reduces confusion if the message source is a pad
and only "src" is printed as source.

element: Set the originating pad as message source in gst_element_found_tags_for_pad ()

Fixes bug #582588.

element: add gst_element_lost_state_full()

Add a gst_element_lost_state_full() with an extra argument to control
distribution of a new base_time. We will need this for flushing step
operations.

API: GstElement::gst_element_lost_state_full()

adapter: don't use realloc, it does a memcpy

Don't use realloc to grow the scratch area because we don't want the memcpy the
old useless data into the new area before we write our new stuff in it.

docs: update trickmode document

adapter: use g_realloc for resizing the buffer

Use g_realloc for resizing the internal buffer instead of a
less fancy _free/_malloc pair.

adapter: move new member to private struct

Move the new members to a private struct because we don't have enough padding
anymore on 32-bits platforms.

adapter: update some docs

tests: add another test for adapter timestamps

tests: add new timestamp unit test

defs: add new symbol

adapter: add method to keep track of timestamps

Keep track of the timestamp and offset associated with the current head of the
adapter.

API: GstAdapter::gst_adapter_prev_timestamp()

adapter: small cleanups

debugutils: show more pad-details

Show pad activation mode and pad-flags inside the pad. Write down some ideas
about how we could improve the caps layout.