debugutils: Unref pad template after use

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

gst: init taglist gtype to use it in gstvalue

Otherwise it will have a 0 value and GstTagList won't be found
for GstValue functions (serialization/deserialization)

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

Release 1.3.91

Update .po files

po: Update translations

docs: Fix documentation typos and inconsistencies

 * GstGlobalDeviceMonitor was renamed to GstDeviceMonitor
 * Expand GST_MESSAGE_DEVICE to the full enum value names
 * Correct the incorrect references to the GstDeviceProvider interfaces
 * Describe caps arguments for gstcheck interface
 * Add missing docs for GstNetAddressMeta and its add function
 * Add docs for toc helper macros
 * Avoid refering to GstValueList type as done elsewhere

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

docs: Cleanup interface references in docs

 * Delete references to removed interfaces
 * Add missing documentation sections
 * Fix duplicate interface references for GstDevice

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

docs: There is no decodebin2 anymore, don't pretend otherwise

fdsrc: fix error setting when uri is invalid

Elements should always set the GError

libs: gstcheck: check that mutex is locked before g_cond_wait*() is called

Sanity check to catch problems in unit test.

libs: gstcheck: init and clear global mutex and cond variables

tests: fix locking in gstpoll unit test

The mutex needs to be locked when g_cond_wait*() is
called.

gst-uninstalled: add video and base library paths from -bad

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

tools: suppress GLib warnings when gst-inspecting deprecated properties

GLib in git will spew a g_warning() when a property marked as
deprecated via param spec flags is accessed. Suppress this by
setting the appropriate environment variable.

message: Work around g-i/pygobject/gjs bug with ~0 in enums

GST_MESSAGE_ANY was considered a long by pygobject and gjs, and thus
couldn't be used in gst_bus_poll() and similar APIs as they expect an
int-typed enum.

Just use 0xffffffff instead for now.

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

tests: don't use post-GLib 2.32 API in bufferlist test

g_ptr_array_insert() is GLib >= 2.40

pad: Don't unlock while iterating over all sticky events for removal

Otherwise we might end up getting the event removed from elsewhere
at the same time while we're unlocked for g_object_notify().

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

identity: Proxy the accept-caps query

We always work in passthrough mode so there's no point in doing
something more clever in basetransform. Also the basetransform
code leads to problems with incomplete caps and downstream
elements that use GST_PAD_FLAG_ACCEPT_INTERSECT.

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

basesink: reset QoS on segment event

This avoids spurious warnings about slow machine when upstream
sends new segments without flushing.

introspection: Assorted minor introspection and documentation fixes

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

devicemonitor: Stop using g_clear_pointer()

We dont't want to depend on GLib 2.34 for now.

sparsefile: Initialize memory in unit test to make valgrind happy

We were writing unitialized stack memory to the file.

Release 1.3.90

Update .po files

devicemonitor: don't fail when started without any filters

Just show all devices then.

deviceproviderfactory: handle NULL classes argument and match any

device: Add unit tests

devicemonitor: Improve documentation

devicemonitor: Make it possible to add multiple filters

Each filter will include a GstCaps and a set of classes to match

device: Add pre-conditions

GstDeviceMonitor: Rename from GstGlobalDeviceMonitor

DeviceProvider: Rename from DeviceMonitor

dataqueue: Hide from bindings

Other languages have their own data structures that are more convenient to
use.

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

queuearray: Hide from bindings

Other languages have their own data structures that are more convenient to use.

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

miniobject: Add missing (nullable) annotations

gst_mini_object_replace() can take NULL mini-objects.

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

message: Application and element messages should not have NULL structures

It does not make sense for them.

introspection: add some missing allow-none annotations to in params

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

introspection: add nullability annotations to out and inout params

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

introspection: add missing (nullable) annotations to return values

Support for (nullable) was added to G-I at the same time as nullable
return values.  Previous versions of G-I will not mark return values as
nullable, even when an (allow-none) annotation is present, so it is
not necessary to add (allow-none) annotations for compatibility with
older versions of G-I.

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

