warnings: avoid set-but-unused warnings with load-save disabled

uri: fix wrong G_GNUC_MALLOC

_get_protocols() points to const memory in 0.10
despite the non-const return value.

libs: sprinkle some G_GNUC_MALLOC

Maybe gcc can do something clever with that, or at least
warn us if we don't save the return value somewhere.

gst: sprinkle some G_GNUC_MALLOC

Maybe gcc can do something clever with that, or at least
warn us if we don't save the return value somewhere.

buffer: add gst_buffer_{set,get}_qdata()

Allows people/us to attach arbitrary metadata to buffers.

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

API: gst_buffer_set_qdata()
API: get_buffer_get_qdata()

pad: Remove g_warning() if pad accepted caps that are not a subset of the pad caps

This check is correct but unfortunately it's impossible to implement
in a threadsafe way because the caps could have changed in the meantime.

Fixes bug #659606.

multiqueue: check filled state of queues even if another one is empty

This will avoid a case where overrun is never signalled if some
stream never produces any data, causing playbin2 to not end preroll.

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

baseparse: Return template caps instead of other side's peer caps if get_sink_caps vfunc is not implemented

Using gst_pad_proxy_get_caps() breaks backwards compatibility with old
parsers because it will propagate the other side's fields like "parsed"
and "framed" and also breaks parser/converters.

Fixes bug #664221.

gst-inspect: print current value as default value

Instead of printing separate 'Current' and 'Default' values
(the former obtained via g_object_get() and the latter from
the property GParamSpec), simply print the Current value as
the Default value. This is the right thing to do for almost
all elements and avoids confusion if a subclass of a base
class chooses a different default than the base class.

gst, controller: replace g_list_prepend + reverse with GQueue

Android: build audio controller example

Add buildsystem hooks for building the audiocontroller example
with the NDK.

Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@collabora.com>

gstvalue: consider lists and ranges equal if they hold the same set

This fixes caps operations when different elements advertise some
of their caps' properties differently (eg, for audio channels, either
a range from 1 to 2, or a list of 1 and 2).

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

tests: add a subset test for structure

docs: also fix wrong call order for controller in manual

tests: pad: add test to verify flushing behaviour

Seems like a trivial case, but this was actually broken in 0.11 recently.

docs: fix Since: markers for API added after 0.10.35

win32: update .def file for new API

API: gst_caps_is_strictly_equal()

gstcaps: remove unneeded use of gint64

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

gstvalue: quicker version of intersection when we do not need the result

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

gststructure: simplify return statement in gst_structure_can_intersect

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

gststructure: early out when we know a value cannot be a subset

If two values can be ordered, but are unequal, they are
necessarily distinct, thus one cannot be a subset of the other.

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

gstvalue: quicker test for substraction emptiness

When we do not care about the actual resulting set,
but only whether it is empty of not, we can skip a fair bit
of GValue juggling.

Add a function that does so, since we cannot just pass NULL
to the existing API as it may be part of the API contract.

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

gststructure: rejig test ordering for speed

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

basetransform: Only use the cached transform on strictly equal caps

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

caps: Add gst_caps_is_strictly_equal

introspection: add Value annotations for GST_SECOND, GST_MSECOND, GST_USECOND, GST_NSECOND constants

gobject-introspection won't parse them properly otherwise.

Still need to force the right type though (either GstClockTime or
guint64), but Type: xyz has no effect for me here, so someone with
a newer g-i needs to test this.

Some other defines are also missing, e.g. GST_CLOCK_TIME_NONE.

configure: suppress unused variable warnings if gst debugging is disabled

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

configure: fix typo around GLIB_EXTRA_CFLAGS in GST_ALL_CXXFLAGS

queue: use statically allocated GQueue

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