controller: Export boxed type copy/free functions for GstControlPoint

base: Export boxed type copy/free functions for the remaining types

ptp: Unref timeout GSource for delay requests

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

gsturi: Fixed incorrect escaping of path as a generic string

The gst_uri_construct function was escaping the location string
as a generic uri string. This is incorrect since the slash('/')
characters are reserved for use in this exact case. The patch
changes the escape_string function mode to handle the path correctly.

In 1.14 this function will be deprecated.

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

basesrc: Hold object lock while updating latency

Otherwise in gst_base_src_query_latency(), it ended up
sometimes thinking it wasn't -1 when it was actually.

object: Add missing annotations to get_value_array() / get_value_g_array()

Same as already used in GstControlBinding.

parse: Make gst_parse_context_copy() public for bindings

debugutils: add missing E character to the legend

The E character on pads indicates the presence of the EOS flag.

tools: dist new gst-stats man page

Fixes meson build from tarball.

meson: fix gstprintf test linking

Must link against gstprint helper lib to use private symbol.

meson: add options to set package name and origin

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

Release 1.12.0

Update .po files

message: Don't pass a NULL debug string to g_utf8_validate()

g_utf8_validate() crashes on NULL, but NULL is valid for the debug
string nonetheless.

adapter: Check if meta transform_func is NULL before using it

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

basetransform: Check if meta transform_func is NULL before using it

An untested pointer segfaulted in webkit while playing video
on imx6 sabrelite. It turned out that the imx plugin didn't
implement the meta transform function.

The following GST_DEBUG trace was visible:
gstbasetransform.c:1779:foreach_metadata:<conv2> copy metadata
                                                GstImxVpuBufferMetaAPI

Thread 26 vqueue:src received signal SIGSEGV, Segmentation fault.

(gdb) bt
 0x00000000 in ?? ()
 0x73f8d7d8 in foreach_metadata (inbuf=0xc9b020, meta=0x474b2490,
                  user_data=<optimized out>) at gstbasetransform.c:1781
 0x73eb3ea8 in gst_buffer_foreach_meta (buffer=buffer@entry=0xc9b020,
                  func=0x73f8d705 <foreach_metadata>,
                  user_data=user_data@entry=0x474b24d4)
                  at gstbuffer.c:2234

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

gst: check non-null before dereference

It is possible to use gst_deinit() without registering the base
classes. For example, when using gst_init_get_option_group() and
call the program with an invalid parameter. In that case,
gst_deinit() will lead to a segmentation fault, since there is a
dereference to a pointer that is null.

This patch validates if the type is non-null before dereferencing
it.

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

gst-uninstalled: add opencv to bad libs

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

Release 1.11.91

Update .po files

po: Update translations

Automatic update of common submodule

From 60aeef6 to 48a5d85

Meson: also build and install gst-stats-1.0 and it's man page.

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

Add very simple man page for gst-stats.

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

doc: Fix some doctsing making GI happy

doc: Fix GstChangeState docstring

info: Move debug output in for_each_threshold_by_entry() to TRACE level

It's otherwise appearing many, many times in logs and usually is nothing
you're interested in.

tests: fix message leak in seek test

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

basetransform: fix typo in debug log output

This unbalanced closing parenthesis is leftover from the commit
8b739d91e7. It used to wrap the caps but we don't seem to do that in
the current code.

So, just remove it. No functionality has been changed.

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

meson: A couple for GIR-generation fixes

meson: Add controller enum file as generated sources

Avoiding build failure like
https://ci.appveyor.com/project/thiblahute/gst-build-ge9m5/build/1.0.2857

meson: dist new meson build script

controller: include new proxycontrolbinding header

And fix includes in docs to just include the main header (the
include for the proxycontrolbinding was wrong in the docs).

.gitignore: ignore more

meson: Only check the first parameter

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

meson: Check bison and flex are recent enough

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

Automatic update of common submodule

From 39ac2f5 to 60aeef6

value: Accept NULL as a structure

Some GstStructure properties default to NULL, so it should
be a supported value.

With unit test.

controller: Fix build with srcdir!=builddir

controller: Add missing sources/headers to the GIR build

meson: And actually make the controller mkenums rules work

meson: Add check for glib-mkenums

controller: Generate GLib enums automatically

Don't use deprecated g_object_newv()

Use g_object_new() instead which nowadays has a shortcut for the
no-properties check. It still does an extra GType check in the
function guard, but there's a pending patch to remove that
and it's hardly going to be a performance issue in practice,
even less so on a system that's compiled without run-time checks.

Alternative would be to move to the new g_object_new_properties()
with a fallback define for older glib versions, but it makes the
code look more unwieldy and doesn't seem worth it.

