gstpad: make some debug statements more verbose

ghostpad: Make some debugging more verbose

Also, remove an unnecessary #include in input-selector

GstSegment: Fix doc description string last_stop->position

inputselector: fix clock leak

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

tools: output gst-inspect errors to stderr

value: fix crash serialising a 0 flags value when there's no name for it

Fixes segfault when doing gst-launch-1.0 -v -m camerabin
(encodebin notifies a 0 value for its "flag" property).

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

gst: log performance warning debug message if glib emulates atomic ops

gst: use configure-detected or externally provided glib-mkenums

To ease cross-compilation.

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

Automatic update of common submodule

From 668acee to 4f962f7

configure: bump gtk-doc req to 1.12 (mar-2009)

This allows us to e.g. unconditionally use gtkdoc-rebase.

memory: add _make_writable

docs: mention some media type changes in porting-to-0.11.txt doc

docs: minor update to porting doc for child proxy lookup method

And a typo fix.

allocator: make a copy with the same alignment

When making a copy of the memory allocated from the default memory allocator,
make sure the new copy has the same alignment as the original memory.

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

baseparse: make seeking in DEFAULT format work if the subclass can convert for us

We only deal in TIME format ourselves, but if the subclass can handle
converting other formats into TIME format, we can support that too.

Fixes seeking in DEFAULT (sample) format with flacparse,
and the flacdec unit test.

tools: minor fixes to gst-launch man page

preset: implement child_proxy support

Elements such as the GstIirEqualizerNBands would so far not store the properties
of their children. Now we also grab the properties of child elements and try to
restore them.

inputselector: Wait for other streams to advance on unselected pads

Otherwise we end up dropping a lot of data in the case where data starts
arriving on the non-selected pad, resulting in big gaps in stream switching

inputselector: More debug statements

inputselector: Don't forward stream-start sticky events

Only one STREAM_START event should be let through, else it will
confuse downstream elements that think a new stream is starting
whereas in fact we are just switching to a different input.

In the future we might want to let them through but with the same
sequence number.

docs: Add new basesrc/basetransform API to the docs

basetransform: getters for pool and allocator

Sometimes a transform filter would need the buffer pool or the memory
allocator negotiated by the base class, for example, for querying different
parameters, such as a bigger number of buffers to allocate by the buffer pool.

This patch expose a two getters accessors: one for the buffer pool and the
other for the memory allocator.

basesrc: getters for pool and allocator

Sometimes the sources would use the buffer pool or the memory allocator for
something else than just allocating output buffers; for example, querying for
different parameters, such as a bigger number of buffers to allocate by the
pool.

This patch expose a two getters accessors: one for the buffer pool and the
other for the memory allocator.

registry: remove some unused and in their current form pointless API

Not so useful: just adds/reads stuff from an internal GList without
actually doing anything with those paths, so remove for now:

 gst_registry_add_path
 gst_registry_get_path_list

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

parse: fix up for gst_child_proxy_lookup() only working on child proxy interfaces

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

childproxy: make gst_child_proxy_lookup() a proper GstChildProxy method

No longer accept any old GObjects. This makes things nicer for
bindings. If a utility function that handles both nicely
is deemed worthwhile, we can still add one to gstutils.

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

value: when serialising arrays or lists, handle types we can't serialise more gracefully

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

consistencychecker: add some more details to failure messages

Mention pad where the problem occured, and the event name.

tests: fix collectpads test

After an EOS we must send a FLUSH_STOP event if
we want to send data again.

event: fix leak in gst_event_parse_stream_start()

gst_structure_id_get() will make a copy of the string
extracted, but we're assigning it to a const gchar *.

tests: make pipeline test valgrind clean

tests: fix pipeline unit test

Which was disabled because it failed.

scripts: fix unterminated quoted string in create-uninstalled-setup.sh

docs: mention gst_video_format_parse_caps() in porting guide

docs: fix up docs a bit

childproxy: fix up g-i annotation for _lookup() paramspec return value

No ref is returned here.

win32: update .def file for new buffer functions

baseparse: fix reverse playback with upstream demuxers that support it

