Fix some caps leaks in pad template creation code

gst_pad_template_new() does not take ownership of
the caps passed to it, so we need to unref the caps.

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

waylandsink: fix display re-connection (redraw pending)

Reset redraw_pending at display disconnection, so we can re-connect
later from a clean state (avoid endless buffer drops).
Add a log to inform of dropped buffers.

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

opencv: add dewarp plugin

new plugin that dewarp fisheye images

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

x265: fix a minor string leak

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

autoconvert: fix a reconfigure event leak

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

uvch264src: Add meson build support

uvch264src: Fix wrong argument order to force-key-unit event

CID 1373421

check: Use gst_check_run_suite

Ensures that xunit reports get generated when requested

smoothstreaming: Fix minor use after free reference

Just a matter of reordering a g_free() /after/ cleaning out all the
members.

CID 1396746

videoanalyse: update docs

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

androidmedia: some files are missing in the release tarball

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

mpeg4videoparse: determine intra of frame at frame parse time

... rather than when determining when to end the frame.
The opportunity to do so might not come when forced to drain,
and it seems nicer anyway to do so at parse wrapup time.

dash: Define debug category used in the isoff code for the test too now

dash: Include debug of gstisoff on dashdemux category

Move debug category of gstisoff from default to dashdemux

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

audiobuffersplit: Don't try to output anything if samples_per_buffer == 0

This happens if we had no CAPS event yet but e.g. got an EOS event. We
would then try to output a 0-sized buffer, but getting that from the
adapter will give an assertion, return NULL and then crash.

dashdemux: Add first_offset of sidx box to base offset

sidx has "first_offset" syntax which represents the distance in bytes
from anchor point to media fragment. If present, we should adjust
sidx_base_offset.

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

dashdemux: Ensure sidx parsing when updating fragment info

Fix seek fail with On-Demand profile mpd which has no indexRange attribute

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

configure: improve QPA check for cross compilation

Prefix the $QPA_INCLUDE_PATH with $PKG_CONFIG_SYSROOT_DIR just
like when checking for opencv dir.

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

dvb: move adapter number set-up by way of environment to dvbsrc

This logic did not belong to the channel configuration
parser (only used by dvbbasebin) but to dvbsrc, which
is the element directly using this value and honoring
the "adapter" property.

Allows previously non-working cases like this to work:

GST_DVB_ADAPTER=1 gst-launch-1.0 dvbsrc delsys=11 modulation=7 frequency=689000000 ! fakesink

dvb: parsechannels: replace missleadign error message

Possible failure cases also include not finding the
requested channel.

dvbsrc: add DTV_STREAM_ID to the >= v8 (minor) group

Drop redundant comment while at it.

dashdemux: Drain remaining buffer in adapter

Previous patch allows sidx box parsing from incoming buffers.
Since the incoming buffer boundary might be over sidx box,
there can be remaining buffer in isobmff parser adapter.

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

configure: Remove left-over file

check: Use non-deprecated API

player: Add deprecated compatibility functions for the audio/video/subtitle stream getters

player: Move audio/video/subtitle stream list getters into the correct namespace

player: Add get track number media info API

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

Remove various unported plugins

If they were not ported after 4+ years it seems unlikely that anybody is
ever going to need them again. They're still in the GIT history if
needed.

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

gst: Fix memory leaks in usage of gst_element_get_request_pad() API

The return value has to be unreffed at some point.

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

tools: dist gst-project-maker script

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

dashdemux: Always parsing sidx for On-Demand profile

The indexRange attribute is optional and if it's not present, we should
extract the sidx box from incoming buffers.

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

dash: Split gst_isoff_sidx_parser_add_buffer() function

To parsing sidx box itself with isobmff parser,
split gst_isoff_sidx_parser_add_buffer() into two parts.

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

hlsdemux: Extract ID3 tags, strip them from the output and provide them as tags

They often don't only contain the PCR information but also other
metadata, like title. Give this information to the pipeline.

Also strip the tags from the stream as we a) already parsed them now and
b) decoders don't like these tags to happen in the middle of the stream
(i.e. the start of each fragment) and tagdemux only can strip them off
the beginning and end.

tsdemux: Don't leak a taglist if a taglist has already been created

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

qt: improve element and property descriptions a bit

win32: fix distcheck

win32: remove generation of pre-made config.h for msvc builds

This is no longer needed with the Meson build.

Remove generated .spec file

Likely extremely bitrotten, and we should not ship this anyway.

