teletexdec: Simplify negotiation code

No need to use a while() loop if we're going only going to use
the first structure of the caps ...

CID #1341748

aggregator: Fix leak in unit test

GST_PAD_PROBE_HANDLED means that we should've unreffed the probe data,
it was handled by us in one way or another.

.gitignore: Ignore some more test temporary files

tests/compositor: Add test for aggregator pad numbering

Tests that the behaviour in 7a5cb5a473 is being conformed to.

aggregator: Fix strcmp test for sink template

aggregator: Don't try to be too smart while allocating pad names

Previously, while allocating the pad number for a new pad, aggregator was
maintaining an interesting relationship between the pad count and the pad
number.

If you requested a sink pad called "sink_6", padcount (which is badly named and
actually means number-of-pads-minus-one) would be set to 6. Which means that if
you then requested a sink pad called "sink_0", it would be assigned the name
"sink_6" again, which fails the non-uniqueness test inside gstelement.c.

This can be fixed by instead setting padcount to be 7 in that case, but this
breaks manual management of pad names by the application since it then becomes
impossible to request a pad called "sink_2". Instead, we fix this by always
directly using the requested name as the sink pad name. Uniqueness of the pad
name is tested separately inside gstreamer core. If no name is requested, we use
the next available pad number.

Note that this is important since the sinkpad numbering in aggregator is not
meaningless. Videoaggregator uses it to decide the Z-order of video frames.

lv2: Port filters to GStreamer 1.0 and use lilv instead of slv2

Properly separate files as we will not have only one single base class
for all elements as we used to with 0.10, but the same way it is done
with ladspa, we subclass GstAudioFilter, GstBaseSource etc...

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

gtk: Fix logging in base widget and fix desc of GL sink

Set a default category for gtkgstbasewidget lest the logging go to the 'default'
category where it can't be found easily

rtmpsrc plugin : add timeout option

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

mxfmux: Instead of releasing request pads on stop(), clear them only

Request pads are requested by applications and as such should only be released
by them again. Instead of releasing them when stopping the muxer, just clear
their state so that they can be used again when starting the muxer again.

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

player: Add is_live flag to the GstPlayerMediaInfo

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

sdp: Add new sdpsrc element

This can read from an SDP file or get the SDP data via property. Works
around the problem that sdpdemux inside decodebin fails because the
GST_STATE_CHANGE_NO_PREROLL is not observed by the pipeline.

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

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

gtk/gl: don't assert when gdk doesn't provide a GL context

Allows the application to check whether gtkglsink is supported by setting
the element to READY.

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

mpegtsbase: plug EIT string leak

dvbbasebin: unref pat table after usage

Remember to unref the GPtrArray of the PAT table after
use

dvbbasebin: plug some leaks

Plug various leaks in dvbbasebin.

mpegtspacketizer: simplify variable reset

Just set it directly to 0, instead of checking and reseting

dvbsrc: print time in same format

Makes it easier to compare

ahc: use g_slice_new instead of g_slice_new0

In many cases, we use g_slice_new0 and then immediately overwrite the
allocated memory. This is inefficient. Since we're going to immediately
overwrite it, we might as well use plain g_slice_new.

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

d3dvideosink: post message to application for unhandled keyboard/mouse events

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

dfbvideosink: post message to application for unhandled keyboard/mouse events

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

gtkbasesink: post message to application for unhandled keyboard/mouse events

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

ahcsrc: remove probe_properties

It's a residue of 0.10.

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

bad: use new gst_element_class_add_static_pad_template()

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

ahc: eliminate AHC*_CALL macros

Currently, we use AHC*_CALL macros to call many of the Camera functions.
However, we already have helper classes to call the Camera functions, so
eliminate the macros.

As a nice side-benefit, we also get improved error handling and
reporting when something goes wrong calling these functions, because a
GError gets populated, and we log a GST_ERROR when something fails. This
was harder to do using macros, as all error handling was hidden from the
caller.

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

ahc: use gst unref functions

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

ahc: use gst new object functions

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

interlace: fix negotiation with fixed framerate downstream

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

gl/gtk: Fix compiler warning in example

fxtest.c: In function ‘main’:
fxtest.c:190:3: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
   GtkWidget *window;
   ^~~~~~~~~

Back to development

Release 1.8.0

Update .po files

vulkan: Minor string fix

vulkan: VK_API_VERSION has been removed

In a stable release too, naughty.

Use VK_API_VERSION_1_0 instead

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

gl/proto: sync operations are available on GLES 3.0

Without the GST_GL_API_GLES2 bit set, we will not even attempt to look
for the function pointers in the core library and will fallback to
glFlush/glFinish.

gl/build: add GST_GL_HAVE_DMABUF to gstglconfig.h

