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

segment: add method to offset the segment running-time

Add a method that can apply an offset to the calculated running-time of
a segment.

collectpads: take offset into account for expected segment position

The firt valid segment position is start + offset.
Also add some more debug and a FIXME

info: debug segment offset field as well

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

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

filesrc: don't try to seek to -1 offset

The offset can be -1 when we are configured in TIME format. Instead of
failing the seek and erroring, do what and offset of -1 is supposed to
do and simply read from the current offset.

basesrc: demote error to warning

This is not an error. A subclass returning FALSE for is_seekable() is one way of
saying that we can't operate in pull mode.

multiqueue: Fix hanging if shut down while handling a serialized query

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

baseparse: remove pending_segment as it was being misused

It wasn't required, instead baseparse was using it to check the media
caps to identify if it was handling audio or video.

The pending_segment was removed and a checked_media boolean
replaced it for a more accurate naming.

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

baseparse: push pending events before GAP event

A GAP event is handled as an empty buffer by sinks and they expect
to receive start up events before GAP events (like a segment).

This is important specially if there is a GAP at the beginning of
a stream (before any buffers) so that the segment event can be
pushed downstream before the GAP

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

baseparse: refactor pending events pushing

Refactor code repeated 3 times to a common function

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

design/part-toc.txt: update design docs

pad: Add unit test for adding/removing blocking probes while a pad is blocked

And make sure that these new probes are actually called if they should
instead of silently blocking the pad forever.

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

pad: Check if new probes need to be called when adding/removing some

This allows blocking a pad, add a new blocking probe, removing
the first probe and then having the second probe called. Which
could then decide that data-flow should actually continue
instead of blocking now.

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

tee: Remove dyn lock

It was used for pad-alloc in 0.10 but currently is completely unused
and not necessary. All pad access is protected by the tee object lock
and keeping another reference to the current pad.

tee: Keep another ref to our one and only srcpad around while pushing

A pad probe on that pad might otherwise just release the pad, drop
the last reference and cause great misery.

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

pad: Keep an extra ref of the pad when calling an IDLE probe immediately

The callback might destroy the pad.

pad: Only call IDLE probes if we are actually idle

Also only check the data types for non-IDLE probes. When we
are idle, we have no data type obviously.

Previously we were calling IDLE probes during data flow whenever
a non-blocking probe would be called. The pad was usually not idle
at that time.

nettimeprovider: remove unnecessary NULL check

Error is never NULL when we break out of the loop.

COVERITY CID 1037151

Revert "nettimeprovider: Remove dead code"

This reverts commit 9649cd4ca19a75d0319117a77a7609e4b8c96533.

This is not right, and it's also not what coverity
is complaining about.

queue: Remove unneeded checks

item is guaranteed to be non-null.

COVERITY CID 1037152
COVERITY CID 1037153

nettimeprovider: Remove dead code

err is always NULL by the point we reach this line

COVERITY CID 1037151

taglist: fix since marker again (1.3 -> 1.4)

Since markers should point to the appropriate stable version.

taglist: fix since-marker in docs (we're in 1.3.X)

taglist: add a tag for midi base note numbers

Audio files containing sampled instruments can have metadata describing the note
that was played on the instrument.

pad: Don't ignore probe callback return value when immediately calling IDLE probe

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

scripts: git-update.sh: fix for non-master branches

Pull from tracking branch instead of origin/master, so
that this works with e.g. 1.2 as well.

parse: Use GLib malloc/free/realloc functions

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

scripts: create-uninstalled-setup: re-use existing master branch if it exists

When creating separate checkout for non-master branches.

docs: fix memory leak of appsink example in manual

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

Automatic update of common submodule

From dbedaa0 to d48bed3

po: update for string changes

po: set gettext domain in Makevars so we don't have to patch the generated Makefile.in.in

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

utils: Add round down 128 macro for completeness

utils: Add round up 128 macro

funnel: Setting the PROXY_CAPS flag on the srcpad does not make much sense

funnel outputs whatever one of the upstreams currently outputs, a caps
query to a random upstream does not give the right answer here.

funnel: Proxy CAPS and ALLOCATION queries

multiqueue: post 100% buffering if single queue is not linked

This makes buffering stop in case a stream switch happens. This is
important for adaptive streams that can disable not-linked streams
to avoid consuming the network bandwidth.

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

docs: fix project links

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

basesrc: use segment start if DTS for first buffer is unset

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

gst-uninstalled: add gstreamer-vaapi paths

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

tests: add unit test for bitreader, bytereader and bytewriter with no inlining used

bytereader: add inline variant of gst_byte_reader_init()

docs: fix docs for gst_queue_array_peek_head()

parse: Don't define yyscan_t twice

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

docs: moving plugins: minor 0.10 -> 1.0 fix

Spotted by Jay Fenlason

queue: don't ignore event return value

Pass the event return value upstream.
Remove strange goto construct.

gst-launch: Handle taglist copy failure

If we couldn't copy the tags, just return instead of trying to use bogus
values.

gst-inspect: Index features are no more

So remove code that will never be used

gstvalue: Fix comparision of double range

Checking twice the lower bound is great (you never know, it might change
between the two calls by someone using emacs butterfly-mode), but it's a bit
more useful to check the higher bound are also identical.

Detected by Coverity

controller: Fix out-of-bounds detection

We want to abort if we higher than the maximum *OR* lower than the minimum
accepted value.

Detected by Coverity.

tests: add unit test for registering the same category twice

info: return existing category if a debug category is registered twice

If a category with the same name is found when creating a new
one, the found category is returned instead of an invalid pointer.
Fixes issue with gst-vaapi (which uses an internal copy of the
codec parsers) caused by commit ccba9130.

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

docs: Fix typos in function/object descriptions

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

Fix some typos in code comments and debug messages

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

docs: clarify encoding of strings in GstStructures and taglists

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

basesrc: Set format to TIME if do-timestamp is TRUE

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

tools: gst-launch: don't try to remove already-removed GSource from main loop

It's considered a programming error in recent GLib versions now.
We may already have removed the source by returning FALSE from
the callback if it was fired. Fixes warning with newer GLibs
when interrupting a pipeline with Control-C.

info: Make sure the same category is not added twice