tests: multiqueue: fix eos count on test for not-linked case

From the test case:

/* This test creates a multiqueue with 2 streams. One receives
 * a constant flow of buffers, the other only gets one buffer, and then
 * new-segment events, and returns not-linked. The multiqueue should not fill.
 */

If one of the queues goes EOS and the other returns NOT_LINKED the stream
can be considerered EOS as a NOT_LINKED means that one of the branches has no
sink downstream that will block the EOS message posting.

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

adapter: Adapt gst_adapter_copy() for bindings

This is done by introducing a new gst_adapter_copy_bytes() call that
returns a GBytes structure.

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

docs: fix multiqueue docs for new template names foo_%d -> foo_%u

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

docs: update plugin docs

pad: simplify gst_pad_link_get_name() and fix Since marker

Has added benefit that compiler might warn if more values
are added to the enum.

pad: add debug helper for GstPadLinkReturn names

Add a helper like gst_flow_get_name() for GstPadLinkReturn. Use this in core.
API: gst_pad_link_get_name()

tee: use store_sticky events add add more logging

Use the pad as object for logging to get more context. Use
gst_pad_store_sticky_event() instead of sending the event. This avoids a warning
as here the pad is not yet linked and we actually don't want to send anyway.

queue2: if buffering is disabled while buffering, post 100% message

Avoids stall waiting for buffering to reach 100%

queue2: remove unused variable

buffering_iteration was never used

queue: queue2: preserve last flow result when pushing events

Avoids mistakenly returning _OK when downstream is still
_NOT_LINKED on subsequent received pad pushes

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

multiqueue: if buffering is disabled while buffering, post 100% message

Avoids stall waiting for buffering to reach 100%

multiqueue: do not reset last push result when pushing events

Use the last result as a default when pushing a item from a single queue,
otherwise the status gets reset to _OK when pushing events.

This causes problems when mistakenly activating a not-linked stream
that is being ignored upstream as it is not being used (adaptive
scenarios), it will make the multiqueue post a buffering message
on a pad that won't receive buffers

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

buffer: invert meaning of GST_BUFFER_FLAG_TAG_MEMORY

It's nicer to only have it set when something noteworthy
happened and otherwise unset.

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

pad: fix gst_pad_add_probe() return value docs

Also fix comment typos and add more detail in the logs.

fakesink, identity, fakesrc: fix debug printing if TAG_MEMORY buffer flag

The IN_CAPS flag does not exist any more.

ghostpad: use gst_pad_get_peer to acquire a reference to the target pad

This ensures that the lock of the internal pad is held while referencing
it's peer (= the target pad), which ensures that the peer is not
going to be unlinked/destroyed in the meantime.

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

ghostpad: hold a reference to the target pad while unlinking it

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

.gitignore: Ignore gcov intermediate files

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

docs: Fix typos and remove unknown annotations

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

docs: use the new markdown for ordered list.

This was plain text that had all list items one after the other (including a
repeated number). Now it will atleast look good when processed with gtk-doc
1.20.

Automatic update of common submodule

From fe1672e to bcb1518

docs: fix problems introduced by c068b225fef5a9bf0

- Fix failing build
- Drop added trailing whitespace

buffer: add return values to g_return_if_fail

FIxes previous commit.

docs: convert the examples to use gtk-doc markup, instead of docbook

The gtk-doc markup is less intrusive and better handled when creating docs for
language bindings. The titles (where used) where not adding much.

docs: add some more new API do docs

bufferpool: only release buffers with writable memory

Check if the memory is writable before releasing the buffer into the
pool again.
Add unit test for this scenario.

buffer: add function to check writability of memory

Check if memory is writable in a buffer and thus is exclusively owned by
this buffer.

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