Merge remote-tracking branch 'origin/master' into 0.11

warnings: avoid set-but-unused warnings with load-save disabled

baseparse: fix broken default caps query

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

Merge remote-tracking branch 'origin/master' into 0.11

uri: fix wrong G_GNUC_MALLOC

_get_protocols() points to const memory in 0.10
despite the non-const return value.

Merge remote-tracking branch 'origin/master' into 0.11

Conflicts:
gst/gstbuffer.h
gst/gstbufferlist.h
gst/gstcaps.h
gst/gstdatetime.h
gst/gstelementfactory.h
gst/gstevent.h
gst/gstghostpad.h
gst/gstindexfactory.h
gst/gstiterator.h
gst/gstmessage.h
gst/gstminiobject.h
gst/gstpipeline.h
gst/gstquery.h
gst/gstsegment.h
gst/gststructure.h
gst/gsttaglist.h
gst/gsturi.h
gst/gstvalue.h
libs/gst/base/gstbitreader.h
libs/gst/base/gstbytereader.h
libs/gst/base/gstbytewriter.h

Note: can't use G_GNUC_MALLOC with GstCaps return
values in 0.11 because of the EMPTY+ANY singletons.

libs: sprinkle some G_GNUC_MALLOC

Maybe gcc can do something clever with that, or at least
warn us if we don't save the return value somewhere.

gst: sprinkle some G_GNUC_MALLOC

Maybe gcc can do something clever with that, or at least
warn us if we don't save the return value somewhere.

Merge remote-tracking branch 'origin/master' into 0.11

docs: update porting docs.

buffer: add gst_buffer_{set,get}_qdata()

Allows people/us to attach arbitrary metadata to buffers.

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

API: gst_buffer_set_qdata()
API: get_buffer_get_qdata()

pad: fix blocking probe emission

If we are dealing with a blocking probe, only then check if one the
blocking flags of the hook matches.
Add some more debug.
Make the pad unit test less racy.

Add test for PAD_PROBE_TYPE_BLOCK and PAD_PROBE_TYPE_BLOCKING

utils: fix debug of query result

query: fix typo in doc

Causes a warning from the introspection scanner.

check: drop caps argument from gst_check_setup_{src,sink}_pad

Calling set_caps at that point is not useful in 0.10 (FIXME comment!), and in
0.11 it is totally pointless: the caps event doesn't stick to a flushing pad.

defs: update for byte_writer_put_buffer

Merge branch 'master' into 0.11

Conflicts:
gst/gstpad.c
libs/gst/base/gstbaseparse.c

pad: Remove g_warning() if pad accepted caps that are not a subset of the pad caps

This check is correct but unfortunately it's impossible to implement
in a threadsafe way because the caps could have changed in the meantime.

Fixes bug #659606.

multiqueue: check filled state of queues even if another one is empty

This will avoid a case where overrun is never signalled if some
stream never produces any data, causing playbin2 to not end preroll.

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

tests: Check for dataflow with incompatible caps

This test currently fails, but is there to ensure we fix this issue
and keep it fixed, since it completely breaks delayed negotiation
use-cases.

This behaviour started breaking since
dd65aae9a177f7b11dcef0f690a78d698f667cd4

tests: Add comments to gstpad tests

tests: Remove "#if 0" block for behaviour that now works

tests: Remove commented block

This behaviour is actually tested in test_push_unlinked

tools: make gst-inspect print a nasty debug message for non-grata property type

Try to eradicate properties of long/ulong/char/uchar type.

event: add STICKY_MULTY events

Add a new event flag for sticky events so that multiple events of that type can
be stored on a pad at the same time. Change the _get_sticky_event() function to
loop over the multiple events of a type.
Change the foreach function to make it possible to removed and modify the sticky
events on a pad.
Use an variable size array now to store the events. This could later be
optimized some more.

bus: handle NULL message src in debug

bufferlist: avoid reading past the array

When the foreach function told us to remove the buffer from the list, decrease
the length of the array or else we might read past the last item in the array.

baseparse: Return template caps instead of other side's peer caps if get_sink_caps vfunc is not implemented

Using gst_pad_proxy_get_caps() breaks backwards compatibility with old
parsers because it will propagate the other side's fields like "parsed"
and "framed" and also breaks parser/converters.

Fixes bug #664221.

bytewriter: add method to write out a buffer

In 0.10, this can be done with a one-liner by using GST_BUFFER_DATA/SIZE with
put_data. A 0.11 user has to resort to gst_buffer_map, which is less convenient
and might require a memcpy internally.

API: gst_byte_writer_put_buffer()

pad: take peerpad correctly

Don't take the peerpad too early, it might change because of the
probes.

pad: rework sticky events

Rewrite sticky events, trying to make it a bit more simple.
When sticky events are pushed on a srcpad, store them in the sticky event
array and mark the event with received = FALSE.
When the sticky event is successfully sent to the peer pad, make
received = TRUE.
Keep a PENDING_EVENTS pad flag that is set when one of the events is in
the received = FALSE state for some reason.
when activating a sinkpad, mark all events received = FALSE on the peer
srcpad.
When pushing a buffer, check the PENDING_EVENTS flag and if it is set, push all
events to the peer pad first.

baseparse: also let caps events go through

gst: add new flags

adapter: fix return type of _map() to gconstpointer

Fixes compiler warnings on OSX:
gstadapter.h:82: warning: type qualifiers ignored on function return type
gstadapter.c:412: warning: type qualifiers ignored on function return type

const gpointer is not the same as gconstpointer or const void *.

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

defs: update defs files

padd return value from _mini_object_replace()