Fixes deprecation warnings when building against newer GLib versions.

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

test: Add test for serializing/deserializing NULL strings

Release 1.11.90

Update .po files

structure: Don't print warning on NULL strings or pointers

Putting NULL for those is a valid serialization for the NULL value.

gst: Use GstClockTimeDiff instead of GstClockTime as type for GST_SECOND and friends

They were (signed!) gint64 before because of G_GINT64_CONSTANT() already
and they are actually used in signed calculations.

With this change we at least ensure that an integer type of the correct
size is used for GI (it was using gint before).

gst: Update some more types and values of combined-flags constants

gst: Cast combined-flags constants to their respective target types

This makes C++ compilers a bit more happy without having the user of the
constants cast. It also provides the correct type information to GI.

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

gst: Set values and types for combined-flags constants in GI annotations

value: Properly set value and type for GST_FLAG_SET_MASK_EXACT in GI annotations

Storing a -1 inside an unsigned integer confuses GIR based bindings
generators.

completion: Try to avoid parsing summary

In GES, the summary refers to options that are only available when built
against gst-valdiate. Those where picked by our regex. This patch add a
initial grep to try and filter-out as best as possible the content to
which we will extract the command list.

meson: Add bash completion support

completion: Place the completion helper in libexec

This patch reorganize the bash completion scripts in order to install
the binary helper (gst-completion-helper) in libexec path rather then
share folder. Most Linux hierarchy compliance requires that no binary
executable are placed in share. We also cleanup the unused .pc entries
and remove copy pasted parts of the script. Note that other project
including the common helper, should now use $_GST_HELPER to read
the binary executable gst-completion-helper. This helper is not longer
version, as it is placed in a versionned subfolder
(libexec/gstreamer.10) just like the other helpers (scanner and ptp).

gstvalue: fix GstValue leak in structure_field_union_into

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

streamcollection: fix racy user-after-free

The issue happens when the structure is printed by the logging
subsystem: the object is included in the log, and this will cause the
full object printout to be done there. However, after dispose, the queue
was already cleared, so the access to it (to print the object) would
assert, as the queue was already freed. The patch changes it so that the
queue is merely empty, and only freed in _finalize.

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

debugutils: add pad EOS flag in pipeline dot file dumps

But only if set. Helps debug issues with EOS propagation.

tools: gst-launch: print structure property notifies nicer

One less layer of escaping, but still lots of ugly \.

Fix win32 libgstreamer.def ordering

This should fix make distcheck

paramspec: Fix array validation logic

A paramspec validation should modify the content to match what the spec
requires and return TURE if a modification happened. This previous
implementation would only fix the first element of the array and return.
It was also return TRUE for empty array, while no modification was
needed.

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

array/fraction: In param types, use get_type() function directly

The GST_TYPE macro points to global variables initialized by the
first call to get_type. This is not an issue if you call gst_init()
but unfortunatly pygi will need to acces the param type before
init can be called. This removes an assertion.

fraction/array: Make get_type() thread safe

Those aren't suppose to be called from multiple thread, but all
fundamental get_type() function are thread safe. Fix it to
be consistent and it may help if we change the typing mechanism
in GStreamer come day.

gstutils: Add helpers to get/set array properties

This is to help bindings access properties of type GST_TYPE_ARRAY.
This function will get/set the property and convert form/to
GValueArray.

New API:
  gst_util_set_object_array
  gst_util_get_object_array

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

structure: Add get/set_array/list using GValueArray

This adds a binding friendly interface to get and set arrays
and list into GstStructure.

New API:
 - gst_structure_set_array
 - gst_structure_set_list
 - gst_structure_get_array
 - gst_structure_get_list

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

gstvalue: Add transformation to/from GValueArray

This allow transforming a GValue of type G_TYPE_VALUE_ARRAY to
and from GST_TYPE_ARRAY/LIST.

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

queuearray: Add G_BEGIN_DECLS and G_END_DECLS to make it usable from C++ code

paramspecs: Use gst_value_array_get_type() for GstParamSpecArray type

When registering GstParamSpecArray, use the gst_value_array_get_type()
function to get the type, rather than the GST_TYPE_ARRAY macro, which
gets it from the _gst_value_array_type, which is in turn only
initialised during gst_init()

Fixes criticals with (python) bindings that look up all the
types from the gobject-introspection info as soon as they
are imported.

/usr/lib64/python3.5/site-packages/gi/module.py:178: Warning: g_param_type_register_static: assertion 'g_type_name (pspec_info->value_type) != NULL' failed
  g_type = info.get_g_type()