Don't just return FALSE for seek events with negative rates when
operating in push mode. An upstream demuxer may support this just
fine, so if we're not operating in pull mode always check upstream
first if it can handle the seek event. This fixes reverse playback
where the upstream demuxer supports it (e.g. with qtdemux). The
same code would work fine in 0.10, because baseparse will just
call the default pad event handler if FALSE was returned from the
baseparse event handler, and the pad event handler will just
forward it upstream. In 0.11 the baseclass or subclass is
responsible for chaining up to the parent class or forwarding the
event upstream in any case.

Disable reverse playback in pull mode for now, there seems to
be something going wrong with the segment configuration in that
case.

basetransform: do not error on not-negotiated

Don't error out too early and let upstream decide if it can
workaround a not-negotiated problem

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

basesrc: retry on not-negotiate if a reconfigure is pending

Before erroring out on not-negotiated returns, check if the pad
has the reconfigure flag set and retry.

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

pad: add gst_pad_needs_reconfigure

Add an alternative version of gst_pad_check_reconfigure that doesn't
clear the reconfigure flag.

Useful for increasing error resilience without duplicating the
reconfigure code in pad task functions.

API: gst_pad_needs_reconfigure

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

pad: add GST_PAD_LINK_CHECK_DEFAULT to GstPadLinkCheck

This allows introspection-based bindings to access
Gst.PadLinkCheck.DEFAULT instead of
Gst.PAD_LINK_CHECK_DEFAULT.

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

buffer: mark gst_buffer_wrapped* data as array

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

introspection: fix some warnings generated by g-ir-scanner.

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

buffer: convert gst_buffer_* macros to functions

GObject Introspection does not support macros.
This is needed for bindings. We can still add back
macros or inline functions again later if we think
it's worth it.

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

netclientclock: fix printf format in debug message

bufferpool: fix max_buffers handling

When max_buffers > 0 and the pool is empty, actually try to allocate more
buffers up to the max_buffers limit.
We need to add a counter for this to count how many buffers we allocated and
check this against the max_buffers limit.
Reorganise and clean up some code.

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

netclientclock: simplify by using g_socket_condition_timed_wait()

No need to use a custom main context and custom timeout sources,
just use g_socket_condition_timed_wait() instead, which was added
for exactly this case.

Also seems to help with the unit test deadlocking with glib 2.33.x

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

gstobject: fix double string escaping in gst_object_default_deep_notify()

Make output of gst-launch -v readable again.

last-message = "event\ \ \ \*\*\*\*\*\*\*\ \(fakesink0:sink\)\ E\ \(type:\ tag\ \(20510\)\,\ GstTagList-stream\,\ taglist\=\(taglist\)\"taglist\\\,\\\ video-codec\\\=\\\(string\\\)H264\\\,\\\
 minimum-bitrate\\\=\\\(uint\\\)636611\\\,\\\ bitrate\\\=\\\(uint\\\)980729\\\,\\\ maximum-bitrate\\\=\\\(uint\\\)1116707\\\;\"\;\)\ 0x15bc760"

vs.

