codecparsers: h264: fix memory leak in GstH264PPS.

The gst_h264_parse_pps() function dynamically allocates the slice
group ids map array, so that needs to be cleared before parsing a
new PPS NAL unit again, or when it is no longer needed.

Likewise, a clean copy to the internal NAL parser state needs to be
performed so that to avoid a double-free corruption.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: fix typo in GstH264VUIParams description.

codecparsers: h264: add support for Recovery Point SEI message.

The recovery point SEI message helps a decoder in determining if the
decoding process would produce acceptable pictures for display after
the decoder initiates random access or after the encoder indicates
a broken link in the coded video sequence.

This is not used in the h264parse element, but it could help debugging.

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

codecparsers: h264: add nal_reader_skip_long() helper.

Add nal_reader_skip_long() helper function to allow an arbitrary number
of bits to be skipped. The former nal_reader_skip() function is too
limited to the actual cache size.

Use this new function to simplify gst_h264_parser_parse_sei_message()
default case, that skips unsupported payloads.

v2: made args consistent from header to source file.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

videoaggregator: fix a refcount error when keeping the buffer

We take a ref on the pad's buffer at the beginning so we need to
unref when we are done in all cases.

codecparsers: vp8: move up built-in range decoder private data.

Use the first _gst_reserved[] slot to hold the built-in range decoder
private data. The first slot was formerly the buffer size, which was
then promoted to semi-public namespace when it got integrated as git
commit 2940ac6.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

glcolorconvert: free pixel swizzling information

glmixer: remove redundant/unused lock

Use the GstObject lock instead

glmixer: remove redundant creation and free of GPtrArrays

Also plugs a memory leak

glmixer: plug a memory leak for the caps

gst_query_set_caps_result() takes a ref on the caps

aggregator: plug a memory leak of the srccaps

glmixer: intersect with the filtercaps if available

gl: update gles2 compatability header

mpegts: Unify API : GstMpegts / gst_mpegts_ / GST_*_MPEGTS_

It was previously a mix and match of both variants, introducing just too much
confusion.

The prefix are from now on:
* GstMpegts for structures and type names (and not GstMpegTs)
* gst_mpegts_ for functions (and not gst_mpeg_ts_)
* GST_MPEGTS_ for enums/flags (and not GST_MPEG_TS_)
* GST_TYPE_MPEGTS_ for types (and not GST_TYPE_MPEG_TS_)