it's exposed in public API so hiding it in an AC_DEFINE for config.h only
works when building libgstgl itself.  Attempting to use libgstgl (especially
on egl platforms) will throw a compilation error.

mxfdemux: Fix query memory leak

Peer query isn't being freed in case of GST_QUERY_SEEKING.

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

vulkan: Ship missing headers with tarballs

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

h264parse, h265parse: fix handling of downstream force-key-unit events

The parser handles the downstream force-key-unit event incorrectly,
it tries to parse it as an upstream force-key-unit event, does not
check the return value, and then uses uninitialized memory in
"all_headers" boolean variable.

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

glmemory: unset GL_UNPACK_ROW_LENGTH in opengl3

If the user uploads their own texture without setting the unpack length, then
then the result will have the appearance of stride mismanagement due to
an incorrect row length.

gl/examples/qt: restrict the GL API to opengl where needed

Until the examples are ported to opengl3/gles2, they will not work with any
other GL api.

gl/examples/gtk: call XInitThreads

Fixes some sporadic X11 threading assertions.

glutils: remove custom code for setting caps features

Just use gst_caps_set_features() instead.

glfilter: only allow the same src/sink caps when we are in passthrough mode

If we are given caps with extra features (like the overlay composition
features), we can only deal with that when we are in passthrough mode.

Previously we were bailing entirely and not allowing passthrough filter elements
with things like textoverlay.

Fixes the following pipeline (assuming glfilter supports passthrough):
gl ! textoverlay ! glfilter ! ... ! glimagesinkelement

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

glupload: deal with the ANY caps feature correctly

When transforming, xplode it out into the necessary caps features both
with and without the passthough features.

Fixes negotiation in the following class of pipelines:

gl ! textoverlay ! glupload ! glimagesinkelement

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

wayland: fix null pointer dereference on error

gnome-shell doesn't the support wl_scaler interface which makes creating a
wayland display fail creation.

Found in the generic state changes test

glstereo{mix,split}: allow running on GLES 2/3

It's mostly supported for GLES 2.x, fully supported on GLES 3.x

openslesringbuffer: Warn if the position reported by OpenSL is higher than what we queued up so far

This would hint at wrong position reporting, and apparently sometimes happens
after a seek.

mxfmux: Fix typo in JPEG2000 colorspace

sRGC -> sRGB

Release 1.7.91

Update .po files

tsdemux: add video/x-h265 to template caps

mpegts: Don't leave freed programs in the hash table

When the sub-class claims a program for later freeing, make
sure it's not left in the hash table, or it can cause crashes on shutdown.

Make sure tsdemux frees any program it has kept around at shutdown
if it wasn't freed already.

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

videoparsers: h265: Fix segfault while transforming hevc to nal aligned bytestream

Create temporary ParseFrame and copy the nal size buffer region
for each nal unit like we did for h264.

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

amc: Correctly handle NULL input buffers

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

amcvideodec: getOutputBuffers() returns a NULL array when a surface was configured

So don't error out if it does.

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

gleffects; give each effect a unique long name and description

Gives applications that scrape the factory details more detailed and unique
details on the exact element.

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

glshader: some compatibility changes for GL 1.4

GL 1.4 (with GL_ARB_shader_objects) doesn't have glIsProgram or glIsShader
equivalents.  As they are simply assertions, skip them when there isn't a
valid function pointer.

glfilterapp: update for the use of shaders

Fixes black output when placed in pipelines (using the default drawing).

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

glfilter: retrieve the shader attributes from the GL thread

Otherwise we will receive bogus values

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

gl/examples/cube*: choose opengl by default

The examples don't work with any other GL API.

Also fix the yuv example to not translate the cube out of the clipping area.

gl/examples/fxtest: add needed glupload to the pipeline

Fixes a negotiation failure in the example

mpegvideoparser: Handle non-hierarchical profiles again

This is a regression from since mpegvideoparser was switched to
use the codecparsing library.

The problem is that the high bit of the profile_and_level is used
to specify non-hierarchical profiles and levels. Unfortunately we
were discarding that information.

Expose that escape bit, and use it in the element

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

glimagesink: Fix window memory leak

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

tsdemux: Don't reset/recalculate segments with accurate seeks

When dealing with accurate seeks, we must send out a segment which
is exactly what is requested.

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

glvideomixer: signal continuation in reset

We want to iterate over all the pads, not just the first one.  Fix by returning
TRUE in the GstAggregatorPadForeachFunc.

Removes a GST_IS_GL_CONTEXT() assertion on shutdown with >2 inputs
using gst-launch.

gldisplay: make readding the same context a no-op

With e38af2304427db908a16bbae0e60aa68be1ba5b5 returning the correct contexts,
gst_gl_display_add_context() was susceptible to causing infinte loops when
adding the same GstGLContext more than once.  Fix and add a test for
gst_gl_display_add_context().