base: assorted introspection fixes and additions

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

base: add (nullable) annotations to return values

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

tests: add another buffer list test case

tests: port and re-enable buffer list tests

And remove some which don't apply any more.

tests: enhance the gstcaps test_features to also test gst_caps_set_features()

Compliments my previous patch for gst_caps_set_features, which would
previously assert and leak the old GstCapsFeatures if the caps already
had a GstCapsFeatures and you were trying to replace it with a new one.

caps: unset the parent refcount of the old features before freeing them in gst_caps_set_features()

Otherwise gst_caps_features_free() asserts and the features structure is leaked

baseparse: avoid returning _OK for _NOT_LINKED

When the parser receives non-aligned packets it can push a buffer
and get a not-linked return while still leaving some data still to
be parsed. This remaining data will not form a complete frame and
the subclass likely returns _OK and baseparse would take that
as the return, while it the element is actually not-linked.

This patch fixes this by storing the last flow-return from a push
and using that if a parsing operation doesn't result in data being
flushed or skipped.

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

elements: fix copyright and remove gtk-doc chunk

Trivial as it may be, this code was mostly copied from
somewhere else. The gtk-doc chunk is not needed, since
it's not public API.

filesrc: Ignore seek error on non-seekable files

This make it works with FIFOs.

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

Back to development

Release 1.3.3

Update .po files

po: Update translations

tests: add unit test for gst_caps_is_any() and _is_empty()

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

caps: gst_caps_is_any() should return TRUE or FALSE

Not some flag value instead of TRUE. Fixes code like
gst_caps_is_any() == TRUE.

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

device: rename "klass" and get_klass() to "device-class" and _get_device_class()

There's some precedent in GstElementFactory, but a
"klass" property just seems weird.

benchmarks: capsnego: add --loops command line option

And default to 50 loops.

benchmark: capsnego: use GOptionContext for option parsing

tests: fix compiler warnings in gstvalue tests

Calling GST_VALUE_HOLDS_*(&v) now results in a compiler
warning about value!=NULL always being false, so check
type directly in those cases.

value: simplify GST_VALUE_HOLDS for our boxed and fundamental types

