Automatic update of common submodule

From 357b0db to fd7ca04

gstbin: unlock _get_state() on error

When an error message is received on the bus, mark the bin as being in the error
state and unlock all current _get_state() calls with an error.

Fixes #505770

checks: add multi-thread test for tagsetter

See #619533.

tagsetter: make sure only one thread creates the TagData

tagsetter: protect tagsetter operations with a lock

So we don't crash when a muxer tries to add tags from two
threads at the same time, eg. because it received tag events
on two input pads simultaneously.

See #619533.

caps: use our macros more often in the code

caps: add append_structure_unchecked

This is useful when we know that caps is !NULL, writable and structure is
!NULL too.

tests: rename testsuite

Previous name was only applicable to a few of the tests.

docs: xref function name

caps: use our macos more

design: more planning on lazy caps.

structure: API: Add gst_structure_fixate_field_string()

design: collect ideas for having lazy caps

Design doc for having on the fly evaluated caps (see bug #618853).

docs: add links for GSource priorities

Now it is xreffed with the glib docs, where the priority scale is explained.

benchmark: add commandline parameters for capsnego

Allow to specify the graph size and offer two flavours (audio/video).

benchmarks: add a benchmark for capsnegotiation

The test builds a tree like graph having conversion and basetransform elements.

benchmarks: use gst_util_get_timestamp() instead of own implementation

basesink: add jitter to debug output

miniobject: cleanup type registration a little

We can make some structs const static with little effort.

pad: don't print WARNING debug statements for normal things like EOS, part II

Automatic update of common submodule

From 4d67bd6 to 357b0db

caps: comment and whitespace cleanup

Make comment more specific, reposition it and add more of the kind.
Move one ifdef'ed function around.

utils: Simplify fractions before doing calculations that could cause overflows

... to prevent some unnecessary overflows from happenening.

utils: GCD is 0 if both parameters are 0, don't divide by zero

And turn overflow checks from assertions into simple checks to
return FALSE.

utils: Simplify result of gst_fraction_multiply()

faq: updated line about jack output

tests: Read return value to make clang/icc happy

Revert "pad: don't check twice for changed caps per push"

We need to check the pad caps on the srcpad as well as on the sinkpad. Revert
this commit as it removes the check on the srcpad and can leave the srcpad
unnegotiated (or negotiated with wrong caps)

This reverts commit 07dc1e5b49580a89bfef27ff27476d51fb3ce2c2.

pad: don't check twice for changed caps per push

gst_pad_chain_data_unchecked() does the same check already.

basesrc: reflow to truncate caps just once

We get writable caps from the intersection (unless it failed). As we truncate
those anyway, we don't need to manualy copy the first structure.

tools: fix gst-run wrapper to work on Windows

Fixes #617625

docs: document that gst_byte_writer_put_string*() writes the terminator too

docs: clarify the pull_range functions

Clarify the gst_pad_pull_range(), GstBaseSrc::create(), gst_pad_get_range()
and GstPadGetRange functions a little.

Fixes #617733

basesrc: improve debugging

utils: use reffed _get_caps() version

We don't need to have a writable copy so we can use the _reffed
version instead.

tags: Adds geo location direction tags

Adds 3 new geo location tags involving direction and
movement of capture. Those are:

API: GST_TAG_GEO_LOCATION_CAPTURE_DIRECTION
API: GST_TAG_GEO_LOCATION_MOVEMENT_DIRECTION
API: GST_TAG_GEO_LOCATION_MOVEMENT_SPEED

Fixes #617223

tags: Adds GST_TAG_DEVICE_MANUFACTURER and GST_TAG_DEVICE_MODEL

Adds those new tags to describe the device manufacturer and
model used to create medias.

API: GST_TAG_DEVICE_MANUFACTURER
API: GST_TAG_DEVICE_MODEL

Fixes #615941

pwg: remove confusing metadata example with 0.8 code

Fixes #534314.

manual: add minimal tag reading example

Should probably put that into tests/examples and figure out how to
get it included automatically, but can't be bothered right now.

Bump GLib requirement to 2.20

See http://gstreamer.freedesktop.org/wiki/ReleasePlanning/GLibRequirement

buffer: only warn if metadata is not writable when it should be, don't return as well

Make sure we execute the same code path in git versions and in releases,
so just warn when metadata isn't writable when we want it to be instead
of bailing out.

element: make 'adding flushing pad' warning more useful

This is a pretty common issue with ghost pads, let's make
the warning more helpful and tell people what they need
to do to fix it.

gst-launch: add -p option to disable play handler.

Same logic as for the fault handler. This is useful for some debug/tracing tools
that need to grab SIGUSR1 and SIGUSR2 them self.

basesink: implement percentage position and duration queries

If upstream does not handle them, then implement those ourself.

basesink: use gst_pad_peer_query instead of reinventing.

queries: add more logging

Log human readable formats and log query result.

caps: Do not allow fixating empty caps

Passing empty caps to gst_pad_fixate_caps() is invalid, as empty caps
cannot be fixated.

caps: Use G_GNUC_WARN_UNUSED_RESULT for make_writable()

People often call
  gst_caps_make_writable (caps);
instead of
  caps = gst_caps_make_writable (caps);
and cause a bug. Warning about an unused return value helps here.

See https://bugzilla.gnome.org/show_bug.cgi?id=616541#c2 for an example.

gst: Use GError boxed type from GObject 2.25.2 instead of our own if possible

Back to development.

docs: add some more docs for the events

Release 0.10.29

Update .po files

tests: add more tests for controller

The tests verify that bug #616846 is indeed fixed.

controller: Fix gst_interpolation_control_source_find_control_point_iter

The logic in that function is broken. Various NULL-checking bandaids for
guaranteed non-NULL variables didn't even help there.

This patch updates the function to check if a previous item exists
before fetching it instead of after. This makes all other tests
unnecessary.
In particular, it makes the check for an empty list unnecessary, because
for empty lists the only iter is the begin iter (and the end iter) and
so the new check catches that case.

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

0.10.28.3 pre-release

doap: update repository info from cvs->git and maintainers

Automatic update of common submodule

From fc85867 to 4d67bd6

docs: Consistently use MyFilter instead of sometimes ExampleFilter in the example

Fixes bug #615579.

pad: add enums for custom flow return success and error codes

This way people can just #define their own custom flow returns to
one of these without having the compiler (esp. gcc-4.5) complain
about comparing integers to an enum or the enum not being listed

Fixes #615880.

API: GST_FLOW_CUSTOM_SUCCESS_1
API: GST_FLOW_CUSTOM_SUCCESS_2
API: GST_FLOW_CUSTOM_ERROR_1
API: GST_FLOW_CUSTOM_ERROR_2

lfocontrolsource: Use correct setter for double GValues

tags: doc fixes

Adds missing ':' to tags docs

bin: fix bogus variable type

The result of gst_iterator_find_custom() is not a GstIterator *.

0.10.28.2 pre-release

po: update translations

structure: log what structure string we failed to parse

bin: fix refcount when removing elements during state change

When an element is removed from a bin because it caused a state change error,
don't unref the child twice.
Add some more debug info.
Add a unit test for this error.

Fixes #615756

tests: more LDFLAGS -> LDADD fixes

build: $(LIBM) belongs into LIBADD not LDFLAGS

build: when building executables, put libs to link to into LDADD instead of LDFLAGS

Use foo_LDADD instead of foo_LDFLAGS to specify the libraries to link to.
This should make sure arguments are passed to the linker in the right
order. See #615697.

configure: Remove -Wcast-align

Apparently gcc warns that GstMiniObject is not castable to
GstEvent/Message/Buffer due to them containing 64bit variables, even
though ARM hackers claim that those only need 4byte alignment. And as
long as gcc behaves that way, this warning is not very useful.

So we'll remove the warning until this problem is fixed.

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

configure: remove superfluous return statements at end of AC_TRY_{LINK,COMPILE} blocks

Spotted by JF Mertens. See #614767.

configure: Change check for uint128_t

Check for ability to divide uint128_t values, since that what
we actually use it for (in gstutils.c).  The existence of a
uint128_t type doesn't mean the compiler can actually generate
code for it.  Also make sure that we can actually link the
result successfully.

Fixes bug #614767.

docs: minor moving-plugins addition

launch: make -q be more quiet

Convert some g_print into PRINT so that they are not printed when the -q option
is selected.

queue2: add some more debug info

fdsrc: allow specifying the size in bytes on the uri

Parse a size=value from the query string to specify a size. This is interesting
when reading from a file descriptor that actually has a size (and is not
stat-able, such as the socket of an http connection)

queue2: when EOS we know the duration

When we are EOS, we don't need to do an upstream query for the duration in bytes
because we already know it is the offset of the last written byte.

registrychunks: Initialize typefind/element factory registry chunks with zeroes

This makes valgrind stop complaining about reading unitializated memory,
which is not initialized because it's just compiler-added struct padding...

Automatic update of common submodule

From d66a8c3 to fc85867

.gitignore: add new .svg file in docs

docs: use informalfigure tag to not syntax highlight the content

docs: add concept map

Add a graphviz dot file. Add rules to render it to svg and include in docs.
Nodes are clickable. It is an attempt to show how things fit together.

docs: add a few code snippets that show how to use gst_message_parse_*().

build: bump autoconf requirement to 2.60 for gobject-introspection.m4

Require autoconf 2.60 (which was released in June 2006).

Fixes #600718.

parse: fix more compiler warnings

Fix 'grammar.tab.c:815:6: warning: "YYENABLE_NLS" is not defined'
compiler warning and the same for YYLTYPE_IS_TRIVIAL. The two
translated strings aren't particularly helpful, so just define
YYENABLE_NLS to 0.

parse: fix compiler warning

Fix 'grammar.y:668: passing argument 1 of ‘g_free’ discards qualifiers
from pointer target type' compiler warning.

message: add Since: markers

tests: gstsystemclock: don't leak the system clock

build: fix out of sourcedir build for check

Move the internal header to nodist (as we copy it around anyway).
Use builddir in pattern substitution for it.
Fixes #61483.

docs: fix some typos

basesrc: fix gst_base_src_new_seamless_segment()

Keep track of pending newsegment in gst_base_src_new_seamless_segment()
to avoid pushing newsegment update before newsegment.

docs: improve event docs

Rephrase first paragraph of section docs. Add detail to eos event docs.

gst-indent: Add --leave-preprocessor-space for indent 2.2.11

It was previously broken, which is why we never needed it. This keeps backward
compatibility with indent <= 2.2.11

libs: point gobject-introspection scanner to .la files

Point g-ir-scanner to the .la file of our library, which hopefully
makes it find the right dependencies in all cases (ie. our locally
built libgstreamer and not the system-installed one). This is also
how it's done in Gtk+ and how it's documented in the wiki, see
http://live.gnome.org/GObjectIntrospection/AutotoolsIntegration

Based on patches by Vincent Untz and Alan Knowles.

Fixes #603710.

utils: Use G_GNUC_CONST instead of G_GNUC_PURE for conversion functions

Fixes bug #614629.

tests: Don't forget to unref the newsegment event

common: Update to latest revision for new suppressions

tests: add test for updating playback rate

Tests if a seek with both positions being GST_SEEK_TYPE_NONE is handled.

docs: add copy and conv function to buffer2 draft