bin: use statically allocated GQueue

Because we can.

collectpads2: use flags enum instead of guint in structure

queue2: use statically allocated GQueue

ghostpad: Don't cache internal proxy pad target

The internal proxy pad target is simply a cache of the internal proxy pad
peer. This patch uses the well implement GstPad peer handling to obtain the
target. This fixes issues with target not being set in both direction when
two ghostpads are linked together (empty bin).

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

adapter: simplify gst_adapter_take_list()

Use a stack-allocated GQueue to assemble our GList.

configure: make GLIB_EXTRA_CFLAGS overwritable

Make 'make GLIB_EXTRA_CFLAGS=...' work.

filter: deprecate gst_filter_run()

It's not really used outside of core at all, and has
serious namespace issues. If anyone feels the need to
revive this one, please use a less generic name space.

API: deprecate gst_filter_run()
API: deprecate GstFilterFunc

registry: don't use soon-to-be-deprecated gst_filter_run()

Lines-of-code savings are negligible anyway.

pluginfeature: deprecate gst_plugin_feature_type_name_filter()

It's only used internally anyway and the helper struct
has namespace issues.

API: deprecated gst_plugin_feature_type_name_filter()
API: deprecated GstTypeNameData

taglist: add to_string and new_from_string functions

We want to make GstTagList opaque and not have people use
GstStructure API on it.

API: gst_tag_list_to_string()
API: gst_tag_list_new_from_string()

taglist: avoid pointless tag name -> quark lookups

We never get a tag name quark from a caller, it's always a
string, from which we'll try to look up our tag info in the
hash table, so change the hash table key from quark to string.
Avoids a bunch of pointless string => quark lookup in the
global quark table. We need to do an extra string => quark
conversion now when we copy a taglist, but in that case we're
in a slow path anyway.

taglist: add gst_tag_list_is_equal()

API: gst_tag_list_is_equal()

structure: identical structures are definitely equal

basetransform: Also fush the cache when changing the upstream caps suggestion

basetransform: Add debug output when returning a cached transform

collectpads2: Fix refcount handling if a buffer was dropped due to clipping

collectpads2: Merge the clip and prepare_buffer function into one

collectpads2: Merge clipping API from old collectpads

basesink: make default query function name show up in gst-inspect

collectpads2: avoid hanging in case of sparse newsegment events

... in the extent that a non-waiting pad (so indicated by newsegment)
turns out to provide the best buffer, which is then forced to waiting
for book-keeping purposes, but that should only be temporary.

See bug #415754.

collectpads2: Use G_DEFINE_TYPE instead of GST_BOILERPLATE

collectpads2: Add to the documentation

win32: Add new collectpads2 API

base: Add collectpads2

This handles muxing of sparse/subtitle streams and has
lots of cleanup. Still missing is special support for
live streams but this can be added later without breaking
API/ABI.

Based on the version from the videomixer plugin.
https://bugzilla.gnome.org/show_bug.cgi?id=415754

basetransform: Fix refcount leak

Don't leak peercaps and a ref to the basetransform when returning
the cached caps.

controller: fix wrong order of calls in the docs.

event: clarify docs for step event

registry: add support for GST_REGISTRY_REUSE_PLUGIN_SCANNER=no

This will make sure we spawn a new plugin scanner helper for each plugin
to be introspected, which helps with making sure we don't load too many
shared objects (libs, plugins) at the same time on systems where there
is a hard limit like on Android.

A better version might re-use the scanner for up to N times, though
it's not clear whether that would actually improve things dramatically.

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

caps: No need to require writable caps for _append/_merge second caps

The second caps ownership is transfered, no need to require it to
be writable from the caller function. Instead, _append and _merge
make it writable on their own.

Discovered because of an assertion on encoding-profile.c in
_get_input_caps using _merge but not passing writable caps.

baseparse: Fix documentation, it's pre_push_frame and not pre_push_buffer

basetransform: cache transformed caps where appropriate

Speeds up negotiation a fair bit on a contrived pipeline
with a dozen colorspace conversions.

Hopefully clears out the cache every time it ought to.

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

Fix broken android building due to name change in 01d87250a845e55d6

basebarse: Add detect vfunc to allow subclasses to do format detection before anything else

API: GstBaseParseClass::detect()

