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

identity: fix potential buffer overflow

Coverity 1037155

downloadbuffer: reset read and write positions

Reset the read and write positions right after we open the file or flush
it. We are also in the buffering state with 0 percent buffered when we
start.

info: first handle all miniobjects, then GObjects

First handle all miniobjects before we attempt to dereference the first
field pointer and look at the GType. With the recent glib change to
speed up G_IS_OBJECT, this causes crashes on miniobjects otherwise.

info: GstDateTime does not have a GType as first field

GstDateTime does not have the GType as the first field so we can't use
it to detect its type.

info: use macros to check types

Use the macros to check the type of objects instead of directly poking
at the first field.

globaldevicemonitor: connect sync-message signal on the right object

Fixes criticals at runtime and makes stuff actually work.

typefind: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

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

Revert "miniobject: Add missing (nullable) annotations"

This reverts commit 96361e9b5c5d00dc7712ff3a9acfbe10df7cd9fe.

This was not supposed to be pushed yet!

bufferpool: It's pool, not poo... even when talking about flushing

miniobject: Add missing (nullable) annotations

gst_mini_object_replace() can take NULL mini-objects.

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

tests: multiqueue: fix leaks

docs: convert NULL, TRUE, and FALSE to %NULL, %TRUE, and %FALSE

This should help improve documentation generated for
languages other than C.

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

docs: fix type in GstObject docs

bufferpool: fix gst_buffer_pool_has_option() documentation

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

element: set pads need-parent flag to false when removing

When a pad is added the need-parent flag is set to true, so when
they are removed the flag should be set back to false

This was preventing GstPads to be reused in elements (removed and
later re-added). A unit tests was added to verify that this is
working now.

The use case is tsdemux that has a program-number property and
allows the user to switch programs. In order to do that tsdemux
will remove the pads of the current program and add from the new
ones. The removed pads are kept in the demuxer for later if the
user selects the old program again.

multiqueue: post buffering message when queues flush

The buffering status goes back to 0, so inform the application about it

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

gitignore: Ignore VIM swap files

pad: two minor docs fixes

flowcombiner: beautify headers a little

docs: add GstFlowCombiner

base: include flowcombiner header from base.h

flowcombiner: add GstFlowCombiner

Adds a utility struct that is capable of storing and aggregating flow returns
associated with pads.

This way all demuxers will have a standard function to use and have the
same expected results.

Includes tests.

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

pad: store last flow return and provide acessor function

Stores the last result of a gst_pad_push or a pull on the GstPad and provides
a getter and a macro to access this field.

Whenever the pad is inactive it is set to FLUSHING

API: gst_pad_get_last_flow_return

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

bufferpool: Add method and virtuals to set flushing state

Currently there is no other way to unlock a buffer pool other then
stopping it. This may have the effect of freeing all the buffers,
which is too heavy for a seek. This patch add a method to enter and
leave flushing state. As a convenience, flush_start/flush_stop
virtual are added so pool implementation can also unblock their own
internal poll atomically with the rest of the pool.  This is fully
backward compatible with doing stop/start to actually flush the pool
(as being done in GstBaseSrc).

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

basetransform: Passthrough ALLOCATION queries in passthrough mode even if we had no caps yet

Or if the element does not care about caps at all.

Also remove an assigned but unused local variable.

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

po: update

po: update POTFILES

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

Back to development

Release 1.3.2

Update .po files

Automatic update of common submodule

From 211fa5f to 1f5d3c3

value: Add some positive testcase for string deserialization

docs: remove reference to Mandrake and packages we no longer provide

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

docs: fix typo

pluginloader: fix compiler warning on windows

gstpluginloader.c:584:1: error: label 'beach' defined but not used

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

elements: don't depend on libgio just for g_io_error_from_errno()

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

sparsefile: keep it private as helper API for downloadbuffer

There's no expectation that any other element or applications
might want to use this helper API any time soon, so keep it
private for the time being. There were open questions regarding
portability and binding-friendliness too.

This also removes the gio dependency of -base again.

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

docs: pick up GstBaseParse hierarchy and properties

docs: expose GstPushSrcClass in documentation

Might come in handy in case someone wants to derive from it.

pluginloader: Don't leak pluginloader in error cases

CID #1212154

caps: Don't leak features on error cases

If we fail to parse fields, we would end up leaking the features we
parsed just before

CID #1212152

basetransform: Correctly reset configuration

When pool can't we use, and we fall back to default pool, we need to
correctly reset that pool configuration.

nettimeprovider: Use non-freed variable

address is only used temporarily. Use the proper variable instead.

CID #1212189