bufferpool: Use TAG_MEMORY to check memory before releasing

Tag allocated buffers with TAG_MEMORY. When they are released later,
only add them back to the pool if the tag is still there and the memory
has not been changed, otherwise throw the buffer away.
Add unit test to check various scenarios.

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

buffer: add a new flag to track memory changes

Add a flag to check if the memory changed in a buffer.

buffer: remove wrong comment

Refcount and writability are not related for memory objects.

bufferpool: refactor free_buffer

Make a do_free_buffer method to also decrements the number of allocated
buffers. Stop will now be successful when all buffers are freed.

Automatic update of common submodule

From 1a07da9 to fe1672e

debugutils: add a legend to pipeline dumps

We use a couple of symbols to represent states/flags. Add a short explanation for them.

debugutils: Print if there is a task started from a pad

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

structure: Use get_uint64() in gst_structure_get_clock_time()

Its code is identical.

structure: Add getters for int64 and uint64 values

basesrc: Do not send eos when seeking after last buffer

If pushing the last buffer triggers a seek from downstream, do not
go into EOS if a new segment was requested.

Contains unit test

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

gst-uninstalled: remove insanity and the old gst-openmax

gst-uninstalled: Add paths to gst-devtools/validate

multiqueue: If we only have a single pad, don't consider all pads not linked and grow the queue indefinitely

check: also use the glob on GST_CHECKS when forcing to run broken tests

GST_CHECKS can be simply "test*" to run run all tests (including those that are
marked broken). Update the sparse comments a bit to tell how this works.

queue2: don't truncate the temp file on shutdown

We want to keep the downloaded file untruncated so that we can use it
again later.

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

queue2: Fix merging of ranges

Make a method to get the seeking threshold. If data is further away from
this threshold we want to perform a seek upstream.
When the current downloaded range can merge with the next range,
actually include the data of the next range into the current range
instead of discarding it. Also decide if we seek to the write position
of the merged range or continue reading.

basesrc: in automatic_eos mode, don't modify the size

Don't set the size to -1 in automatic_eos mode (which also updates the
duration to -1). We only want automatic_eos mode influence the maxsize
calculations without any side effects.

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

pwg: Update raw properties

Using info from gst-plugins-base/docs/design .