The rationale for chosing that is:
* the namespace is shorter/direct (it's mpegts, not mpeg_ts nor mpeg-ts)
* the namespace is one word under Gst
* it's shorter (yah)

mpegtssection: Free data in error cases

Data is marked as "transfer full", if we have an error we need to
free it.

glfilter: pass through the allocation query when in passthrough mode

Otherwise two GL elements on either side will fail to use the same
GL context.

glfilter: prefer passthrough for non-sysmem caps

glshader: enable glshader on GLES2

glcolorscale: do passthrough on same caps

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

glfilter: handle NULL decide_query which means passthrough

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

glfilter: prepend intersection to src caps

Prefer to stay in the same memory space if possible.
Also it let a chance to do passthrough.

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

gl: enable glvideomixer on GLES2

dvbsrc: add copyright/license stub to header

dvbsrc: documentation fixes

glvideomixer: bas output width/height on the pad properties

Allows automatic negotiation of the size in the following case:
gst-launch-1.0 glvideomixer name=m sink_0::xpos=0 sink_1::xpos=320 ! glimagesink \
    videotestsrc ! m. \
    videotestsrc pattern=1 ! m.

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

glvideomixer: don't clobber already allocated shader

curls?ftpsink: Fix memory leaks due to new error handling

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

mpegts: More doc fixes

Still not done :(

mpegts: more annotation fixes

mpegts: change to GType and from gchar[] to gchar*

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

gl: XInitThreads when env variable is set

This is too allow gst-launch debugging with multiple GL contexts as
well as avoiding segfaulting innocent gtk+ apps that have not called
XInitThreads.

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

gl: add a scary note about initializing thread support for the winsys

We cannot do it as the winsys may crash if we initialize too late.

Example, GLX contexts with Intel drivers:
Intel requires the X Display to be the same in order to share GL
resources across GL contexts.  These GL contexts are generally
accessed from different threads.  Without winsys support it is
nearly impossible to guarentee that concurrent access will not
occur.  This concurrent access could result in crashes or abortion
by the winsys (xcb).

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

mpegts: Fix descriptor_from_dvb_service

Various leaks/overwrites issues

all: Don't declare variables in 'for' loops

badbase: Use the correct LDFLAGS for creating libraries

Otherwise we won't e.g. create .dlls on Windows

compositor: Fix Makefile CFLAGS/LIBADD ordering

We want to use the libraries from -bad if/when present

d3dvideosink: Release D3D surfaces when shutting down the sink

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

openslessrc: Classification of the element factory should be Source/ not Src/

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

videoaggregator: don't clobber already heap allocated video frame

CID # 1223440

videoaggregator: fix up the parent chaining for dispose and finalize

gluploadmeta: reenable GLMemory upload path

dvbsrc: fix diseqc-source prop name in example

dvbsrc: fix nit on warning message

configure: use $ERROR_CFLAGS instead of hard-coding -Werror

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

configure: use -Werror to make the include GL and GLES2 header check guaranteed fatal

Clang will only give a warning for the redefinition of typedef GLenum.
Since master is build with -Werror this will result in a build failure
later in the gl plugin. Add -Werror to the test, so the test result is as
expected. This will allow the gl plugins to build.

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

h265parse: fix hvcC size check

In ISO/IEC 14496-15, the minimum size of a HEVCDecoderConfigurationRecord
(i.e., the contents of a hvcC box) is 23 bytes. However, the code in h265parse
checks that the size of this data is not less than 28 bytes, and it refuses to
accept caps if the check fails. The result is that standards-conformant streams
that don't carry any parameter sets in their hvcC boxes won't play.

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

modplug: don't hardcode c++ standard library

Fixes build with llvm on FreeBSD.

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

Back to development

Release 1.3.3

Update .po files

gl: Need to link to new badvideo library for the video aggregator base class

video: Link with $(LIBM) for rint() and others

po: Update translations

hlsdemux: Fix decrypting fragments

Only reset the decryption engine on the first buffer of a fragment,
not again for the second buffer.  This fixes corrupting the second
buffer of a fragment.

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

rawparse: Use the pad passed to us directly for default event handling

rawparse: default query should be forward on target pad

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

tsdemux: fix up comment

gl: move gles2 compat header to glprototypes

glbufferpool: provide a consistent API regardless of platform

gl: mark library as API unstable

libs:uridownloader: Properly declare APIs as UNSTABLE

libs:video: Properly declare APIs as UNSTABLE

libs:base: Properly declare APIs as UNSTABLE

waylandsink: Fix compiler warning

gstwaylandsink.c:480:14: error: comparison of constant -1 with expression of
      type 'enum wl_shm_format' is always false
      [-Werror,-Wtautological-constant-out-of-range-compare]
  if (format == -1)
      ~~~~~~ ^  ~~

tsdemux: Update comments to the new reality and remove unused variables

libs:video: Add -I$(top_srcdir)/gst-libs to fix distcheck

libs:base: Fix includedir

aggregator: Fix requested pad name

libs: videoaggregato: Do not import videoconvert.h in gstvideoaggregatorpad.h

+ Add a Private structure to the GstVideoAggregatorPad
+ Add some padding

Final fix compilation

build: remove gst-libs/gst/video/ from CRUFT_DIRS

It was gone, but now it's back.

libs:video:Fix compilation

gl:glvideomixer: Add the Compositor in the element metadata class

So it is possible to pick one compositing element from the registry

compositor: Add a new compositor based on the new GstVideoAggregator base class

It is a replacement for videomixer with a similare API

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

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

gl: Port glmixer to the GstVideoAggregator baseclass

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

videoaggregator: Create a new GstVideoAggregator baseclass

This base class has been added to a newly created libgstbadvideo library

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

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

aggregator: Add new GstAggregator base class

This base class has been added to a newly created libgstbadbase library

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

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

androidmedia: Clear exceptions also when we did not provide a GError

androidmedia: Also don't report warnings for queue/dequeue errors during flushing in error cases

... and fix a case where we released an invalid buffer index.

androidmedia: Don't consider input buffer queueing failures as an error during flushing

androidmedia: Clean up flushing code and don't consider output buffer releasing failures during flushing as an error

uridownloader: Include the debug string in the error messages we propagate from t source

hlsdemux: Include the debug string in the error messages we propagate from the
source

hlsdemux: Include a more descriptive error message

hlsdemux: Propagate error messages from the source element up in the hierarchy

Instead of inventing our own generic error strings which are mostly useless.

hlsdemux: Directly convert GErrors to error messages

This will make sure that we don't leak debug information into the actual
error message string and keep it behind the debug string.

gstsrtpenc: unref event when needed

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

srtpenc: improve property comments

Add a Note to the allow-repeat-tx property because it is potentially
insecure when used incorrectly.

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

gstsrtpenc: add allow-repeat-tx property

opengl: add element for transforming video geometry

* add graphene as soft dependency for linear algebra

pnm: link against libgstbase for GstAdapter

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

glframebuffer: don't segfault needlessly with GLES2

gst_gl_framebuffer_use_v2 was using a function that is not available
with GLES2

glfiltershader: port to GLES2

Provide a time variable

win32: update exports for changed h264 codec parser API

pnmdec: Added PBM Support

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

wayland: add unstable API guards to wayland library header

gst-libs: fix uninstalled build of new wayland lib

waylandsink/wldisplay: verify that all the required interfaces have been found on the compositor

This allows waylandsink to fail gracefully before going to READY
in case one of the required interfaces does not exist. Not all
interfaces are necessary for all modes of operation, but it is
better imho to fail before going to READY if at least one feature
is not supported, than to fail and/or crash at some later point.

In the future we may want to relax this restriction and allow certain
interfaces not to be present under certain circumstances, for example
if there is an alternative similar interface available (for instance,
xdg_shell instead of wl_shell), but for now let's require them all.
Weston supports them all, which is enough for us now. Other compositors
should really implement them if they don't already. I don't like the
idea of supporting many different compositors with different sets of
interfaces implemented. wl_subcompositor, wl_shm and wl_scaler are
really essential for having a nice video sink. Enough said.

waylandsink: remove the buffer from the surface when going PAUSED -> READY

This essentially hides the video and allows the application to
potentially draw a black background or whatever else it wants.
This allows to differentiate the "paused" and "stopped" modes
from the user's point of view.

Also reworded a comment there to make my thinking more clear,
since the "reason for keeping the display around" is not really
the exposed() calls, as there is no buffer shown in READY/NULL
anymore.

wayland: install .pc file