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

parse: Make the FATAL_ERRORS flag also work without a GError

Also add a unit tests

gst-launch: fix potential uninitialized variable warning

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

docs: fix common typos emited/eachother/...

utils: Add some attributes and reorganize code to fix compiler warnings

gstutils.c:3659:41: error: format string is not a string literal
[-Werror,-Wformat-nonliteral]
    gchar *expanded = g_strdup_vprintf (stream_id, var_args);

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

pwg: rename the "samplerate" variable to make example code compilable

In one of the examples about gst_my_filter_setcaps() there is a variable
declared as "rate", but then the name "samplerate" is used when setting
the caps.

Use the name "rate" everywhere in gst_my_filter_setcaps().

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

doc: fix forward reference about ghost pads

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

docs: design: fix some fixes

docs: flesh out gst-uninstalled entry in faq some more

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

docs: FAQ update to mention create-uninstalled-setup.sh

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

registry: small cleanups and use object log variants more

private: remove left-over comment

The caps are saved in the registry.

registry: use g_slice_free for slice memory

Avoid memory list corruption, but g_free'ing slice memory.

design: flesh out the tracing design a little more

docs: fix typos in gstobject

docs: Gram and nit fixes for part-meta.txt

docs: Gram and nit fixes for part-element-source.txt

docs: Gram and nit fixes for part-sink.txt

docs: Gram and nit fixes for part-conventions.txt

docs: Gram and nit fixes for part-controller.txt

docs: Gram and nit fixes for part-context.txt

docs: Gram and nit fixes for part-clocks.txt

docs: Gram and nit fixes for part-caps.txt

docs: Gram and nit fixes for part-buffer.txt

docs: Gram and nit fixes for part-bufferpool.txt

docs: Gram and nit fixes for part-buffering.txt

docs: Gram and nit fixes for part-messages.txt

docs: Gram and nit fixes for part-memory.txt

net: Constify a parameter to gst_net_client_clock_new()

Even though this parameter is not used, it should be const to fit in with the
coding standards for other similar parameters. Client code already passes in
const strings under the expectation that they won’t be modified.

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

datetime: Make sure to include gst_private.h before glib-compat-private.h

We need to define the GLib log domain before including glib.h, which is
included by glib-compat-private.h.

docs: Add gst_pad_store_sticky_event to sections.txt

So it appears in the generated documentation

tests/filesrc: Set location in wrong state

Also remove incorrect comment about code possibly not being reachable
that is now exercised by the filesrc unit test.

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

parse: Fix transfer annotations for parse_launch functions.

gst_parse_launchv, gst_parse_launchv_full and gst_parse_launch_full
all return floating refs, the same as gst_parse_launch, which just
calls gst_parse_launch_full internally anyway.

Add a unit test assertion to check it's true.

Spotted by nemequ on IRC.

core: Fix max DEBUG_LEVEL incongruence on 5 vs 9

In the docs and the autocompletion logic the maximum
value jumped incongruently between 5 and 9.

caps: Skip test_subset_duplication until the bug is fixed

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

docs: Fix some reference URIs

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

caps: Add a testcase for subset checks on lists with duplicated items

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

collectpads: Call the collected function while it returns FLOW_OK

This allows us to make sure the elements is EOS and does not have
remaining buffers to be drained.

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

docs: fix function name in qos design docs

tests: use tcase_skip_broken_test() to skip broken multiqueue test

So that we get a warning in the output that reminds us that
something needs to be fixed.

check: Disable multiqueue test_output_order check

The check itself is racy.

 (CK_FORK=no GST_CHECK=test_output_order make elements/multiqueue.forever).

The problem is indeed the test and not the actual element behaviour.

The objects to push are being pulled out of the single internal queues in the
right order and at the right time...

But between:
* the moment the global multiqueue lock is released (which was used to detect
if we should pop and push downstream the next buffer)
* and the moment it is received by the source pad (which does the check)

=> another single queue (like the unlinked pad) might pop and push a buffer
downstream

What should we do ? Putting a bigger margin of error (say 5 buffers) doesn't
help, it'll eventually fail.

I can't see how we can detect this reliably.

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

value: fix caps serialization when there are caps inside caps

Wrap caps strings so that it can handle serialization and deserialization
of caps inside caps. Otherwise the values from the internal caps are parsed
as if they were from the upper one

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

pluginloader: Check errors on the proper fd

Most likely a copy-paste error from the block before.

If we're going to check for error/closed on the write fd... do it
on the write fd

docs: fix spelling of "generic" in GstBaseSrc's documentation.

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

pad: only check event order when something changed

Check the event order in dataflow only when something changed instead
of for each buffer.

Automatic update of common submodule

From 6b03ba7 to 865aa20

configure: Actually use 1.3.0.1 as version to make configure happy