Merge branch 'master' into 0.11

Merge branch 'master' into 0.11

Conflicts:
configure.ac
gst/gstelement.c
gst/gstelement.h
gst/gstpad.c
gst/gstutils.c
libs/gst/base/Makefile.am
libs/gst/check/Makefile.am
libs/gst/controller/Makefile.am
libs/gst/dataprotocol/Makefile.am
libs/gst/net/Makefile.am
win32/common/libgstreamer.def

basetransform: Be smarter with pad allocs

Avoid doing unnecessary pad-allocs when on passthrough mode.
If multiple basetransform elements are on a pipeline, they
would do a pad-alloc for each received buffer, each element
would do this, so we would have lots of pad allocs on the
pipeline for a single buffer being pushed through it.

This patch attempts to reduce this amount by avoiding
doing pad-allocs if the element has already done it
after the last pushed buffer. So it will only be allowed
to do a new pad-alloc after it has pushed a buffer, so we get
1x1 pad-alloc and buffer ratio

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

docs: fix typo in gst_index_new() docs

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

fakesink: print new MEDIA4 flag as well

miniobject, buffer: steal miniobject flag for use as MEDIA4 buffer flag

This was required to add a new MEDIA4 buffer flag for indicating
progressive/mixed telecine video buffers. There is no space for
additional flags in GstBuffer, so steal one from GstMiniObject.

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

docs: add some more Since: markers to atomic queue docs

Revert "element: Call ->release_pad() to clean up pad"

This commit changes the request pad behaviour for plugins and applications.

Reopens Bug #402562

The proper fix for that bug is to keep track of created request pads.

This reverts commit a5e44ffffaa6d7a8d7af8dcb77e37990996253a5.

index: add FIXME-0.11: comments

docs: improve index docs

docs: spell-check

fakesink: print buffer flags

element: Call ->release_pad() to clean up pad

Fixes #636011 and #402562.

index: fix creation of writer id for unparented pads

Also do some cleanup in the impl.

value: add (de)serialisation function for uchar

.. since we sadly have a plugin in -good that has a uchar property
(cmmlenc)

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

atomicqueue: fix include order
atomicqueue: fix include order# Please enter the commit message for your changes. Lines starting

bus: make the bus almost lockfree

Use new GstPoll functionality to wakeup the mainloop.
Use an atomic queue on the writer side to post the messages.
The reader side it protected with the lock still because we don't want multiple
concurrent readers.

defs: fix defs file for new symbols

atomicqueue: use correct array sizes

atomicqueue: fix docs some more

atomicqueue: add refcounting and docs

atomicqueue: make sure a min initial_size is used

atomicqueue: add an atomic queue

Add an atomic queue. The queue can be used from multiple threads simultaneously
and without taking any locks or doing any blocking operations. This makes it
highly scalable for things like the bus, bufferpools and object recycling.

check: fix a leak in the bus unit test

deinit: add progress type class unref

utils: tell also what pad a pad is already linked against

file{sink,src}: Check if non-URI characters are escaped, but only for the URI not the location property

file{src,sink}: Fix unit tests

filesink and filesrc should return exactly the same URI as passed
and must not escape path separators.

filesink: Fix escaping of URIs

Especially don't escape / as path separators

filesrc: Fix escaping of file uris

Fixes bug #642393.

message: add timeout to progress message

Add a timeout member to the progress messages to let the application know about
the timeout so that it can do some gui things with it.

design: mention timeout in the progress message

design: make progress draft official

message: rename category to code

message: add new message quark

docs: add more standard categories

gst: register new type

check: add progress message unit test

message: add progress message functions

docs: update progress field

Avoid naming the progress free text field 'message' as it conflicts with the
message itself.

queue2: don't read beyond the end of file upstream in pull mode

... which could lead to a premature eos being reported downstream,
rather than a successful partial read which would result when
performed directly on e.g. basesrc.

utils: return real error in compatible link check

We need to ensure we call gst_pad_check_link() with the two pads in the correct
order. The order depends on wheter we iterate src or sink pads.

Signed-off-by: Chen Rui <rui.chen@tieto.com>

pad: Check sinkpad for flushing

Check the sinkpad for the flushing state before calling the chainfunction on the
pad. We do this by checking the cache (which is also cleared on the srcpad when
the sink is set to flushing).

Fixes #641928

basetransform: Check for pad alloc caps when suggestion is not fixed

If after computing the suggestion with downstream caps we still have
a non-fixed suggestion caps try to intersect with the input caps
of the pad alloc to avoid useless renegotiations.

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

basesink: improve duration calculation

Keep track of the average distance between incomming timestamps and
use that to estimate the frame duration when buffers have no duration set on
them.

basesink: improve rate calculation

When there is no duration on input buffers, assume the rate is 1.0
instead of (the undefined) 0.0.

basesink: improve average duration calculation

Improve the calculation of the duration. When we have no input duration set on
the input buffers stop is set to start and then we end up using a 0 duration in
the average calculation.

basesink: rename variable

Rename an internal variable to better reflact what its value means.

parse-launch: trim whitespaces

parse-launch: fix typo in pad-list length comparision

It was comparing the length with itself.

Fixes #642071.

Automatic update of common submodule

From f94d739 to 1de7f6a

gst-launch: pretty-print datetime tags

gstreamer.doap: update mailing list host

basesink: fix some comments

basesink: keep track of earliest QoS timestamp

Keep track of the earliest allowed timestamp according to the latest
QoS report and drop buffers before that time. Activate this filter
when throttling is enabled. We could later also activate this in the
other QoS cases.