Fix printf format compiler warnings on OSX/64bit

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

tests: improve pad tests

pad: fix locking order error

ghostpad: fix print format

pad: Add destroy notify to pad functions

Add _full variants of the pad function setters that take a destroy notify.
Make some macros that make the old method name pass NULL to this new
function.

pad: Merge pad mode activation functions

Add the pad mode to the activate function so that we can reuse the same function
for all activation modes. This makes the core logic smaller and allows for some
elements to make their activation code easier. It would allow us to add more
scheduling modes later without having to add more activate functions.

docs: update design doc

also fix default alignment value (0 == no-alignment)

query: improve scheduling query

Turns some boolean arguments in the scheduling query to flags, which are easier
to extend and makes the code easier to read.
Make extra methods for configuring and querying the supported scheduling modes.
This should make it easier to add new modes later.

query: move flags closer to buffering query

pad: add parent to activate functions

pad: fix scheduling mode enums

GstPadActivateMode -> GstPadMode
GST_PAD_ACTIVATE_* -> GST_PAD_MODE_*

typefind: fix for new getrange method signature

gst_type_find_helper_get_range_ext -> gst_type_find_helper_get_range

pad: add parent to other functions

Add parent to chain, chain_list, getrange and event functions.

collectpads: move fields out of reserved and restore padding

Do the 0.11 ABI changes. Add extra fields for destroy_notify and drop the qdata
hack. Rename _add_pad_full to _add_pad and remove the old _add_pad.

add parent to internal links

pad: add parent to the query function

GstPadFlags: rename flags GST_PAD_* -> GST_PAD_FLAG_*

plugins: remove obsolete parent checks

pad: keep the parent alive when requested

Add a new pad flag NEED_PARENT that ensures that the parent of a pad is
reffed and not NULL when the event, query and internal links functions
are called.
When a pad is added to an element automatically make sure the NEED_PARENT flag
is enabled.

don't require parent element to proxy

update porting doc

pad: move query convenience functions together

Move the caps convenience functions to the other query functions.

_query_peer_*() -> _peer_query_*()

_accept_caps() -> _query_accept_caps()

_peer_get_caps() -> peer_query_caps()

pad: _get_caps() -> _query_caps()

utils: fix docs

utils: fix the proxy functions

fix the proxy functions for query_accept_caps and query_caps to use the pad
forward helper functions which correctly forwards on the internally linked pads.

pad: improve some flag macros

pad: fix debug line

pad: remove getcaps and use caps query

Remove the getcaps function on the pad and use the CAPS query for
the same effect.
Add PROXY_CAPS to the pad flags. This instructs the default caps event and query
handlers to pass on the CAPS related queries and events. This simplifies a lot
of elements that passtrough caps negotiation.
Make two utility functions to proxy caps queries and aggregate the result. Needs
to use the pad forward function instead later.
Make the _query_peer_ utility functions use the gst_pad_peer_query() function to
make sure the probes are emited properly.

query: add caps query

query: remove GST_QUERY_LAST

fix docs

controller: remove functions to add/remove controlled properties

Make that implizit with attaching/detaching controlsources. This is a lot easier
and has less invalid state (controlled property without control source).

plugins, tools: update for get_protocols() return value change

uri: clean up header files

Tabs to spaces.

urihandler: fix return type of get_protocols()

urihandler: fix return type of _get_uri_type()

Return a GstURIType and not a plain guint.

tests: update unit tests for URI handler API changes

elements: update fd + file sources and sinks for GstUriHandler changes

urihandler: pass GError argument to gst_uri_handler_set_uri()

Also let gst_uri_handler_set_uri check already if the protocol
is supported, so that not every uri handler has to do that
itself.

urihandler: make _get_uri() return a copy

For thread-safety.

urihandler: remove "new-uri" signal

No one but filesrc used that API. Should probably be replaced by
requiring an "uri" property instead, and then objects can do a
notify on that. Also removed interface structure padding, it's
not needed.

Merge remote-tracking branch 'origin/master' into 0.11

Conflicts:
tools/gst-inspect.c

gst-inspect: print current value as default value

Instead of printing separate 'Current' and 'Default' values
(the former obtained via g_object_get() and the latter from
the property GParamSpec), simply print the Current value as
the Default value. This is the right thing to do for almost
all elements and avoids confusion if a subclass of a base
class chooses a different default than the base class.

gst, controller: replace g_list_prepend + reverse with GQueue

docs: fix typo in buffer docs

basetransform: fix caps unref in transform_caps filter subset check

I did not test this, but the code looked very wrong.

query: do not return a ref from parse_accept_caps

Makes this exactly like gst_event_parse_caps. This is what current code
expects, so it fixes some leaks.

.h: fix header files

Ensure correct indentation and retab
Make sure all structure have padding

.h: fix header files

Ensure correct indentation and :retab.
Make sure all structures have padding
Fix up some old ABI additions.

docs: fix invalid xml

buffer: avoid < -1 sizes

buffer, memory: make size arguments where -1 is allowed signed

win32: update .def file for latest API changes

controller: merge controller into gstobject

This make the controller even more lightweight (no extra object, no extra lock,
less indirections). For object that don't use the controller the only 'overhead'
is a 3 unused fields in the gst_object structure.

netmeta: avoid using g[u]long in headers

adapter: remove flush from _unmap

basesrc: implement a default get_caps function

Don't rely on the return value of a vmethod to trigger the default
implementation but make a real defaul implementation of the method that the
subclass can chain up to.

gstbuffer: remove incorrect assertion

Offset and sizes have no bearing on each other here.

win32: Update def files

gst: More introspection annotations