/usr/lib64/python3.5/site-packages/gi/module.py:212: Warning: g_type_get_qdata: assertion 'node != NULL' failed
  type_ = g_type.pytype
/usr/lib64/python3.5/site-packages/gi/module.py:226: Warning: g_type_get_qdata: assertion 'node != NULL' failed
  g_type.pytype = wrapper
/usr/lib64/python3.5/site-packages/gi/module.py:226: Warning: g_type_set_qdata: assertion 'node != NULL' failed
  g_type.pytype = wrapper

baseparse: Don't forget error returns when processing more

If parsing returns a non-OK flow return in the middle
of processing an input buffer, don't overwrite that
if a later return is OK again - the subclass might
return not-linked in the middle, and then discard
subsequent data without pushing while returning OK.

A later success doesn't invalidate the earlier failure,
but we should continue processing after not-linked, so
as to keep parse state consistent.

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

tools: replace mentions of 'mad' on the gst-launch-1.0 man page

The 'mad' plugin has been removed. Mention mpg123audiodec instead.

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

value: fix union of int range and int when extending on a side

The internal representation uses bounds scaled by the step

Add tests to catch those cases

queue: avoid return flushing if we have a not-linked

Return the correct flow return instead of returning always flushing.
This would cause queue to convert not-linked to flushing and making
upstream elements stop.

Based on the previous patch for queue2.

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

queue2: avoid return flushing if we have a not-linked

Return the correct flow return instead of returning always flushing.
This would cause queue2 to convert not-linked to flushing and making
upstream elements stop.

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

tests: bin: add more tests for suppressed flags

Add tests to confirm flags are persisted even after removing
elements that have those suppressed flags

buffer: handle gst_buffer_map failures

downloadbuffer: unlock mutex in error case

gstvalue: Do more checks when guessing at flagset strings

If guessing that a string matches a flagset, be more thorough
at checking that the string following a string of hex:hex:
actually looks like a flag set string. Add some unit tests
to catch more cases.

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

multiqueue: Make min-interleave-time a configurable property

Remove a FIXME about making the minimum interleave
buffering a configurable property

elementfactory: promote factory not found log message to WARNING

In most cases people really want to know when an element
could not be created.

inputselector: Always proxy position/duration query

active-pad switch causes reconfigure event with lock taken,
and upstream element might query the current position or duration
before returning the reconfigure event.
Meanwhile, gst_input_selector_get_linked_pad() is used to get srcpad
inside of default query handle, and it takes also lock.
Since inputselector is still locked by active-pad switch, and so the query
cannot be handled further.

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

info: document that logging macros don't need newlines at the end

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

pad: add since marker to docs for new API

win32: update .def file for new API

registry: Only scan plugin files that end with an extension

Not file that would for some reason end with 'so' or 'dll', etc...

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

pad: Add API to get the current state of a task

Avoiding the user to need to deal with the locking himself etc.

API:
  gst_pad_task_get_state

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

meson: Add an option to disable usage of libunwind

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

meson: Update version

Back to development

Release 1.11.2

Update .po files

value: Add deserialization for arrays/lists outside GstStructures

This is mostly useful for properties of those types when used in
gst-launch or similar.

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

value: Add a type abbreviation for GstFlagSet in serialization

value: Always add the type name to elements when serializing arrays/lists

But only when serializing outside of GstStructures, because in case of
GstStructure the type is already preprended to the array/list and the
GstStructure API makes sure that they have the same "generic" type so
deserialization works properly.

This keeps serialization of GstStructures the same as before, and the
GstCaps unit tests already test for that. However when serializing
standalone arrays/lists get the types added now.

value: Move list/array serialization/deserialization functions from GstStructure to GstValue

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

paramspecs: Add GstParamSpecArray for GST_TYPE_ARRAY typed properties

These are mostly useful to get our automatic
serialization/deserialization from strings and simple usage from
gst-launch or similar.

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

bytereader: fix peek value when scanning for 00 00 01 with non-0 offset

We would add the offset a second time in _scan_for_start_code()
when we found a result, but it's already been added to the data
pointer at the beginning of _masked_scan_uint32_peek(), so the
peeked value would be wrong if the initial offset was >0, and
we would potentially read memory out-of-bounds.

Add unit test for all of this.

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

info: put () around macro arguments

Put braces around macro arguments or else we might run into problems
with operater precedence.

deviceproviderfactory: ignore empty classes

deviceproviderfactory: compare class against NULL

gstdeviceproviderfactory.c:501:20: error: comparison between pointer and zero character constant [-Werror=pointer-compare]
     if (classes[0] == '\0')