See #638891

basesink: use new QoS type

Use the new QoS type and send throttling QoS messages.

basesink: add property to configure a throttle-time

Add a property to configure the throttle time on a sink. The
property is not yet used.

See #638891

event: add QoS event type

Add a parameter to the QoS event to specify the QoS event type.
Update docs and add unit test.

See #638891

clock: fix parameter docs

design: tweak docs a little

design: update QoS document

Add new QoS types and talk about the new throttle QoS message.

docs: fix some typos in the bufferpool draft

events: fix g-i annotation for gst_event_new_tag() which takes ownership of the tag list

docs: mention that it's necessary to set the state of elements added to an already-running pipeline

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

design: add draft for first ideas for a bufferpool feature

Add a first draft with some ideas and use cases for the implementation
for bufferpools. The purpose is to be able to make elements negotiate
their buffer requirements as well as provide an infrastructure to
preallocate and reuse buffers in an easy way.

docs: clarify the NOMINAL_BITRATE docs

Tell that its a target bitrate and actual values might be different.

poll: trigger rebuild setup in _new

Failing to do so in the Windows case (implicitly triggered otherwise)
would have a subsequent _wait return immediately leading to high CPU
usage timeout loops.

Fixes #640675.

info: make adding/removing of gst_debug_log_default() work properly

Make adding/removing gst_debug_log_default() work reliably in all
circumstances. The problem was that depending on platform and linker
flags the function argument might resolve to different addresses,
which made it impossible to remove the default log function added
in gst_init() from application code (because the pointer values
didn't match). The new approach should keep things simple by passing
NULL for the default function, which the code in libgstreamer can
then handle.

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

Revert "info: use the publicly visible address to fix the tests"

This reverts commit eb56687a6dfd207507a4ca000eae53f93b5e33ea.

While this commit may have fixed a problem on one of the build bots,
it didn't actually fix the original bug reported for win32.

Also, it causes other problems, such as the lookup failing when
called from C++ code (gst-phonon, amarok).

This needs to be fixed differently.

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

queue2: properly identity dequeued event as such

... which avoids terminating with ERROR rather than UNEXPECTED.

gst-uninstalled: use $GST_PREFIX variable

This makes it easier to change the prefix by editing the script.

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

taglist: add a new "encoded-by" tag

Usecase: ID3v2 TENC ("Encoded by") frame.

API: GST_TAG_ENCODED_BY

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

net: use socklen_t where appropriate rather than specific type

In particular, fixes Cygwin build where socklen_t is defined as int
in line with native win32 api definition.

bus: honour any per-thread default main context set via g_main_thread_push_thread_default()

Makes gst_bus_add_watch(), gst_bus_add_watch_full(), gst_bus_add_signal_watch(),
and gst_bus_add_signal_watch_full() convenience functions automatically pick up
any non-default main contexts set for the current thread via
g_main_thread_push_thread_default().

valve: Only set discont on the first buffer after drops

Reset the discont member after setting discont on the first buffer after
dropping.

GstElement: Fix warning with GCC 4.6

gstelement.c: In function ‘gst_element_get_request_pad’:
gstelement.c:1052:18: error: variable ‘tmp’ set but not used [-Werror=unused-but-set-variable]

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

identity: print unset buffer timestamps or durations as 'none'

Like fakesink and fakesrc do.

typefind: don't take object lock for reading the found caps

Once we switch to normal mode, we're not typefinding anymore and thus the caps
will not change. Therefore can avoid the object lock in the data-flow path.
The locking was added in order to fix bug #608877.

docs: fix a few more typos

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

docs: flesh out typefindelement docs some more

Mention that have-type signal may be emitted from streaming
thread or application thread, and fix a typo.

typefind: code and comment cleanups

Make code two places of the code the pushes the buffer store more similar. More
comments and debug logging.

trace: ensure messages are \0 terminated

nettimeprovider: handle invalid network addresses earlier

Handle inet_aton() return code.

checks: add a comment to indicate that we intentionally leave out the 'break'

registry: remove dead code

The GError is only used for the mmap operations. If we have an error we handle
and clean it there already.

docs: small controller api docs improvement

typefind: canonicalize signal name

docs: mention have-type signal in the docs.

docs: minor gst-launch man page fix

Transmitter/receiver pipelines were mislabelled. Spotted by Majin.

multiqueue test: Remove workaround for pad_task hangs

Remove code that isn't needed any longer, which sets the multiqueue
to PLAYING and back before unreffing, in order to avoid a deadlock
waiting for gstpad tasks that were never started. The problem seems
to have been fixed long ago.

design docs: fix 2 typos in part-MT-refcounting

design docs: part-gstbin.txt fix typo

Signed-off-by: Yang Xichuan <xichuan.yang@tieto.com>

design docs: fix a few typos and a thinko

clock: API: Add function to re-init periodic GstClockIDs

gstpluginloader: do not leak the description string

The description string was changed to an inlined string a while back.
(But: no need to intern the const strings here, we just use the interning
to avoid allocating duplicates and make memory management easier,
since the strings will be around for the life-time of the app anyway).

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

buffer: clarify docs

design: update buffering doc

Add strategies to buffering doc

docs: add missing "Since: 0.10.32" markers for GstClock

Since tags were missing for gst_clock_single_shot_id_reinit()
and GST_CLOCK_DONE.

docs: Fix GstQueue2:ring-buffer-max-size property Since tag (0.10.31, not 0.10.30)

docs: add missing "Since: 0.10.31" marker for gst_structure_get_date_time()