This is called with the first buffers until the subclass has finished detection
and only afterwards the original buffers are handled as before. The vfunc allows
detection of the stream format without breaking the upstream framing.

pad: not only describe conditions in the docs, also check them in the code

When blocking pads, check if the pad is in the rigt direction. Log some info
for the developer and return FALSE, instead of just locking up.

message: protect against null message sources

Message sources can be null, check if it is before trying to
access its name.

baseparse: add since doc to new getcaps function

logging: more logging and prefer human readable details over memory locations

debugutils: improve display of ghost- and proxypads

Handle virtual links between ghost and proxypads when iterating pads instead of
when linking. Besides using less code this provides a more accurate picture.

baseparse: add getcaps function

Adds a getcaps function to the sink pad to make parsers propagate
downstream caps restrictions to upstream.

The pipeline "audiotestsrc num-buffers=100 ! faac ! aacparse !
"audio/mpeg, version=(int)4, stream-format=(string)adts" ! filesink"
wouldn't work because aacparse wouldn't propagate the adts restriction
upstream to faac.

This patch adds a default getcaps to the sink pad to simply proxy
downstream caps and also adds a 'get_sink_caps' function pointer
to GstBaseParseClass for subclasses that need more refined getcaps.

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

basesrc: also update the stream time

baseparse: Fix handling of queued frames

gst_base_parse_push_frame() already frees the frame, no need to
do it another time again.

bus: give watch source a name

Give our GSource a meaningful name. Source names can be
used for debugging and profiling, for example with
systemtap or gdb.

gstbuffer: Add transfer annotations for gst_buffer_join()

bus: fix typo in the docs

debugutils: show if an element is state-locked

logging: use _OBJECT variants more

buffer: Use an inline function instead of a macro for gst_buffer_replace()

This gives us type checks by the compiler and more useful compiler errors.

event: Use an inline function instead of a macro for gst_event_replace()

This gives us type checks by the compiler and more useful compiler errors.

basesrc: properly adjust start time

When we do a non-flushing seek and closed the current segment,
make sure that we open the next segment from where we closed.

utils: catch invalid instance sizes in gst_type_register_static_full()

Add guards to catch overly large instance sizes.

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

gstparse: prefix generated parser functions so they don't get exported

Don't export those 35-something random _gst_parse_yy* symbols. These were
never in any header files and also blacklisted from our .def files, in
case anyone wonders.

pipeline: Use pipeline category for one more log message

Makes debugging easier.

GST_PTR_FORMAT: Add GstBuffer ptr format and use in GST_SCHEDULING

GstBuffer pointers can now be printed using GST_PTR_FORMAT. This is used
in the very useful GST_SCHEDULING debug logs in gstpad.c and allows for
easier and more information tracking of buffer progress through a
pipeline with just debug logging.

baseparse: send duration message when updating internal duration

baseparse: Return success if optional start/stop method is not provided

This allows to not implement the optional start/stop methods.

gstquery: Make debugging message more informative

For all the newcomers out there who still don't know the values of
GstQueryType enum by heart...
... and old-timers who've got better things to do :)

baseparse: make estimating the position in query handler actually work

No point estimating if we don't set the result afterwards.

baseparse: answer position query in stream time and try upstream first

Let the demuxer have first say as well.

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

gst.h: include header for atomic queue

tests: add minimal test for GstAtomicQueue

Just new + free.

basetransform: send delayed events earlier

Some elements (such as videorate) might push buffers early,
for instance in in transform_ip. We want events (and in particular
any NEWSEGMENT event) to be pushed before that.

This fixes transmageddon wedging on converting a file starting
with a non zero offset to Ogg.

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

gst: Fix compiler warnings on 64 bit mingw-w64

Fixes bug #660083.

examples: fix bogus g_object_unref in helloworld example

GMainLoop is not a GObject.

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

basetransform: delay serialized events when src caps are not set yet

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

pad: Set caps on pad before checking if the pad is linked

This allows the setcaps handler and notify::caps to link
the pad downstream and doesn't require hacks to always
provide a peer to the pad, like in decodebin2.

caps: use g_value_take_string() and gst_value_get_caps() instead of accessing internal fields

pad: add more debug logging for other chain function code path as well

pad: fix up printf format in debug message

