caps: ensure writable caps prior to modification

bufferpool: check min/max_buffers

query:fix copy-and-paste problem

Replace master with 0.11

utils: add and improve debug messages

... so they end up in a more expected debug category rather than oblivion.

back to devel

Release 0.11.3

basetransform: remove automatic and undocumented setting of always_in_place

... which controls how to (forcibly) deal with (non-)writable data and
is not necessarily related to identical caps.

In particular, it is also not so helpful anymore with a more advanced
GstVideoFilter subclass which always has a transform_ip method currently,
even though its subclass may not have a corresponding _ip method.

Merge branch 'master' into 0.11

Conflicts:
configure.ac

dist net directory only once

query: Only allow fixed caps in the accept-caps query

baseparse: do queries more directly

Just call our internal query function instead of going through the pad and the
query handler etc.

adapter: add some performance debug

pad: improve docs of get/pull_range

Improve the docs of the get/pull_range functions, define the lifetime of the
buffer in case of errors and short reads.
Make sure the code does what the docs say.

buffer: improve gst_buffer_new_wrapped_full()

Make it possible to wrap all kinds of memory by exposing all properties to
gst_buffer_new_wrapped_full(). This makes it possible to also create writable
memory without a free function or memory with extra padding.

multiqueue: handle serialized queries

basetransform: make more stuff private

basetransform: small cleanups

pad: change the semantics of get/pull_range a little

Make it so that one can specify a buffer for get/pull_range where the downstream
element should write into. When passing NULL, upstream should allocate a buffer,
like in 0.10.
We also need to change the probes a little because before the pull probe, there
could already be a buffer passed. This then allows us to use the same PROBE
macro for before and after pulling.
While we're at the probes, make the query probe more powerful by handling the
GST_PAD_PROBE_DROP return value. Returning _DROP from a query probe will now
return TRUE upstream and will not forward the probe to the peer or handler.
Also handle _DROP for get/pull_range properly by not dispatching to the
peer/handler or by generating EOS when the probe returns DROP and no buffer.
Make filesrc handle the non-NULL buffer passed in the get_range function and
skip the allocation in that case, writing directly into the downstream provided
buffer.
Update tests because now we need to make sure to not pass a random value in the
buffer pointer to get/pull_range

typefind: proxy allocation query

event: fix docs a little, alloc_buffer is gone

bufferpool: split bufferpool configuration

Make separate methods to control the bufferpool and the allocator used by the
bufferpool.
Make it possible to change the allocator of a pool.

query: rework the ALLOCATION query

Separate the bufferpool and allocator hints in the allocation query, some
of the values don't always make sense together.
Keep the bufferpool and its configuration together.
Keep the allocator and its parameters together.
Allow for multiple bufferpool configurations in the query.

pad: comment and debug improvement

utils: improve debug

also fix a potential memory leak

defs: update

GstBufferPoolParams -> GstBufferPoolAcquireParams

Because those flags are not from the bufferpool but for the acquire function.

memory: group allocation parameters in a struct

Group the extra allocation parameters in a GstAllocationParams structure to make
it easier to deal with them and so that we can extend them later if needed.
Make gst_buffer_new_allocate() take the GstAllocationParams for added
functionality.
Add boxed type for GstAllocationParams.

filesrc: only update buffer size on short read

query: fix copy function

Copy the structure too.

memory: fix maxsize after align

when we align the data pointer, make sure to update the maxsize.
Add some more debug

queue: remove useless PROXY_ALLOCATION flag

memory: Add 0 padding

Change gst_allocator_alloc() so that we can also spicify flags and padding.
Add 2 new flags to mark the memory 0 prefixed/padded. This allows us to
remove some resizes in the base classes.
When allocating memory, memset prefix and padding with 0 when the flags tell
us to.
On resize, clear the zero padding flags if we can't guarantee the memory is
still 0 filled.
Update tests.

query: also include padding in ALLOCATION query

Negotiating padding is needed on second thought so include it in the
ALLOCATION query.
Make the bufferpool take padding into account when allocating.
Make basesrc take padding into account.
Use padding and prefix when allocating in basetransform.

basesrc: take prefix into account when allocating

Take into account the prefix that we received from the allocation query and use
it to allocate and resize a larger buffer.

bufferpool: free owned discarded pool config

pad: implement DRAIN handling

When we forward the DRAIN query and there is nothing to forward it to, assume we
are drained.
When a basesink receives a drain query, reply with TRUE.

query: add new drain query

With the new serialized downstream queries we can implement a drain query that
makes an element waits until a downstream element replies to the query.

pad: make serialized queries push sticky events first

Before we can proceed with a serialized query, we need to be sure that all
sticky events were pushed.

queues: warn when receiving a serialized event

.. until we implement it.

queue: add support for serialized queries

pad: take stream lock on serialized queries

pad: enforce correct query direction

query: register queries like events

Also register queries with a QueryType that allows us to check if the event is
sent in the right direction. Add a serialized query type because we will need
this for the allocation query.
Remove the QueryTypeDefinition stuff, it is not used anymore and we now use
custom queries and separate API for them.
Update defs.

adapter: add more debug

bin: remove old compat mode

caps: small docs update

caps: remove gst_caps_union()

Remove gst_caps_union(), use gst_caps_merge(). This function was not used
anymore and it is unclear what the difference is with _merge().

configure: bump AS_LIBTOOL version

API was added to collectpads2

configure: backport AS_LIBTOOL version from 0.10.36 release

Might fix issues with missing symbols for people who install GStreamer
from source and at some point jumped back and forth between git master
and the 0.10.36 release (or 0.10. branch).

docs: fix function name and typo

basetransform: get template caps only once