Boxed types can't be derived from, and we don't support
deriving from our special fundamental types (the code
checks for GType equality in most places.

GstDevice: Document GstDevice and related classes

Fix funnel EOS handling and wrong unittest

When no data is coming from sinkpads and eos events
arrived at one of the sinkpad, funnel forwards the EOS
event to downstream. It forwards the EOS because lastsink pad
is NULL. Also the unit testcase of the funnel is not checking
the correct behavior as it should. The unit test case should
fail if one of the sink pad has already EOS present on it and
we are trying to push one more EOS.

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

gstvalue: optimise checks for lists

Our fundamental types are non-derivable, so we can
just check for equality. Also avoid doing the same
check multiple times in a couple of places.

gstvalue: use g_assert() in internal function for already-checked things

So these get compiled out for releases.

gstvalue: add internal _can_compare_unchecked()

gstvalue: add internal _list_concat() that takes ownership of input values

Avoids unnecessary copies.

structure: simplify value type checks in getters

Just check for GType equality in common cases.

value: Add a FIXME 2.0 for a fraction ranges optimization

Currently we leak the internal representation of them as two GValues that
contain a fraction. Without this we could store fraction ranges as
  data[0] = (min_n << 32) | (min_d)
  data[1] = (max_n << 32) | (max_d)
and wouldn't require an additional allocation per range.

value: Make sure to cast int range values to guints before storing them

Otherwise negative values will sets all of the 64 bits due to two's
complement's definition of negative values.

Also add a test for negative int ranges.

win32: update exports

value: Store integer ranges directly in a GValue without additional allocation

Micro optimization to save some allocations. Next step to do this
with fraction ranges too.

gst_private: Fix duplicate definition

gst: Store more basic type GTypes in variables

Micro optimization to change a function call to a variable access
for all our basic types.

value: Store our fundamental type GTypes in variables

Micro optimization to change a function call to a variable access
for all our basic types.

gstvalue: Speed up gst_value_intersect/_subtract

Both gst_value_intersect and gst_value_subtract will call
gst_value_compare if one of their arguments isn't a list.

gst_value_compare will then re-do a check to see if one of
the arguments is a list (for the special case of comparing a unitary
value with a list of length 1).

The problem is that the various G_VALUE_HOLDS represent an expensive
amount of calling gst_value_compare (almost half of it) to see if
the provided arguments are list. These checks can be done without
when we know that the arguments aren't lists.

* Create a new "nolist" gst_value_compare which avoids that special
  case comparision

Benchmarks:
 valgrind/callgrind: average speedup in instruction calls for
 gst_value_intersect and gst_value_subtract is around 56% (Makes 63%
 of the calls it used to take previously)

 tests/benchmarks/capsnego: With default settings (depth 4, children 3
 607 elements), time taken for transition from READY to PAUSED:
   Before : 00.391519153
   After  : 00.220397492
    56% of the time previously used, +77% speedup

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

tests: remove some cruft from the bufferlist test

Buffers no longer carry caps, and bufferlists don't have
groups where buffers may need to be merged into one any more.

tests: add test for gst_buffer_list_remove()

bufferlist: fix buffer leak in _remove()

flowcombiner: fix g-i transfer annotations

flowcombiner: Fixed GBoxedCopyFunc

I'll just quote the most interesting man in the world:

"I don't usually push commits, but when I do I don't compile it
first"

devicemonitor: some docs additions and fixes

win32: add exports for new get_type() function

flowcombiner: keep a ref to the pads we're using

Needed for use via the boxed type.

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

flowcombiner: add boxed type for bindings

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

introspection: minor annotation additions

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

introspection: include gstversion.h in GIR generation

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

bytereader: Use concistant derefence method

This is minor style fix to not mix *var and var[N].

bytereader: Use pointer instead of index access

Currently the scan uses Boyer-moore method and its performance is good.
but, it can be optimized from an implementation of view.

The original scan code is implemented by byte array and index-based access.
In _scan_for_start_code(), the index is increasing from start to end and the
base address of the byte array is referred to as return value.

In the case, index-based access can be replaced by pointer access, which
improve the performance by removing index-related operations.

Its performace is enhanced by approximately 8% on arm-based embedded devices.
Although it seems trivial, it can affect the overall performance because the
_scan_for_start_code() function is very often called when H.264/H.265 video is
played.

In addition, the technique can apply for all architectures and it is good in
view of readability and maintainability.

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

globaldevicemonitor: prettify header

tests: add unit test for queuearray expansion from 1

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

queuearray: fix expanding size of queue from 1

Without we would not actually expand and access
memory beyond the allocated region for the array.

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

dataqueue: clear up documentation of gst_data_queue_new

The gpointer argument is passed to all three callbacks, not just one.

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

introspection: fix some minor annotation bugs

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

base: use correct syntax in documentation more consistently

Previously, many constants were prefixed with # or unprefixed,
some functions and macros were prefixed with # instead of suffixed
with (), etc.

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

baseparse: Pass rate of input segment to output segment

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

gstbuffer: factor three flags-to-string loops

info: make printing datetimes work with GST_PTR_FORMAT

datetime: change internal implementation to mini object

And move type stuff from GstValue to GstDateTime.

downloadbuffer: fix uninitialized variable

downloadbuffer: improve start/stop in buffering query

The start and stop should represent the currently downloading region.
The estimated-total should represent the remaining time to download
the currently downloading region. This makes it a lot more useful
for applications because they can then use those values to update
the fill region and use the estimated time to delay playback.
Update the docs with this clarification.

identity: add static and const where appropriate