bufferpool: fix max_buffers handling

When max_buffers > 0 and the pool is empty, actually try to allocate more
buffers up to the max_buffers limit.
We need to add a counter for this to count how many buffers we allocated and
check this against the max_buffers limit.
Reorganise and clean up some code.

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

netclientclock: simplify by using g_socket_condition_timed_wait()

No need to use a custom main context and custom timeout sources,
just use g_socket_condition_timed_wait() instead, which was added
for exactly this case.

Also seems to help with the unit test deadlocking with glib 2.33.x

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

gstobject: fix double string escaping in gst_object_default_deep_notify()

Make output of gst-launch -v readable again.

last-message = "event\ \ \ \*\*\*\*\*\*\*\ \(fakesink0:sink\)\ E\ \(type:\ tag\ \(20510\)\,\ GstTagList-stream\,\ taglist\=\(taglist\)\"taglist\\\,\\\ video-codec\\\=\\\(string\\\)H264\\\,\\\
 minimum-bitrate\\\=\\\(uint\\\)636611\\\,\\\ bitrate\\\=\\\(uint\\\)980729\\\,\\\ maximum-bitrate\\\=\\\(uint\\\)1116707\\\;\"\;\)\ 0x15bc760"

vs.

last-message = event   ******* (fakesink0:sink) E (type: tag (20510), GstTagList-stream, taglist=(taglist)"taglist\,\ video-codec\=\(string\)H264\,\ minimum-bitrate\=\(uint\)856039\,\ bitrate
\=\(uint\)1019748\,\ maximum-bitrate\=\(uint\)1116707\;";) 0x11149e0

miniobject: check writability

fix the writability check for miniobjects. We should check the shared counter.

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

allocator: Set the alignment at the correct place in GstAllocationParams

Back to development

Release 0.11.93

tests: remove silly test_fail_abstract_new check

Our check would make sure that GLib segfaults when
someone tries to instantiate an abstract type, which
is an extremely useful thing to check for.

In newer GLibs this is fixed and we get an abort with
a g_error() now it seems, so let's just remove this
check entirely.

examples: don't put things with side effects inside g_assert()

They will be defined away to NOOPs otherwise in release builds.

win32: update for stream-id API additions

parse: fix for new GstChildProxy::child-added signal callback signature

Fixes crash with gst-launch-1.0 uridecodebin uri=... suburi=... ! ..

bus: Add allow-none to the function argument of gst_bus_set_sync_handler()

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

docs: Make sure scanner gets required libraries

consistencychecker: print which event we received before stream-start

basesrc: don't try to answer URI queries with NULL URIs

Should make unit tests in -base that use appsrc a bit happier.

event: Update for stream-start event API changes

event: Add new stream-id field to the stream-start event

This is supposed to allow uniquely identifying a single stream.

inputselector: Use the first created pad by default

This guarantees a bit more consistency in which input stream will
be selected by default. It would previously be the first pad on which
an event/buffer/query was received ... which was racy and non-predictable.

element: Specify the order of pad iterators

The order of returned pads wasn't specified before, so let's specify
it and use an order which might prove the most useful : the order in
which pads were added to the element.

If someone changes the order, make sure users of those iterators from
now on don't rely on that order !

check: add tcase_skip_broken_test() define

Skips broken tests but logs an ERROR-level message to
draw attention to that fact.

tests: update .gitignore for queuearray test binary

tests: fix spurious netclientclock test failures

Give clocks a bit more time to synchronise.

win32: update generated files

input-selector: use generic marshaller for "block" action signal

Automatic update of common submodule

From 94ccf4c to 668acee

buffer, defaultmem: add option to poison memory before freeing it

Might be useful to track down certain bugs.

gst: ref/unref taglist scope enum in gst_init()

Fixes make check and distcheck

plugin: warn if plugin name starts with a "

This can easily happen as side-effect of the plugin name
in GST_PLUGIN_DEFINE no longer being a string in 0.11, but
a name to G_STRINGIFY.

docs: update porting-to-0.11 document with a "soft" API changes checklist

Point out some API changes that the compiler won't
be able to warn about.

tools: fix printing of partial dates in gst-launch

Revert "tools: print TOC scope"

This reverts commit ee6ab7c93638a6519acb976699a6ad149d520a95.

The application will probably only ever receive global TOCs,
so don't really need this.

win32: add new tag list scope symbols

typefind: send segment_done event in addition to segment_done message

basesrc: Add default handler for URI query in GstURIHandler subclasses

check: unbreak fail #define

The fail() definition was changed to not fail with non-GCC compilers,
unfortunately the change was incorrect and appended the first argument
of fail to the expression string instead of making it the message.

This change does mean that fail() now requires a message to be passed
along.

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

buffer: Update annotations

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

utils: Update annotation for get_compatible_pad

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

uri: Fix wrong 'array zero-terminated=1' annotation for strings

docs: update TOC design docs a little

event: make TOC event multi-sticky

We need to send two kinds of TOCs downstream as events,
and need both to stick to the pads.

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

tools: print TOC scope

toc: add GstTocScope and require it in the constructor

This is because we need to be able to signal different TOCs
to downstream elements such as muxers and the application,
and because we need to send both types as events (because
the sink should post the TOC messages for the app in the
end, just like tag messages are now posted by the sinks),
and hence need to make TOC events multi-sticky.

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

scripts: create-uninstalled-setup.sh: check for basic build tools and deps

.. before checking out stuff.

tag: Add a scope to taglists

This specifies if a given taglist applies to the complete
medium or only this specific stream. By default a taglist
has a stream scope.

Fixes bug #677619.

segment: add offset field

Add an offset field that is used to track at what position the segment was
updated. This is used to set the running time to 0 when we do a flushing
seek that doesn't update the position.

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

Update for new seeking variable name

When seeking, the start value and type are now called start and start_type.

segment: small cleanup

Move the code to update the segment at the end of the function.

Update docs and .def file for taglist API change

segment: remove redundant checks

We don't need to check the segment format anymore because we asserted on them
being equal before.

tests: improve segment tests

allocator: remove user_data from alloc vmethod

Remove the user_data from the alloc vmethod. Subclasses that implement a new
alloc function can also implement their own vmethod to pass extra arguments. We
can then also require that custom allocators implement an alloc function so that
gst_allocator_alloc() always works.

tests: remove segment accumulation checks

Remove the checks because there is no more segment accumulation.

taglist: make GST_TAG_APPLICATION_DATA also a GstSample

That way additional meta-data can be passed along with it.

taglist: gst_tag_list_get_buffer*() => gst_tag_list_get_sample*()

Image tags and other tags are now of GstSample type.

gst-launch: print image tags and other GstSample tags properly

These tags are now of type GstSample not GstBuffer.

collectpads: remove unimplemented api

We can always add this back if we need it. Fixes parts of #670852.

baseparse: also account for frame size when merely scanning for frame

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

baseparse: remove obsolete function parameter

typefind: require bytes before typefinding

Require that we have some bytes in the adapter before we attempt to typefind.

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

update spec file with latest changes

bufferlist: pass index as gint to _insert

Make the idx argument of _insert() a gint because we allow -1 as a value.
Improve annotation.

plugins: print flags better

print the buffer flags as a hex number so that it becomes easier to see what
flags are set.

gstpoll: Improve warning message when re-adding fd to fdset

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

Automatic update of common submodule

From 98e386f to 94ccf4c

value: add GstTagList serialisation/deserialisation

So we can serialise/deserialise taglists inside structures,
which used to work automagically before because GstTagList
was just a typedef to GstStructure (same for the GType),
but now that it's a separate GType we need to register
explicit functions for this.

Helps with GDP stuff in pipelines/streamheader tests.

po: Update .po files

tests: gstbuffer: add tests for some mulitple map combinations

miniobject: fix sharedness check

miniobject: refuse write when object is shared

In all cases, refuse to write an object when it is shared by more than one
object (also when the object was locked before).

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

tests: gstbuffer: extend buffer copy test

queue2: set buffering-left to 0 on 100% buffering

Set the buffering-left field in the query to 0 when we are completely buffered.
Improve the debug.

queue2: fix buffering query

Fix the buffering query, fill in the right buffering-left and estimated-total
values.

queue2: fix the buffering-left in the buffering message

The buffering-left field in the buffering message should contain a time estimate
in milliseconds about for long the buffering is going to take. We can calculate
this value when we do rate_estimates.

message: improve buffering message defaults

Remove the estimated-total field, this should not be part of the buffering
message.
Set the default value of buffering-left to 0 when the percent is 100.

pad: fix debug line

Use QUERY_TYPE on query types.

ghostpad: remove custom function

Remove custom pad functions, the default ones are better.

pad: add PROXY_SCHEDULING flag

Add a flag that makes the default query handler forward the scheduling query.

utils: fix docs

pad: improve query caps function

In the proxy_query_caps function, also filter against the filter in the query.
We don't need to filter against the filter in the query anymore in the default
caps query function because we already did this in the proxy_query_caps.

basesink: handle -1 step amounts

Define a 0 and -1 step amount. They used to almost do the same thing but now, 0
cancels/stops the current step and -1 keeps on stepping until the end of the
segment.

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

query: fix gst_query_parse_nth_allocation_pool() annotation

It returns a ref to the pool.

check: Avoid deadlock

Queries will be sent when pipeline goes down to NULL, which would
result in the probe being called ... but can't take the lock.

gstghostpad: Forward queries in both direction

Use the peer of the internal pad to forward them, instead of the
target which only exists for the ghostpad (and not the internal
proxy pad).

docs: More entries

queue: answer SCHEDULING query

Instead of letting the default query handler fail.

queue2: handle CAPS event and drop it if operating in ring buffer mode

Fixes "Unexpected event of kind caps can't be added in temp file"
warning when doing download buffering.

buffer: make _foreach_meta more powerful

Make _foreach_meta return FALSE when the foreach function returned FALSE.

bufferlist: improve foreach function

Make the foreach function return FALSE when one of the function calls returned
FALSE.

buffer: add more debug

basesink: fix debug string

parse: fix some debug

parse: only escape spaces outside of quotes

When we escape spaces to keep arguments together, only escape when the space is
outside a "" string.

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

Revert "parse: escape \ with a \ as well, so that we don't lose the \ when unescaping"

This reverts commit dd9fedb41f1ada8e1f8bd5346fccd3d068d543cb.

This is not the right place to escape the \, we should only escape the spaces to
keep the arguments together that were provided as one group (with quotes on the
shell).

utils: set return type of gst_parse_bin_* to GstBin for introspection

nettimepacket: add missing array annotation to gst_net_time_packet_new

introspection: add missing array annotation to gst_formats_contains

bin: aggregate durations like in adder

Stop querying the duration once an element return unknown and return unknown
as a final result. This avoid eventually cutting off a stream too early.
Add a tests to docuement the behavior.

datetime: just return NULL on short input strings instead of a warning

We want to be able to use this function on random non-NULL input,
this should not result in a runtime-critical.

baseparse: fix seekability querying with formats with headers like FLAC

Move code that checks for upstream seekability and all that to
the right place, otherwise it will never be done for formats
that have headers such as FLAC, as handle_and_push frame will
be called the first time only after headers have been processed
(and framecount is > 0). This then makes us report that we
can't seek, which disables the seek bar in totem.

plugins: embed GstAueueArray in dataqueue struct as well