meson: actually remove config.h.meson as well

protection: Fix annotations for gst_protection_select_system()

plugin: Stop plugin symbol name at first .

This is because the python plugin ends up named
libgstpython.cpython-35m-x86_64-linux-gnu.so so we need to stop
at the first dot.

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

meson: don't need config.h.meson any more

Meson does the largefile support automatically nowadays, and
can generate a config.h from configuration_data() without a
template as input.

meson: make C++ compiler optional

It's only used to build tests to see if our headers are C++ clean.

meson: check: generate .gir file

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

parse: Make gst_parse_context_copy() public for bindings

tools: dist new gst-stats man page

Fixes meson build from tarball.

Dist meson ptp helper install script

Fixes meson build from tarball.

leaks: Handle subclasses in filters even for unhandled/lazy loaded types

Using typename in the set of unhandled types instead of the quark so
that we also handle subclasses as with other filters.

debugutils: add missing E character to the legend

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

gst-plugin: allow '-' in plugin file name

'-' will be translated to underscore when determining symbol name

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

gst: ref_sink() some more floating references returned by g_object_new()

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

gst: Clear floating flag in constructor of all GstObject subclasses that are not owned by any parent

I.e. most of them unfortunately.

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

allocator: ref_sink() the global sysmem allocator after creation

It's not owned by the first one to ask for it, but by this very code.

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

gst: Don't ref_sink() GstObject subclasses in instance_init/constructor

This is something bindings can't handle and it causes leaks. Instead
move the ref_sink() to the explicit, new() constructors.

This means that abstract classes, and anything that can have subclasses,
will have to do ref_sink() in their new() function now. Specifically
this affects GstClock and GstControlSource.

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

gst: Correctly annotate functions taking floating reference parameters and returning floating references

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

gst: Handle floating references consistently

If a function takes a floating reference parameter, it should also be
sinked in error cases. Otherwise the function behaves differently
between error and normal cases, which is impossible for bindings to
handle.

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

gst: Fix floating reference inconsistencies in error cases

If a function takes a floating reference and sinks it, it should also do
that in error cases. I.e. call ref_sink() followed by unref().

Otherwise the reference counting behaviour of the function will be
different between the good and the error case, and simply inconsistent.

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

Also use default visibility for plugins symbol

Remove plugin specific static build option

Static and dynamic plugins now have the same interface. The standard
--enable-static/--enable-shared toggle are sufficient.

meson: fix gstprintf test linking

Must link against gstprint helper lib to use private symbol.

gstconfig.h: use default visibility for GST_EXPORT for gcc/clang too

This will be needed later when we switch to using -fvisibility=hidden.

gst: add GST_DEPRECATED_FOR() and also export deprecated symbols

Can't use a #ifndef GST_DISABLE_DEPRECATED guard around deprecated
functions any more, as they won't get exported then. Besides, we
get a nicer error message from the compiler telling us what function
to use instead this way.

gst: mark symbols explicitly for export with GST_EXPORT

One omission: gst_allocator_sysmem_get_type() was
exported but never in any public header file.

libs: controller: mark symbols explicitly for export with GST_EXPORT

libs: base: mark symbols explicitly for export with GST_EXPORT

libs: net: mark symbols explicitly for export with GST_EXPORT

meson: add options to set package name and origin

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

docs: use the full path to ignore dirs

docs: remove stray ',\' from doc comment

buffer: Add GstReferenceTimestampMeta

This is a meta that generically allows to attach additional reference
timestamps to a buffer, that don't have to relate to the pipeline clock
in any way.

Examples of this could be an NTP timestamp when the media was captured,
a frame counter on the capture side or the (local) UNIX timestamp when
the media was captured.

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

meson: bump meson version requirement to 0.40.1

bus: Add function to get the file descriptor of the bus

This is useful for integration with other event loops that work by
polling file descriptors. G_IO_IN will always be set whenever a message
is available currently.

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

plugin: Unify static and dynamic plugin interface

This patch changes the entry point of each plugin in order to unify the
interface for static and dynamic plugin. What we do is replace the
current static plugin interface and extend the dymamic one. The plugin
entry was a C structure, name "gst_plugin_desc". With this patch, the
interface is now:

  GstPpluginDesc *gst_plugin_<name>_get_desc(void);

The reason we change the C structure into function, is that it is
potentially more common to have function pointers, avoiding possible
binding language limitation. Additionally to that. This change prevents
the symbols from clashing between plugins, allowing to build once the
plugin (assuming you have -fPIC).

On the plugin loader side, we symply derive the shared object basename
to extract the plugin name. If this symbol is not found, we fallback to
gst_plugin_desc for backward compatibility.

This has one side effect, which is that the shared objects now need to
be named after their plugin name. This is generally the case with few
exceptions. The benifit of this limitation is that you can control the
gst_plugin_<name>_desc clash at file level.

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

meson: fix search path for setcap

debugutils: make local variable static

meson: fail silently in ptp helper post install script

.. in case permissions/capabilities could not be set.

meson: use cc.find_library()

find_library() was deprecated and removed in later versions.

meson: add gst-ptp-helper

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

meson: fix indentation

No tabs please.

helpers: remove old cruft from CLEANFILES

These files are no longer built, so no need to clean them.

g-i: no need to load registry in g-i scanner

Don't generate gst-element-check-1.0.m4 on the fly and fix meson build

This will interfere with 'git pull'. You will have to remove the
old generated gst-element-check-1.0.m4 manually if you're pulling
on a dirty build directory, sorry.

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

meson: install gst-element-check-1.0.m4

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

harness: Abort when failed to construct the specified pipeline

gst_harness_new_parse() returns without any error even if it doesn't
find the specified element.  Then a succeeding call to
gst_harness_set_sink_caps_str() causes an error like this:

Unexpected critical/warning: gst_pad_push_event: assertion 'GST_IS_PAD (pad)' failed

This is a bit cryptic and doesn't give users any clue what was going
on.

gst_harness_new_parse() calls gst_harness_add_parse() with a newly
created empty harness and the given pipeline description string, but
gst_harness_add_parse() does not have a way to propagate the error
back to the caller.  Since the function, gst_harness_add_parse(), is a
public API, it's not a good idea to change its signature.  This patch,
instead, makes the function to g_error() when it discovers any error.
With this change the same error prints:

** (myelement-test:25345): ERROR **: Unable to create pipeline 'bin.( myelement )': no element "myelement"

The current implementation of gst_parse_launch_full() doesn't return
partially constructed pipeline when GST_PARSE_FLAG_FATAL_ERRORS is
specified, however, this patch also adds a check for it.

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

Back to development

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.