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

info: Protect __categories list in get_category with lock too

docs: add missing files for distribution

 * add some documentation files in docs/design
 * add docs/list-ulink.xsl so check in docs/manual works

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

pad: add ACCEPT_INTERCEPT flag

Make a new flag on the pad that tweaks the default behaviour of the
accept-caps function. By default it will check for a subset of the
query-caps result but this is not always desirable. The query-caps
result contains all the constraints to make a good caps decision
upstream but sometimes, like for parsers, not all the constrained caps
fields are known upstream and then a subset check would fail. Switching
to an intersection makes this work again.

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

multiqueue: Wake up on reconfigure event

After patch bda406c4, the state of the singlequeue was set to OK, but nothing
would then wake up the thread, as the other wakeup functions only look at
singlequeues that are marked as having received as not-linked.

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

docs: add missing docs, fixing doc errors

 * add many missing declarations to sections
 * GstController has been removed, update docs
 * skip GstIndex when generating documentation
 * rephrase so gtkdoc doesn't imagine return value
 * add missing argument description for gst_context_new()
 * document GstOutputSelectorPadNegotiationMode and move to header-file

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

docs: cosmetic changes in references/decriptions

 * fix typo GstBufferFlag -> GstBufferFlags
 * fix typo GstFeatures -> GstCapsFeatures
 * fix typo GstAllocatorParams -> GstAllocationParams
 * fix typo GstContrlSources -> GstControlSource
 * do not refer to gstcheck as an object
 * make references gtk_init() and tcase_set_timeout() not be references
 * gst_element_get_pad() renamed gst_element_get_static_pad()
 * gst_clock_id_wait_async_full() renamed gst_clock_id_wait_async()
 * _drop_element() is really gst_queue_array_drop_element()
 * gst_pad_accept_caps() was removed, do not refer to it
 * separate GST_META_TAG_MEMORY_STR declaration from description
 * do not describe removed gst_collect_pads_collect()
 * correctly link to GstElementClass' virtual set_context()

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

parse: Manually insert priv_gst_parse_yyget/set_column prototypes for older flex

Older versions of flex (before 2.5.36) don't add the prototype, so it must
be added manually. We can't check by the version number, because Debian/Ubuntu
patched it into their 2.5.35 at some point.

gstpad: drop assertion on gst_pad_peer_query_position

It is a 'both' query, so it can be sent both ways

inputselector: handle gap events

Use gap events to advance the selector's pad position.

This is relevant to keep sync_streams mode working when one of the
streams doesn't have data all the time.

Revert "ghostpad: copy sticky events to SRC ghostpads"

This reverts commit 8162a583a4dd68582bf186e2e47a8f0d68fa1980.

Automatically copying the sticky events makes it impossible for apps
and elements to filter the events with event probes. This causes
regressions (See #719437). The best option is to let the app/element
copy and filter the events themselves after the ghostpad target is
set.

parse: fix segfaulting prototype-mismatch

Now YYDEBUG is always set, so check it's value

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

netclock: Fix docstring for round-trip-limit and uninit access warning.

Fix a typo in a doc string - the property is round-trip-limit, not
roundtrip-limit.

Remove a bogus GST_WARNING that can print an uninitialised variable
and is redundant anyway.

netclock: Add round-trip-limit parameter

Sometimes, packets might take a very long time to return. Such packets
usually are way too late and destabilize the regression with their
obsolete data. On Wi-Fi, round-trips of over 7 seconds have been observed.

If the limit is set to a nonzero value, packets with a round-trip period
larger than the limit are ignored.

Signed-off-by: Carlos Rafael Giani <dv@pseudoterminal.org>
https://bugzilla.gnome.org/show_bug.cgi?id=712385

query: Fix gi annotations of gst_structure_new_custom()

netclock: Fix C99 comment

netclock: Implement rolling-average filter on observations.

Keep a rolling average of the round trip time for network clock
observations, favouring shorter round trips as being more accurate.

Don't pass any clock observation to the clock slaving if it has a
round-trip time greater than 2 times the average.

Actual shifts in the network topology will be noticed after some
time, as the rolling average incorporates the new round trip times.

basesink: Add debug into gst_base_sink_default_query() for accept_caps

gst-launch: exit with an error code when an error occured

If the pipeline failed to pre-roll or the user interrupted the
execution then set the exit code to a positive value.

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

gstutils: Escape stream id format in comments

These must be escaped for gtk-doc to parse the comments without warnings.

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

gst: g_memmove() is deprecated

Just use plain memmove(), g_memmove() is deprecated in
recent GLib versions.

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

ghostpad: copy sticky events to SRC ghostpads

Update the sticky events on SRC ghostpads when retargeting. This ensures
that the ghostpad has the exect same sticky events as the target pad. We
don't want to do this for SINK ghostpads, they got the events from
downstream and we don't want to overwrite them with the target pad
events.

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

pad: move debug function closer to the enum it debugs

pluginloader: check read/write before closed

first try to read or write on the socket before checking the closed state. This
makes sure we handle all data on the socket before erroring out.

poll: improve debug

So that we can see the return values of functions in the log.

tests: fix GstBus unit test with latest GLib

g_source_remove() works on the default main context, and
we're doing things with a custom context. Fixes warning
with newer GLib versions.

docs: cosmetic since marker fixes

baseparse: ensure to preserve upstream timestamps

... rather than have subclass coming up with an internally parsed one.

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

bin: Resync iterator if necessary

value: Lists with all equal elements are equal to a single value

Otherwise caps containing f={X, X} are not compatible with f=X

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

systemclock: add Since markers for new API

win32: Really update the def files

win32: Update def files

gst-uninstalled: Also export LD_LIBRARY_PATH for gst-plugins-gl

gst-uninstalled: export gst-plugins-gl DYLD_LIBRARY_PATH

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

collectpads: Always send SEEK events to all pads, even if one fails

collectpads: Update documentation for flushing seek handling

collectpads: Don't leak seek events

collectpads: implement flushing seek support

Implement common flushing seek logic in GstCollectPads. Add new
API so that elements can opt-in to using the new logic
(gst_collect_pads_src_event_default) and can extend it
(gst_collect_pads_set_flush_function) to flush any internal
state.

See https://bugzilla.gnome.org/show_bug.cgi?id=706779 and
https://bugzilla.gnome.org/show_bug.cgi?id=706441 for the
background discussion.

API: gst_collect_pads_set_flush_function()
API: gst_collect_pads_src_event_default()

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

tests: collectpads: add flushing seek tests

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

tests: collectpads: tweak stub _collect to push all buffers

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

tests: collectpads: update my email address

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

queue: Don't use gst_buffer_get_size() when possible

Makes qst_queue_locked_dequeue 20% faster

systemclock: Add gst_system_clock_set_default

Used for setting the default system clock that is obtained through
gst_system_clock_obtain(), which is sometimes needed for unit
testing.

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

typefind: use g_get_prgname() for error message

valve: proxy caps and allocation

Proxy the caps queries on the srcpad as well.
Proxy the allocation query on the sinkpad.

Automatic update of common submodule

From 865aa20 to dbedaa0

gst-inspect: Remove some dead code

memory: explicitly cast to GstLockFlags to avoid compiler warnings

segment: resurrect sanitizing start and stop for seeking

basesrc: mind boggling wrap when comparing offsets

baseparse: try first frame pts and dts for a valid start timestamp

baseparse: print proper variable in debug statement