gl/format: use the unsized format for RGB on GLES2

In GLES2 GL_RGB8 doesn't exist so we cannot use it, use GL_RGB
instead.

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

mirror: Fix documentation for the mirror mode

Using the wrong enum there causes it to be randomly mixed with the docs
of the other enums.

https://bugs.debian.org/848297

gl/window: remove unused priv variable

https://ci.gstreamer.net/job/GStreamer-master/7989/console

gl/window: remove use of main_context_push/pop_thread_default()

No-one's using/depending on it (it would have criticalled and not worked)
and it's causing more problems than it's solving.  Store the GMainContext
in the public struct instead for subclasses to optionally use instead of
relying on the push/pop state to be correct.

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

gl/effects: use non-PBO GLMemory for internal textures

middle textures in gleffects do not need to use GstGLMemoryPBO as they
aren't transfering data to/from the GPU. This will cost too much DMA
memory and cause performance issue. Change the allocator to use non-PBO
GstGLMemory.

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

amcaudiodec: Set "is-adts" to 1 for ADTS AAC

Otherwise it fails to decode.

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

glvideomixer: Reject multiview video

glvideomixer does not support it currently and it needs special support
for handling this correctly, and is rather non-trivial to implement for
all formats.

compositor: Reject multiview video

Compositor does not support it currently and it needs special support
for handling this correctly, and is rather non-trivial to implement for
all formats.

videoaggregator: Remove unlock() if set_info() fails

There is not mutex locked here that needs to be unlocked.

sys: Fix caps memory leak from pad template creation

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

gme: Forward other non-EOS events downstream

Allow most events through as-is so at least we don't
warn about missing stream-start. Drop segment and caps,
as those are sent internally.

gst: Don't declare variables inside the for loop header

This is a C99 feature.

dashdemux: Don't declare variables inside the for loop header

This is a C99 feature.

gme: Free decoder instance when going from PAUSED to READY

We only ever cleaned it up before on errors.

msdk: add mjpeg decoder

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

msdk: add mjpeg encoder

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

msdk: Add H.265 decoder

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

msdk: Add H.264 decoder

The decoder only supports system memory output presently.

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

adaptivedemux: Avoid using unset GError

gst_uri_handler_set_uri() doesn't guarantee the GError will be set
if it returns FALSE.

ttmlrender: Fix segfault on _render_text_region()

Fix invalid memory access

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

ttmlparse: Transform caps event

For autoplugging, decodebin/parsebin refers to the caps event.

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

qml: Fix egl being deinitialized on display cleanup

Use the with_egl_display() variant in order to not destroy the
EGLDisplay on destruction.

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

dvbsrc: fix unconditional use of FEC 2/5

It is not defined for < v5 minor 7

dvbsrc: add delivery system to missing modulation check message

A tuning operation can spawn multiple checks. Being
able to differentiate between them makes debugging
easier.

dvb/parsechannels: drop trailing whitespace

player: Don't try to cache the GQuark for GST_PLAYER_ERROR

This is potentially racy (in the unlikely scenario that we get two
first-time calls to gst_player_error_quark() at the same time). This
should not impact anything in terms of performance since it's only on
the error path.

The call itself could just be inlined by making GST_PLAYER_ERROR be
defined to the g_quark_from_static_string() call, but this feels ugly
from an API perspective.

fdkaac: fix the build for older fdkaac versions

Support for 7.1 was added in 0.1.14.

qml/item: also unref the display on destruction

Leaking objects (and a thread!) is never a good idea.

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

tests/examples/qmlsink: scope QApplication/Engine

So they are destroyed before gst_deinit() is run and the leaks tracer
doesn't show false-positives.

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

chromaprint: Fix compilation with chromaprint 1.4

It takes const int16_t * as argument now, while before it was void *. To
make this work with both versions without #ifdef, we omit the const.

dvbsrc: drop needless wrapper around DVB API's dvb_diseqc_master_cmd

Its only purpose was to hold a wait time that was always 0

dvbsrc: drop supperfluous wait in diseqc_send_msg()

It is always zero.

vmncdec: Sanity-check rectangle sizes a bit more thorough

The x/y coordinates could already be bigger than the configured
width/height, and adding the rectangle width/height could cause an
overflow.

dvbsrc: fix unconditional use of SYS_DVBC_ANNEX_C

It is not defined for < v5 minor 6

dvbsrc: fix value for DVB-C annex B field in adapter structure