Which I messed up.

pad: make some debug traces more useful

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

scripts: remove gstcvstest.sh

lfocontrolsource: fix clang compiler warning

Cast enum to int before checking for negative values, which are
impossible according to the enum list.

gstlfocontrolsource.c:652:45: error: comparison of unsigned enum expression < 0
      is always false [-Werror,-Wtautological-compare]
  if (waveform >= num_waveforms || waveform < 0) {
                                   ~~~~~~~~ ^ ~

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

tests: make sure filesrc returns escaped URIs even if the input was unescaped

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

caps: move log messages for caps creation/freeing into TRACE category

Reduce SPAM for GST_CAPS:5.

pad: Only do the subset check in gst_pad_accept_caps() if the pad claims to accept the caps

basesink: make it easy to override the pad query

Add a vmethod to handle the pad query.
Install a default handler for the pad query.
Use the new query function in filesink

Revert "pad: Use gst_pad_accept_caps() instead of manually checking when configuring a sinkpad"

This reverts commit d3cad28da936b037d877dc70c02286b81b680284.

It causes performance problems because acceptcaps() propagates downstream.

pad: Use gst_pad_accept_caps() instead of manually checking when configuring a sinkpad

pad: Print a g_warning() if pad accept caps that are not a subset of its caps

In 0.11 only subsets are supported again as documented instead of also
allowing non-empty intersections.

Revert "basetransform: Use check for subsets and not non-empty intersection to check if caps are compatible"

This reverts commit 5e5cc5e89e7e2858a6352fa4c81a374f6e5a6297.

See bug #658541.

Revert "pad: Check for subsets, not non-empty intersections to check if caps are compatible"

This reverts commit 0c5d50207326d74a4805bcd898bfac887540f12b.

See bug #658541.

docs, gst: typo fixes

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

docs: fix make distcheck

No point removin those empty override files from git, they'll
just be re-created later, so let's tell gtk-doc about them, so
it can clean them up properly.

basetransform: If there's no peer we still have to transform ANY caps in getcaps()

Otherwise elements like capsfilter will return ANY caps if no
peer is present instead of the filter caps. The transform_caps()
vfunc could do transformations to the template caps that do not
result in the unmodified template caps.

docs: cleanup makefiles

Remove commented out parts that we don't need. Remove "the wingo addition" - no
so useful after all. Narrow down file-globs for plugin docs.

docs: escape % in docblob

ghostpad: Use gst_pad_set_caps() instead of manually changing caps

gst_pad_set_caps() does essentially the same but additionally calls
the pad's setcaps function.

Fixes bug #658076.

Automatic update of common submodule

From a39eb83 to 11f0cd5

Automatic update of common submodule

From 605cd9a to a39eb83

basetransform: Use check for subsets and not non-empty intersection to check if caps are compatible

pad: Check for subsets, not non-empty intersections to check if caps are compatible

Pads should only accept caps that are a subset of the pad caps, e.g.
they should accept only caps that have a non-empty intersection and
at least all fields of the pad caps.

Without this a pad that wants for example
  "video/x-h264,stream-format=byte-stream"
will be happy to accept
  "video/x-h264".

docs: add since 0.10.36 on the new _NONE enum values

stepping: use the proper argument order

Fixes warning #188: enumerated type mixed with another type reported by ICC.

fileindex: explicitly cast to the enum types

Fixes warning #188: enumerated type mixed with another type reported by ICC.

typefinder: use GST_TYPE_FIND_NONE instead of 0

Fixes warning #188: enumerated type mixed with another type reported by ICC.

lfocontrolsource: explicitly cast to the enum type

Fixes warning #188: enumerated type mixed with another type reported by ICC.

typefind: add GST_TYPE_FIND_NONE and use it

Fixes warning #188: enumerated type mixed with another type reported by ICC.

baseparse: use the enum values for 0 and don't abuse on gboolean coincidence

Fixes warning #188: enumerated type mixed with another type reported by ICC.

baseparse: put the arguments of g_return_val_if_fail in the proper order

parse: use GST_PARSE_FLAG_NONE instead of 0

Fixes warning #188: enumerated type mixed with another type reported by ICC.

value: explicitly cast to the enum type

Fixes warning #188: enumerated type mixed with another type reported by ICC.