utils: return immediately for -1 conversion

When we are asked to convert -1, we can return immediately with a -1 return
value.

utils: a convert query can have a -1 input value

It is allowed to pass -1 to the src_val for a convert.

basesink: also preroll after a flush with async=false

Make sure to preroll after a flush even when we are async=false.
Add unit test.

Fixes #634965

adapter: improve docs a little.

check: lock src state to avoid error cases

Lock the state of the src element or else the pipeline might go into the error
state when we set it to PAUSED later.

pipeline: avoid using invalid clock times

Be extra careful to not use invalid clock times but give a warning instead. This
should make things work better with faulty clock implementations.

caps: improve some comments about the zigzag intersection

docs: Add unused symbols to proper sections

docs: Add GstDateTime section and update it for latest symbols

queue2: Only call update_buffering if needed.

update_buffering is so big it will never be inlined (and shouldn't),
we therefore move the check outside of it.

queue2: Avoid re-checking many times whether an item is a buffer

Avoids calling 6 times gst_buffer_get_type() for every item coming
through queue2

queue2: Reduce amount of time locks are taken

queue2: Fixup documentation of some properties

queue2: Avoid useless segment_to_running_time() calculations.

* Cache src and sink time
* Use a boolean to known whether src/sink time needs to be recalculated

Avoids 50% of calls to gst_segment_to_running_time()

basesink: Re-using GstClockID instead of constantly recreating one

Makes _sink_wait_clock at least 2 times faster.

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

gstclock: New API to re-use a single shot GstClockID

API: gst_clock_single_shot_id_reinit

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

basesink: Pass along miniobject type through various functions

Avoids doing useless GST_IS_*

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

basesink: Switch enable_last_buffer to an atomic int

Avoids having to take a lock to read/write it.

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

queue: Remove useless checks from e406f7

srcresult was being rechecked in places it couldn't have changed.
queue level was being rechecked in places it couldn't have changed.

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

gstdataqueue: Only emit g_cond_signal when needed

Keep track of which cond we're waiting for and only emit when needed.

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

segment: move g_return_if_fail to where it is needed

version: Take nano version into account in GST_CHECK_VERSION()

If the nano is > 0 the current version should be handled the same as
micro + 1.

pad: Set the event source object if none is set yet in gst_pad_push_event()

Otherwise the source will stay at NULL, the event is passed to the
peerpad via gst_pad_send_event() and then the peerpad is set as
source of the event instead of the originating pad.

queue2: Remove dead assignment and unused variable

queue2: Remove dead assignment and move variable declaration into inner block

queue2: Remove redundant variable

Other than saving an immense amount of 4 bytes of memory this
prevents clang from complaining and keeps the ring buffer state
in a single variable instead of two.

gsttask: Set thread names on Windows with MSVC if a debugger is attached

Fixes bug #632168.

macros: Define restrict keyword if not available

This change always defines the restrict keyword if a
non-C99 C compiler is used. In the case of GCC >= 4
it will be defined to __restrict__, in all other
cases to nothing. This allows to use the restrict
keyword unconditionally.

Bump GLib requirement to >= 2.22

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

Back to development

Release 0.10.31

utils: avoid 'unused argument' warnings caused by GST_BOILERPLATE_FULL

The unused data parameter in the class_init trampoline function
seems to cause warnings with some c++ compilers.

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

docs: some small fixes to the plugin writer's guide

Fix wrongly placed example and weirdly phrased 'note' lacking proper
formatting. Fix missing hint for autogen.sh location and rephrase
'built and installed' sentence. Fix wrongly phrased and redundant
paragraph in PWG

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

manual: Improve states documentation

Be more explicit about being on NULL before unrefs

introspection: Include exported packages information in GIRs

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

0.10.30.5 pre-release

elementfactory: fix caps leak in element factory list utility functions

po: update translations

datetime: Add _from_unix_epoch variants

Adds 2 variants for the gst_date_time_from_unix_epoch function,
one for UTC and another for local time.

API: gst_date_time_new_from_unix_epoch_utc
API: gst_date_time_new_from_unix_epoch_local_time

Fixes #653031

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

math-compat: don't re-define _USE_MATH_DEFINES if already defined

This avoids compiler warnings.

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

0.10.30.4 pre-release

po: update translations

queue2: don't leak pad ref in pull mode when flushing

Fix pad leak when queue2 is flushing or being shut down.

queue2: don't send seeks beyond the end of the file upstream in pull mode

If downstream is operating in pull mode, short-circuit any pulls beyond
the end of the file and return FLOW_UNEXPECTED immediately instead of
sending a seek beyond the end of the file upstream, since this might
confuse upstream elements (and/or http servers, for example). Fixes
playback of apple trailers in totem and youtube/html5 clips in
WebkitGTK+.

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

basetransform: use input position for queries if we have no output position

basetransform: fix reverse negotiation

When the downstream element suggests a new format, pass the suggestion
upstream if we can't convert to it.

Fixes #633147

multiqueue: Fix tracking of unlinked streams.

33082eb9e42c52e4df848195946f1b7bbce768c5 introduced a bug
preventing sparse unlinked streams from advancing properly,
leading to the queue blocking.

Fixes: #633176

tests: Add a multiqueue sparse streams test

Automatic update of common submodule

From 7bbd708 to 011bcc8

0.10.30.3 pre-release

bytereader: const-ify byte reader argument in peek/scan API

Because we can.

elementfactory: Fix 64bit constant

Basically we're not meant to put anything more complex than simple numbers,
due to the definition of G_GUINT64_CONSTANT:
G_GUINT64_CONSTANT(val) (val##UL)

Which previously resulted in .... 1 << 49UL

basesink: recompute correct running time for buffer ending flushing step

Prevents delaying/hanging when resuming PLAYING.

Fixes #632433.

docs: fix misc. gtk-doc warnings in libs

(for gtk-doc 1.15)

Revert "libs/gst/dataqueue: Document gst_data_queue_new_full"

This reverts commit 80727c117703507f790a86b0962ab3d915e5a491.

This doesn't make sense. gst_data_queue_new_full() is already
documented above. And we need the doc blurb for _new() here.

docs: flesh out release instructions a bit more

docs: add some gtk-doc Since: markers

Add some gtk-doc Since: markers, fix one Since: marker,
fix typo.

0.10.30.2 pre-release

po: update translations

gst-inspect: useful factory names for uri-handlers output

Fixes #632236.

Automatic update of common submodule

From 5a668bf to 7bbd708

datetime: Use seconds as double

Use seconds as double to make API similar to glib's
gdatetime. Also move timezone parameter to the
first position, just like glib's.

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

gstdatetime: Move doc outside the ifdefs

Move the datetime documentation of the functions outside the
ifdefs

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

datetime: Use GDateTime if available

Use GDateTime internally on GstDateTime if glib already
provides it.

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

glib-private: Add include protection macro

buffer: add guard to buffer_set_caps() that checks if caps are simple

systemclock: add a missing G_PARAM_STATIC_STRINGS

libs/gst/dataqueue: Document gst_data_queue_new_full

base/gstdataqueue: inline some functions, get levels with memcpy.

queue2: Remove unused argument from find_range()

fdsink: cleanup get_property/set_property

queue: avoid g_cond_signal when we can

Keep track of what cond we are waiting on and only signal when some other thread
is waiting.

miniobject: avoid duplicate type check when freeing miniobject

gst_mini_object_unref() has guards that check the type already, so
we don't really need to re-check it here again while getting the
class (there's not really much point to that anyway, since we don't
check the return value of the get_class, so we'd crash anyway if
we're not dealing with a mini object, the only question would
be if there'd be a warning before the crash or not).

miniobject: Directly increate mini_object in mini_object_free()

Speeds up mini_object_unref by 25% by avoiding the typecheck which
is avoidable here since it is only called on existing miniobjects.

miniobject: Remove confusing DEBUG_REFCOUNT define

the debugging statements will be silenced automatically if debugging
is disabled, and the type check is actually required.

queue2: release queue2 lock before notify

Make sure that we don't hold the lock when we notify the temp-location
property,

Fixes #631853

queue: add debug statement

queue: apply sink segment on the source if queue is empty

Apply the sink segment on the source immediatly when it is received
and there is nothing in the queue.

Solves #482147

bin: fix documentation for iterate_sources

docs: use the gtk-doc shortcuts to get coloured and xrefed example

bin: Initialize variable

bin: Improve tracking of source elements

Track elements tagged with the IS_SOURCE flag in a similar way we track the sink
elements. This allows us to efficiently dispatch downstream events to the right
elements.

basesrc: tag as a SOURCE element

Tag all elements deriving from the basesrc with the IS_SOURCE flag.

element: add IS_SOURCE flag

Add the GST_ELEMENT_IS_SOURCE flag so that we can tag source elements like we
can with sink elements.

registry: g_mapped_file_unref exists already since GLib 2.21.3

basetransform: Report the output position on POSITION queries on the srcpad

There can be a difference between input and output last_stop.

Fixes bug #629410.

Automatic update of common submodule

From c4a8adc to 5a668bf

multiqueue: Remove unused variable and simplify code

oldid was only used when we were doing multiple pops per loop.

Automatic update of common submodule

From 5e3c9bf to c4a8adc

bytewriter: Add missing file

bytewriter: Add unchecked variants of the writing functions

These don't check if there's enough free space available and are
available as inline functions only.

API: gst_byte_writer_put_int8_unchecked
API: gst_byte_writer_put_int16_be_unchecked
API: gst_byte_writer_put_int16_le_unchecked
API: gst_byte_writer_put_int24_be_unchecked
API: gst_byte_writer_put_int24_le_unchecked
API: gst_byte_writer_put_int32_be_unchecked
API: gst_byte_writer_put_int32_le_unchecked
API: gst_byte_writer_put_int64_be_unchecked
API: gst_byte_writer_put_int64_le_unchecked
API: gst_byte_writer_put_uint8_unchecked
API: gst_byte_writer_put_uint16_be_unchecked
API: gst_byte_writer_put_uint16_le_unchecked
API: gst_byte_writer_put_uint24_be_unchecked
API: gst_byte_writer_put_uint24_le_unchecked
API: gst_byte_writer_put_uint32_be_unchecked
API: gst_byte_writer_put_uint32_le_unchecked
API: gst_byte_writer_put_uint64_be_unchecked
API: gst_byte_writer_put_uint64_le_unchecked
API: gst_byte_writer_put_float32_be_unchecked
API: gst_byte_writer_put_float32_le_unchecked
API: gst_byte_writer_put_float64_be_unchecked
API: gst_byte_writer_put_float64_le_unchecked
API: gst_byte_writer_put_data_unchecked
API: gst_byte_writer_fill_unchecked

controller, dataprotocol: make public enum _get_type() functions thread-safe

Not that it is likely to matter in practice, but since these are public
API they should probably be thread-safe.

dataprotocol, lfocontrolsource: fix enum value name in enums that are public API

So run-time bindings can introspect the names correctly (we abuse this
field as description field only in elements, not for public API
(where the description belongs into the gtk-doc chunk).

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

bytewriter: Fix possible infinite loop caused by an overflow

elements: minor performance improvement when doing g_object_notify() for the "last-message" property

Make sure property names passed to g_object_notify() are in the canonical form
(ie. "last-message" not "last_message"), so that g_param_spec_pool_lookup()
doesn't have to do strdup/canonicalize/free for every single notify call.
This only applies when building against older GLib versions (< 2.26).

fakesink: use g_object_notify_by_pspec() and remove work-around for old GLib versions if possible

Use more efficient g_object_notify_by_pspec() if we're compiling against
GLib >= 2.26, and also remove work-around for g_object_notify() thread-
safety issues with older GLib versions if it's not needed any more.

tee: use g_object_notify_by_pspec() if possible

Use more efficient g_object_notify_by_pspec() if we're compiling against
GLib >= 2.26.

fakesrc: use g_object_notify_by_pspec() if possible

Use more efficient g_object_notify_by_pspec() if we're compiling against
GLib >= 2.26.

identity: use g_object_notify_by_pspec() and remove work-around for old GLib versions if possible

Use more efficient g_object_notify_by_pspec() if we're compiling against
GLib >= 2.26, and also remove work-around for g_object_notify() thread-
safety issues with older GLib versions if it's not needed any more.

pads: use new g_object_notify_by_pspec() for caps notifies if available

If we're building against GLib >= 2.26.0, we can use the more efficient
g_object_notify_by_caps(), which avoids the param spec lookup.