Encoded streams might make use of the raw properties, so list them all under foo/* .
For foo/raw, only note which of these properties are mandatory.

I didn't take a closer look at the raw formats yet. Those might still be out-of-date.

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

scripts: create-uninstalled-setup: remove dead http links

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

docs: add the boxed types to the .types.in

This makes them show up in the object hierarchy.

docs: gtkdoc is not good at parsing inline functions in headers

Mark the inline function, so that gtkdoc skips them. Avoids some warnings about
unparsable declarations.

basesrc: Add gst_base_src_set_automatic_eos() API

This defaults to TRUE and if it is set to FALSE it is the subclasses
responsibility to return GST_FLOW_EOS from the create() vmethod once
the stream is done.

docs: Fix location of plugins and registry in home directories

Fixes out of date documentation left over since 0.10.  In 1.0 plugins are
stored in $XDG_DATA_HOME and the registry in $XDG_CACHE_HOME conformant
with the XDG Base Directory Specification[1].

[1]: http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html

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

error: GST_RESOURCE_ERROR_NOT_AUTHORIZED will be available since 1.2.4

error: Add RESOURCE_NOT_AUTHORIZED error

This allows to distinguish normal read failures from read failures
where we miss authorization.

gst-inspect: Fix yet another compiler warning

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

docs: add missing seqnum file for distribution

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

docs: document GST_TAG_*ENCODING environment variables

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

docs: enable parallel build of subdirectories

We can build gtk docs, ADM, PWG and FAQ in parallel.

check: add support for blacklisting checks via GST_CHECKS_IGNORE

tests: fix leak in baseparse test

Or rather make it not show up any more by moving
it from 'definitely lost' into 'possibly lost'.

tests: fix leak in systemclock test

gst-inspect: Make clang happy with our g_vprintf() wrapper

windows: Make GStreamer installation relocatable

Use the technique that is now done in GTK+ so that the plugins do not have
to be installed in c:\gstreamer\lib\<debug>\gstreamer-$(GSTApiVersion),
but can be installed in
<parent_folder_of_gstreamer_main_dll>\lib\<debug>\gstreamer-$(GSTApiVersion),
or as per g_win32_get_package_installation_directory_of_module() allows.

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

gst: clear floating references for GstTask, GstTaskPool and GstCollectPads

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

docs: gst_pad_new_from_*_template and gst_buffer_pool_new constructors return floating references

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

check: Remove a minor leak in unit test

Makes valgrind happy

manual: Fix build by using the correct C file name

manual: Clean up code a bit to be suitable for the docs

manual: Replace manual's effectswitch.c with newer test-effect-switch.c

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

tests: add caps features unit tests

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

caps: When getting capsfeatures and none are there, store sysmem capsfeatures

... instead of returning a reference to a global instance. The caller might
want to change the global instance otherwise, which causes funny effects like
all global instances being changed and at the same time nothing in the caps
being changed.

As the caps might be immutable while we do this we have to do some magic
with atomic operations.

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

caps: Don't get us sysmem capsfeatures if we just check for fixed caps

capsfeatures: Make sure that the static ANY/EMPTY capsfeatures are never mutable

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

docs: fix more gtk-doc warnings

docs: unhide docs for allocator

Unhinde the class docs. Tech gtk-doc about GST_EXPORTS.

bufferpool: more tests and small doc fixes

tools: Support non-ASCII tags

By calling setlocale() to get us multi-byte/UTF-8 support.

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

outputselector: respect the 'negotiation-mode' property

If the segment event is allowed to be pushed to all pads it
will lead to an assertion of 'sticky event misordering:
segment received before caps' in case the pad-negotiation-mode
is set to 'active' or 'none'.

This patch fixes this by making all sticky events follow the
property like the caps event to prevent misordering warnings.

When a new pad is activated the current sticky events on the
sinkpad are forwarded to it in the proper order.

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

poolstress: print speedup

bufferpool: misc cleanups

Review the documentation, comments and logging. Set the initial size to 16 as
the size is rounded up to the next power of two anyway.

bufferpool: add a new testsuite or the pool

Start with some basic tests.

bufferpool: avoid excessive GstPoll activity

Keep an extra write ref on the control socket. This ensures that we
avoid a read/write on the socket when going from non-empty->empty->not-empty.
We remove the write ref only when we actually are empty and we need to
wait for flushing or a new buffer.

This makes the bufferpool benchmark about 30% faster than the pure
malloc implementation.

poll: improve debug

Add object pointer in debug lines.

Automatic update of common submodule

From d48bed3 to 1a07da9

part-toc: emphasize that the later chapters are design draft ideas

part-toc: add format specific information

poolstress: code cleanups

Add a few comments. Tell in the results, which number are from which test.

gst-uninstalled: use print as function

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

tests: capsfilter: add test for pending_events pushing

make sure that pending events are pushed when caps are already
set when a buffer is received

capsfilter: do not forget to push pending events

Push pending events before buffers if caps is already
set

multiqueue: do not reduce single queue below current level

When the single queue size was just bumped by 1 to allow more buffers to
be added, the buffers limit could be reduced to the current level when
setting the max-size-buffers property. This would result in a stall
since the queue would not grow anymore at this point.

Prevent this by not reducing a single queue size below the current
number of buffers + 1.

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

basesrc: do not forget to clear the forced_eos flag

otherwise it will always use the seqnum of the event
sent by the application

basesrc: preserve seqnum of eos events sent by the user

Store the eos event seqnum and use it when creating the
new eos event to be pushed downstream. To know if the eos
was caused by the eos events received on send_event, a
'forced_eos' flag is used to use the correct seqnum on
the event pushed downstream.

Useful if the application wants to check if the EOS message
was generated from its own pushed EOS or from another source
(stream really finished).

Also adds a test for this

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

docs: design: add part-seqnums

Hopefully clarifies how seqnums should be used and copied from
events to events/messages when those are handled.

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

multiqueue: Break the loop immediately if we found an empty queue

No need to continue looking at all the others

multiqueue: Allow growing a queue if all other queues are not linked

In the case where one singlequeue is full and all other are not linked, the
growing of the full queue does not work correctly. The result depends on if
the full queue is last in the queue list or not.

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

pad: Minor code cleanup

!check_sticky()==GST_FLOW_OK is a bit confusing, compared to
check_sticky()!=GST_FLOW_OK.

parse: Additional tests for parser

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

iterator: Properly copy mutexes around when creating a copy of a filter iterator

iterator: Add unit tests for filtering, recursive filtering and locking

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

iterator: Preserve the master lock when creating recursive iterator filters with the same lock

This way we make sure that a) the lock is always taken when checking
the cookie and calling the iterator's next functions and b) it is
not taken while calling any of the iterator filter functions.

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

pad: fix sticky event leak after sticky_events_foreach

events_foreach adds an extra ref when giving the event to the
user function. In case it was unrefed by the user, this extra ref
disappeared, but events_foreach still should unref again to
lose its own ref before removing the event from the array.

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

info: move some env-var checks from gst to gstinfo as well

We were doing some log related initialisation in gst.c after calling
_priv_gst_debug_init(). Just move it there for consistency.

capsfilter: Only set caps on the srcpad if it's activated in push mode

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

parse: Some minor fixes

Fix destructor segfaulting
Expect 0 grammar-ambiguities
Fix order of bin-properties assignment

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

inputselector: removing duplicate field position

It is already stored inside the GstSegment struct and
was only duplicating information. Also removed some
weird positon if/else that would possibly change the
segment that was going to be pushed downstream

multiqueue: prevent buffering forever with playbin

When prerolling/buffering, multiqueue has its buffers limit set
to 0, this means it can take an infinite amount of buffers.

When prerolling/buffering finishes, its limit is set back to 5, but
only if the current level is lower than 5. It should (almost) never be
and this will cause prerolling/buffering to need to wait to reach the
hard bytes and time limits, which are much higher.

This can lead to a very long startup time. This patch fixes this
by setting the single queues to the max(current, new_value) instead
of simply ignoring the new value and letting it as infinite(0)

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

segment: gst_segment_offset_running_time() will be available in 1.2.3

doc: Update sections with the new rounding macros

util: Add GST_ROUND_UP_N and GST_ROUND_DOWN_N

These are generic rounding macro that works for any power of two.

tee: Add unit test for requesting pad names

tee: Make sure to give pads the name that was requested

Also check for uniqueness and make sure we create a new
pad index properly if some were requested with names but
the new one is not.

parse: Make sure to create and link chains in the order as written

Make this work again:
audiotestsrc ! tee name=t  t.src_0 ! queue ! fakesink  t.src_1 ! queue ! fakesink
and this fail again:
audiotestsrc ! tee name=t  t.src_1 ! queue ! fakesink  t.src_0 ! queue ! fakesink

as tee just counts itself and does not care about the pad names we request
from it.

parse: Remove some C99-style comments

parse: Use GSlice for allocating and freeing links and chains

parse: Add comment about why we disable the "tracing"

It did not print anything useful before anyway, everything
was commented out.

Also remove some unneeded struct members.

parse-launch: Add some more failing pipelines

Also convert some comments about valgrind warnings to
FIXME comments. These were leaking since some time already.

parse: Refactor grammar, make it more consistent and fix conflicts

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

docs: ensure GstBufferPools shows up as with GObject features

GstBufferPool is a GstObject, add the _get_type function to the types file.

tests: baseparse: add test for reverse playback on passthrough

Baseparse stores buffers for reverse playback to push on the next
DISCONT, the issue was that it wouldn't ever check for a discont
on passthrough mode as it skips all real parsing. This test
was create to verify this issue and prevent it from happening again

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

tests: baseparse: add basic test for baseparse

Just a small test to check that basic playback works

baseparse: do not accumulate buffers on passthrough mode

If on passthrough during reverse playback, do not accumulate buffers as
baseparse will never check for DISCONT flag to push those buffers.
So just push buffers downstream as if it was forward playback.

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

basesrc: don't confuse GST_PAD_MODE_NONE and PULL

Use a switch-case to explicitly handle all pad-modes. This way we don't log an error when the pad is not yet activated.

gstobject: add FIXME and docs for the disabled notify on parent

We haven't found a way to re-enable emitting notify and deep-notify for parent
changes. Add a FIXME-2.0 and a doc blob on the property. See #693281.

baseparse: do not ignore TIME segments

TIME segments are being ignored and a standard initialized
segment is used instead. This causes issues as not properly detecting
reverse playback or not cliping output based on the segment.

This seems to be a regression from one of the GstSegment/GstEvent
redesigns on the 0.10 -> 1.0 transition

tests: improve check, also check stream-time

tests: add unit test for segment _offset_running_time()

Add a unit test to check that positive and negative offsets are applied
correctly in various cases.

segment: take offset into account in _to_position()

Take the offset into account when converting between running-time and
segment positions.

pad: use new segment offset method to apply the offset

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