rawvideoparse: fix typos in property description

dtls: Set openssl's threadid the 1.0.x way

For pre-1.1.x openssl, a callback to set the thread id needs to be
provided to openssl. In 0.9.x the thread id was an unsigned long. In
1.0.x it was expanded to be able to hold a void*. Here we change to use
the 1.0.x API so that the thread id can always hold a GThread*, even on
platforms like msvc x64 where unsigned long is only 32 bits.

All of this is still #ifdef'd out of existence when building with
openssl 1.1.x or later which changed the thread API again, and does not
need a thread id callback.

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

segmentation: Port to GstOpencvVideoFilter base class

grabcut: Port to GstOpencvVideoFilter base class

retinex: Port to GstOpencvVideoFilter base class

This allow reducing slightly the code. It also fixes a bug caused by
in-place buffer being mapped and never unmapped.

Automatic update of common submodule

From d015674 to 39ac2f5

opencv/meson: check presence of required opencv_contrib headers

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

edgedetect: Avoid extra copy of the output image

Simply produce the result into the output image provided by the base
class. This avoid useless copy. This also removes unchecked buffer map
and ensure that GstVideoMeta is respected (for stride mostly).

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

edgedetect: Remove redundant CAPS event handler

There is already a cv_set_caps() implemented, so just do the
entire work there.

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

cvlaplace: Avoid extra copy of the output image

Simply produce the result into the output image provided by the base
class. This avoid useless copy. This also removes unchecked buffer map
and ensure that GstVideoMeta is respected (for stride mostly).

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

cvlaplace: Remove redundant CAPS event handler

There is already a cv_set_caps() implemented, so just do the
entire work there.

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

cvsobel: Avoid extra copy of the output image

Simply produce the result into the output image provided by the base
class. This avoid useless copy. This also removes unchecked buffer map
and ensure that GstVideoMeta is respected (for stride mostly).

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

cvsobel: Remove redundant CAPS event handler

There is already a cv_set_caps() implemented, so just do the
entire work there.

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

opencv: Remove uneeded buffer writability check

When set to in_place, the BaseTransform class will guaranty that buffers
passed to transform_ip() function are writable.

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

gstglupload: relax EGL context check

Check for GST_GL_PLATFORM_EGL enum instead of type GstGLContextEGL.

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

gstglcontext: is_shared should return FALSE if no group

If a sub class of GstGLContext does not create a group
then it currently crashes:

0 g_atomic_int_get (&share->refcount)
1 _context_share_group_is_shared (context->priv->sharegroup)
2 gst_gl_context_is_shared
3 _default_set_sync_gl

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

meson/ttml: Add missing math library dependency

This code call 'round' which may require linking against the math library
on some compilers.

kmssink: post error message if start() fails

It is expected to post an error message in the bus if the device cannot
be started.

kmssink: configure display mode during set_caps

Configure the display mode when setting the negotiated caps instead of
during showing the first frame.

A framebuffer is required to set the mode. Allocate a buffer object
according to the negotiated caps and use it to set the mode. This buffer
object cannot be freed until another page flip happened on the crtc
(i.e., until the first frame is rendered).

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

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

kmssink: add parameter force-modesetting

The force-modesetting parameter forces the kmssink to ignore already
configured display modes, to configure the display mode itself and use
the base plane for output.

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

kmssink: allow only supported resolutions

If the input buffers have a different size than the display, the frames
would have to be scaled or positioned on the display. The kmssink cannot
decide which behaviour would be appropriate for which use case.

In order to avoid scaling or positioning of the input stream, allow only
the supported connector resolutions in the sink caps.

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

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

kmssink: set mode based on framebuffer configuration

Displays usually support multiple modes. Therefore, the kmssink should
not only support the preferred mode, but any mode that is supported by
the display.

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

kmssink: add mode setting and base plane rendering

The kmssink assumed that the mode was already set by another application
and used an overlay plane for displaying the frames.

Use the preferred mode of the monitor and render to the base plane if
the crtc does not have a valid mode.

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

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

interlace: Writing field-order into the src caps

Writing top-field-first vs bottom-field-first into the src caps

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

docs: add aggregator _get_type functions for signals + properties

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

glmemoryegl: remove spurious #if GST_GL_HAVE_DMABUF conditions

8c2118823b3d42840cc6f48cbdc0e1b342f90b80 had some incorrect preprocessor
conditions that aren't actually needed.  Remove them.

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

gst: Fix caps leak

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