meson: po: use glib preset and read language list from LINGUAS

Supported since meson 0.37, so we can use it now.

miniobject: don't modify memory if it's clearly not a valid miniobject

Add back function guard that checks the refcount in a read-only
operation first, and bail out without modifying the passed-in
memory if it's clearly not a valid mini object. Otherwise we
probably cause more harm than good. We keep the second sanity
check based on the 'real refcount' at the time of the unref
around for now too.

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

basesrc: deprecate non-functional "typefind" property

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

benchmark: fix complexity benchmark

Make complexity benchmark code work for complexity > 1

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

win32: find plugin scanner in libexecdir subdir as configured

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

basesink: fix buffer leaks if preroll failed

buffer is not unreferened if preroll failed

:Detailed Notes:
- Problem : video freeze when switching from pause to 1/2-FF repeatedly
- RootCause : buffer leaks in basesink
- Solution : unref the buffer if prerolled failed

:Testing Preformed:
How to Test :
pause -> 1/2 FF -> resume -> pause -> 1/2 FF ...

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

Generate a gstreamer pipeline diagram on SIGHUP.

Useful for debugging a pipeline that refuses to enter a given state.

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

basesrc: forward SINK_MESSAGE events downstream

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

meson: Fix bashcomp installation

For some reason the double quotes ended up in the path, hence
the helper where installed at "."/share/...

meson: Use join_paths and .set_quoted where possible

meson: pkgconfig: add libunwind/libdw to gstreamer-1.0 Requires.private

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

pkgconfig: Add private requirements

Add libunwind and dw to the .pc Requires.private. Fixes static library
compilation if gstreamer was compiled with one of those dependencies

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

basesink: use GST_CLOCK_TIME macros for readability

Replace some -1 comparison with GST_CLOCK_TIME macros.

collectpads: correct some comments and add more logging

Add more logging to analyze event handling (especially failure cases).

tools: gst-stats: Use standard character escapes

Having '\e' expand to '\x1b' is a gnu extension. I didn't see any
document describing the behavior, but gcc also seems to expand
'\[' to '['.

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

meson: find python3 via python3 module

And rename python3 variable in meson build files for clarity.

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

meson: add option to disable tracer hooks

tests: memory: skip test that depends on debug system if it's disabled

tracing: fix build with tracer hooks disabled

miniobject: make refcount tracing and debug logging reliable

Tracing of the refcounts wasn't thread-safe, and log output of
the refcount values before/after wasn't reliable.

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

plugin: clarify code that deduces plugin name from file name

Make the final else branch explicit for clarity.

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

docs: add a missing const in bytereader docs

This syncs the prototype with gstbytereader.h

basesrc: Removed unused private member qos_enabled

basesrc: Don't reallocate buffers when flushing

Instead of using gst_buffer_pool_set_active() when flushing, use
gst_buffer_pool_set_flushing(), this avoids uneeded reallocation of the
buffers.

basesrc: Don't hold LIVE_LOCK in create/alloc/fill

Holding this lock on live source prevents the source from changing
the caps in ::create() without risking a deadlock. This has consequences
as the LIVE_LOCK was replacing the STREAM_LOCK in many situation. As a
side effect:

- We no longer need to unlock when doing play/pause as the LIVE_LOCK
  isn't held. We then let the create() call finish, but will block if
  the state have changed meanwhile. This has the benefit that
  wait_preroll() calls in subclass is no longer needed.
- We no longer need to change the state to unlock, simplifying the
  set_flushing() interface
- We need different handling for EOS depending if we are in push or pull
  mode.

This patch also document the locking of each private class member and
the locking order.

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

basesrc: Protect access to pool and allocator

This was only partly protected by the object lock. Always take the
object lock to access the currently configured pool and allocator.

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

gst-inspect: Fix memory leak in print_pad_templates_info

gst_static_caps_get function returned allocated memory.
So, It should be free using gst_caps_unref.

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

meson: check for ppoll() as well

meson: Add configinc as include_directory in tools/

Otherwise when the glib is used as a subproject config.h is the glib
one, not ours.

meson: Allow using glib as a subproject

protection: add function to filter system ids

gst_protection_filter_systems_by_available_decryptors() takes an array
of strings and returns a new array of strings filtered by the available
decryptors for them so the ones you get are the ones that you should be
able to decrypt.

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

meson: fix with-package-name option

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

gstplugin: remove gst prefix when loading plugin on MSVC

When building with Meson and MSVC, our plugins don't have a 'libgst'
suffix and are just 'gstfoo.dll', so look for that too.

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

baseparse: sinkcaps can be NULL in default caps negotiation

This was causing harmless assertion about the unreffed caps not being of
type caps.

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

win32: Update .def file

utils: Add a function to get a string representation of GstStateChange

API:
    gst_state_change_get_name

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

element: Add missing values for "to same" state changes

And handle newly added GstStateChange values in GstPipeline

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

utils: Fix leak in failed case of regression overflow checking

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

registrychunk: Fix leak in failed case of reading plugin dependency string

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

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

meson: fix tests build with --werror

Need to pass -DGST_DISABLE_DEPRECATED to avoid warnings when
testing deprecated API such as gst_uri_construct().

Also remove #ifndef GST_DISABLE_DEPRECATED guard from header
file, we don't use those any more for functions, the
GST_DEPRECATED_FOR macro is enough.

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.

I have deleted the escape_string function since it is no longer being
used and have created a unit test for the function. I have also
deprecated this function in favour of the GstUri API.

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

meson: Do not use path separator in test names

Avoiding warnings like:

    WARNING: Target "elements/audioamplify" has a path separator in its name.

bin: Put correct annotations on gst_bin_add() too

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.

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