Fixes glvideomixer gst-validate tests.

Revert "libgstgl: cocoa, eagl: don't marshal GL calls to the context thread"

This reverts commit 797d6415dfd6e111efb2cab544958a67cbf22b17.

We're frozen for 1.8.0 release and this change might have bigger impact.

libgstgl: cocoa, eagl: don't marshal GL calls to the context thread

Execute GL calls without marshalling them to the context thread. In the cocoa
and eagl backends calling gst_gl_context_activate is cheap and therefore calling
it on the current thread and serializing GL calls with a per-context lock is
more efficient (faster and has less overhead) than marshalling everything to the
context thread.

This optimization cuts a large overhead in g_poll (continuously waking up the
context thread) and in g_mutex_*/g_cond_* (waiting for results from the context
thread).

gldisplay: really retrieve glcontext for a specific thread

When requesting a glcontext (regardless of thread), the result was correct.
However, when requesting current glcontext on a specific thread, it could
come up with a glcontext active on another thread.

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

gl: misc docs fixes/additions

gl*memory: document new functionality and objects

docs: add new GL api to .types file for type/signal introspection

docs: ignore some private GL headers

docs/gl: add gl functions/structs from -unused into the docs

glfilter: handle some more unfixed fields when fixating caps

gtk: examples: #define GST_USE_UNSTABLE_API and link with X11_LIBS

X11_LIBS is needed for XInitThreads() and without the #define we get
warnings about the GL API being still unstable.

androidmedia: Remove unused file

ahcsrc is registered by gstamc.c

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

amc: properly deinit when ahcsrc register fails

In the androidmedia plugin_init, we initialize various resources on the
Android device. If anything fails during this series of initializations,
we need to deinitialize any initializations that already occurred.
However, we don't do so if we fail to register the ahcsrc element. Fix
this.

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

ahc: correct error message

The error message is specific to only one of the failure cases and is
misleading in the others. Correct it to be more generic and cover all
the failure cases.

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

ahc: remove unneeded #include <stdio.h>

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

ahc: typo fix

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

hls: Add OPENSSL_CFLAGS to CFLAGS

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

glbasememory: Don't change maxsize at run-time

Maxsize is initialized once and should never change. Allocating data
should have no impact on the selected max size for this memory. This
causing memory map failure as the maxsize would become smaller then
size. This happened when using direct rendering in avviddec on GL that
does not support PBO transfer.

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

qml: Fix leak of the OpenGL contexts

[Matthew Waters]: add NULL checks before unreffing

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

vksink: fix spelling on failed _ensure_data() error message

mpeg2enc: Provide format as a string instead of a char to gst_structure_new

The format was provided as 'I420' instead of "I420", causing a crash.

mpegtspacketizer: handle early PTS conversion when a group has been found

In some cases, the PTS might be smaller than the first observed PCR
value which causes element to apply wraparound leading to bogus
timestamp. To solve this, we only apply it if the PTS-PCR difference is
greater that 1 second to be sure that it's a real wraparound.

Moreover, using unsigned 32 bits values to handle wrapover could end up
with bogus value, so it use pts value to handle it.

Also, convert pcr time to gst time before comparing it to pts.
Since refpcr is expressed in PCR time base while pts is expressed in GStreamer
time.

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

adaptivedemux: Fix REMOVE_SNAP_FLAGS() macro

!(flag1 | flag2 | flag3) is always evaluation to 0. ~ was meant here
instead of !.

CID 1352032.

Release 1.7.90

Update .po files

po: Update translations

videoparsers: h264: Disable passthorugh mode enabling

Enabling passthorugh mode is causing multiple issue:
For nal aligned multiresoluton streams, passthrough mode
make h264parse unable to advertise the new resoultions.
Also causing issues while parsing MVC streams which have two
separate layers (base-view and non-base-view).

This fix is only a temporary workaround.

For MVC, proper fixes needed in many places:
(handle prefix nal unit, handle non-base-view slice nal extension,
fix the picture_start detection for multi-layer-mvc streams etc)

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

sdpdemux: Set caps to application/x-rtp instead of application/x-unknown as returned by the SDP helpers

The SDP helpers can't know if this is going to be RTP, SRTP, or ....

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

sdpdemux: Add pads with the correct names in case an rtpbin is used

They're supposed to be stream_%u and not recv_rtp_src_%u_%u_%u.

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

vkswapper/vkutils: Fix gerror memory leak

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

vkdevice: Fix duplicate assignment of queue variable

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

vksink: Fix GError memory leak

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

win32: update exports for new libgstgl function

amcvideodec: When outputting on a surface, accept all color formats

We don't have to understand them, we handle them as a GL texture.

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