last-message = event   ******* (fakesink0:sink) E (type: tag (20510), GstTagList-stream, taglist=(taglist)"taglist\,\ video-codec\=\(string\)H264\,\ minimum-bitrate\=\(uint\)856039\,\ bitrate
\=\(uint\)1019748\,\ maximum-bitrate\=\(uint\)1116707\;";) 0x11149e0

miniobject: check writability

fix the writability check for miniobjects. We should check the shared counter.

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

allocator: Set the alignment at the correct place in GstAllocationParams

Back to development

Release 0.11.93

tests: remove silly test_fail_abstract_new check

Our check would make sure that GLib segfaults when
someone tries to instantiate an abstract type, which
is an extremely useful thing to check for.

In newer GLibs this is fixed and we get an abort with
a g_error() now it seems, so let's just remove this
check entirely.

examples: don't put things with side effects inside g_assert()

They will be defined away to NOOPs otherwise in release builds.

win32: update for stream-id API additions

parse: fix for new GstChildProxy::child-added signal callback signature

Fixes crash with gst-launch-1.0 uridecodebin uri=... suburi=... ! ..

bus: Add allow-none to the function argument of gst_bus_set_sync_handler()

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

docs: Make sure scanner gets required libraries

consistencychecker: print which event we received before stream-start

basesrc: don't try to answer URI queries with NULL URIs

Should make unit tests in -base that use appsrc a bit happier.

event: Update for stream-start event API changes

event: Add new stream-id field to the stream-start event

This is supposed to allow uniquely identifying a single stream.

inputselector: Use the first created pad by default

This guarantees a bit more consistency in which input stream will
be selected by default. It would previously be the first pad on which
an event/buffer/query was received ... which was racy and non-predictable.

element: Specify the order of pad iterators

The order of returned pads wasn't specified before, so let's specify
it and use an order which might prove the most useful : the order in
which pads were added to the element.

If someone changes the order, make sure users of those iterators from
now on don't rely on that order !

check: add tcase_skip_broken_test() define

Skips broken tests but logs an ERROR-level message to
draw attention to that fact.

tests: update .gitignore for queuearray test binary

tests: fix spurious netclientclock test failures

Give clocks a bit more time to synchronise.

win32: update generated files

input-selector: use generic marshaller for "block" action signal

Automatic update of common submodule

From 94ccf4c to 668acee

buffer, defaultmem: add option to poison memory before freeing it

Might be useful to track down certain bugs.

gst: ref/unref taglist scope enum in gst_init()

Fixes make check and distcheck

plugin: warn if plugin name starts with a "

This can easily happen as side-effect of the plugin name
in GST_PLUGIN_DEFINE no longer being a string in 0.11, but
a name to G_STRINGIFY.

docs: update porting-to-0.11 document with a "soft" API changes checklist

Point out some API changes that the compiler won't
be able to warn about.

tools: fix printing of partial dates in gst-launch

Revert "tools: print TOC scope"

This reverts commit ee6ab7c93638a6519acb976699a6ad149d520a95.

The application will probably only ever receive global TOCs,
so don't really need this.

win32: add new tag list scope symbols

typefind: send segment_done event in addition to segment_done message

basesrc: Add default handler for URI query in GstURIHandler subclasses

check: unbreak fail #define

The fail() definition was changed to not fail with non-GCC compilers,
unfortunately the change was incorrect and appended the first argument
of fail to the expression string instead of making it the message.

This change does mean that fail() now requires a message to be passed
along.

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

buffer: Update annotations

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

utils: Update annotation for get_compatible_pad

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

uri: Fix wrong 'array zero-terminated=1' annotation for strings

docs: update TOC design docs a little

event: make TOC event multi-sticky

We need to send two kinds of TOCs downstream as events,
and need both to stick to the pads.

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

tools: print TOC scope

toc: add GstTocScope and require it in the constructor

This is because we need to be able to signal different TOCs
to downstream elements such as muxers and the application,
and because we need to send both types as events (because
the sink should post the TOC messages for the app in the
end, just like tag messages are now posted by the sinks),
and hence need to make TOC events multi-sticky.

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

scripts: create-uninstalled-setup.sh: check for basic build tools and deps

.. before checking out stuff.

tag: Add a scope to taglists

This specifies if a given taglist applies to the complete
medium or only this specific stream. By default a taglist
has a stream scope.

Fixes bug #677619.

segment: add offset field

Add an offset field that is used to track at what position the segment was
updated. This is used to set the running time to 0 when we do a flushing
seek that doesn't update the position.

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

Update for new seeking variable name

When seeking, the start value and type are now called start and start_type.

segment: small cleanup

Move the code to update the segment at the end of the function.

Update docs and .def file for taglist API change

segment: remove redundant checks

We don't need to check the segment format anymore because we asserted on them
being equal before.

tests: improve segment tests

allocator: remove user_data from alloc vmethod

Remove the user_data from the alloc vmethod. Subclasses that implement a new
alloc function can also implement their own vmethod to pass extra arguments. We
can then also require that custom allocators implement an alloc function so that
gst_allocator_alloc() always works.

tests: remove segment accumulation checks

Remove the checks because there is no more segment accumulation.

taglist: make GST_TAG_APPLICATION_DATA also a GstSample

That way additional meta-data can be passed along with it.

taglist: gst_tag_list_get_buffer*() => gst_tag_list_get_sample*()

Image tags and other tags are now of GstSample type.