Get the template caps of the pads only once, avoids unecessary ref
and unrefs.

caps: delay _make_writable() until needed in _normalize()

Delay _make_writable() until we actually found a list and need to update the
caps.

caps: shortcut simplify earlier

A simple caps is already simplified, no need to check for fixedness.

caps: small cleanup, remove const

caps: small cleanups

caps: small doc improvement

configure.ac: bump required GLib to 2.31.14

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

docs: update porting-to-0.11.txt a little

caps: fix some 0.11 FIXMEs

caps: make _normalize take ownership of input

Make gst_caps_normalize() take ownership of the input so that it can more
intelligently decide when to copy or not.

caps: _do_simplify() -> _simplify()

Rename _do_simplify() to _simplify(). The name was introduced as a replacement
method for a deprecated method but we can now rename it again.
Fix some docs.

caps: improve _do_simplify

Make gst_caps_do_simplify() take ownership of the input caps and produce a
simplified output caps. This removes the requirement of having writable input
caps and the method can make the caps writable only when needed.

tests: fix unit test

with the new caps API, there is more sharing and less copying going on so the
unit test refcounts are different.

update license link to point to 2.1 license, instead of just generic lgpl file, which is always the latest and thus not correct for gstreamer

caps: avoid using in-place oprations

Rework some caps operations so they don't rely on writable caps but instead take
ownership of the input caps and do _make_writable() only when needed.
Remove some const from caps functions, it does not make much sense for
refcounted objects and does not allow us to return a refcount to the const input
caps.
Rework the base classes fixate vmethods to not operate on the caps in-place.
All this saves us around 30% of caps and structure copy and new operations.

structure: add allocation debug

typefind: remove const from refcounted GstCaps

Having const on refcounted objects require us to make copies instead of simply
taking a ref, don't do that.

registry: avoid copy when caps are fixed

Avoid doing a useless copy when the caps are fixed and simplify will not do
anything.

buffer: small optimizations

shortcut heavy work when buffer_resize does nothing.
Avoid an extra _ref when mapping a buffer.
Add some G_LIKELY.

bufferpool: fix array types

docs: update docs

pad: also push sticky events on new event

Make a helper function check_sticky to check and push pending sticky events.
Move the handling of the result of pushing the sticky event inside the
push_event function, we need to mark the event as received when it was pushed
correctly.
Move the sticky events code outside of gst_pad_push_event_unchecked and
make it purely handle sending the event to the peer.
when pushing a sticky event, first store it on the pad. Then check and push any
pending sticky events when we get a serialized or sticky event on a srcpad. This
fixes the issue where sticky events are not pushed when an event is pushed.

pad: store the received result from _foreach

If the foreach function changes the received state of the sticky event, make
sure we remember that.

pad: add comment

test: add test to check sticky events order

Sticky events pushed on an unlinked pad should be stored on the pad. When the
pad is then linked and an event is pushed, the event should be merged with the
already existing sticky events and then the sticky events should be pushed in
the order that they were originally pushed.

test: fix typo in comment

tests: port pipeline/seek test to 0.11

Doesn't fail in 0.11 of course, at least not on my machine.

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

Conflicts:
common
gst/gstpad.h
gst/gsttask.c
libs/gst/base/gstcollectpads2.h

pad, task: improve debug logging

pads, collectpads2: get rid of superfluous brackets around static rec mutex calls

Makes it possible to define those calls to something for tracing.

common: update common module

For make foo/bar.check-norepeat target.

tests: add minimal basesrc ! sink seeking unit test

Should reproduce 'GStreamer-WARNING **: wrong STREAM_LOCK count 0'
warnings (with make pipelines/seek.torture or pipelines/seek.forever
anyway, since it appears to be racy).

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

value: remove gst_value_register_{subtract,union,intersect}_func() API

There isn't really any need to provide public API for that. It's not
used anywhere in practice, and we aim to provide an API that works
for GstCaps, not some kind of generic set manipulation API based on
GValue. Making this private also makes it easier to optimise this
later. We can always put it back if someone actually needs it.

tee: fix refcount error

pad: return ANY for a pad without template

Because gst_pad_get_pad_template_caps() returns ANY when there is no template,
the query caps function should also return ANY when there is no template (and no
pad current caps) instead of EMPTY.

pad: small cleanup

memory: add comment

baseparse: Fix merge mistake

Merge branch 'master' into 0.11

Conflicts:
libs/gst/base/gstbaseparse.c
libs/gst/base/gstbasetransform.c
plugins/elements/gsttee.c

baseparse: arrange for properly disjoint frame flags

basetransform: delay pool activation

Delay the activation of the bufferpool until we actually need a buffer from the
pool.

baseparse: Fix 'self-comparison always evaluates to true'

This was really a bug.

tee: Fix 'use of logical '&&' with constant operand' compiler warning

This is actually a real bug.

basetransform: Fix 'equality comparison with extraneous parentheses' compiler warning

gst: Fix 'comparison of unsigned enum expression >= 0 is always true' compiler warning

basetransform: don't propose_allocation before negotiation

Answer the allocation query with FALSE when we are not negotiated yet because at
that point we have no idea if we need to proxy the allocation query or not.

baseparse: Fix handling of multiple newsegment events

Previously only the last would be pushed, which would cause
invalid running times downstream. This also fixes the handling
of update newsegment events.

baseparse: Also flush the close_segment

Pushing this after flushing will confuse downstream.

baseparse: Remove obsolete code and move gap handling to the correct place

The segment start adjustment code in pull mode should never trigger
anymore because the bisection code earlier would have already made
sure that we're at the desired position.

Also move the gap handling some lines below after sending the currently
configured segments. Otherwise we might fill gaps in a segment that is
not configured downstream yet.