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

adaptivedemux: handle snap seeking without setting any position

When the start_type is GST_SEEK_TYPE_NONE for a forward seek
(or stop_type for a reverse) is not set on a snap seeking operation,
the element should use the current position and then snap as requested.

Also fixes uninitialized variable complaint by clang about
'ts' variable.

dashdemux: sync index and the selected chunk when seeking

Otherwise the chunk selected isn't matched to the index and
the timing will be different, causing it to actually start
from a different position

Automatic update of common submodule

From a253974 to 6f2d209

eglimagememory: add compatibility definitions for EGL dmabuf

e.g. the RPi doesn't have them defined

opus: rename plugin to opusparse for the time being

Until we fix it up and get rid of the opus dependency and
move it elsewhere too.

opus: remove Opus encoder/decoder, moved to -base

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

glcontext: add a method to add a context to another share group

Intended for use with wrapped contexts that are created shared with gst's
gl contexts in order to manage the internal sharegroup state correctly.

e.g. with caopengllayer (which is used in glimagesink and caopengllayersink
on OS X), we create a CGL context from the gst context and the sharing state
was not being correctly set on either GL context and gst_gl_context_is_shared()
was always returning FALSE.

With 11fb4fff80b63b9d67a731d4bb238b6c0a29d774 only flushing with multiple
shared contexts, the required flush was not occuring causing screen
corruption or stuttering.

Note: this didn't affect GST_GL_API=opengl pipelines

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

opus: remove Opus RTP elements, they have moved to -good

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

hlsdemux: tests: pass test name into test setup function

All hlsdemux tests create a GstStructure called "state" that can be used
by test cases to store information during a test. The name of this
structure is arbitrary. When the code was written, the intention was
to use the name of the test, to aid debugging. However, during
development this was lost, so that the state GstStructure is always
given the name "setup_test_variables".

This commit changes this so that the name of the test is used.

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

tests: compositor: drop special case for valgrind timeout

The default one is 6 minutes, the test was using 5 minutes so just
resort to using the default.

For the non-valgrind test also use the default 20 secs instead of
reducing it to 6s. No real reason to set a custom value here.

tests: compositor: add tests for caps queries

Verifies that proper caps are returned based on what downstream
restricts.

videoaggregator: fix caps queries to allow proper renegotiation

When caps are already negotiated it should be possible to
select formats other than the one that was negotiated. If downstream
allows alpha video caps and it has already negotiated to a non-alpha
format, caps queries should still return the alpha caps as a possible
format as caps renegotiation can happen.

Includes tests (for compositor) to check that caps queries done after
a caps has been negotiated returns complete results

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

vtdec: Negotiate the decoder in set_format() already

Don't wait until later, we want to know here if the codec can be opened or not
for the requested format. This was removed (accidentially?) by
  119e09eac315f79ac2cf45b4441ad1d932130614

Without this decodebin has no way to switch to a different decoder if this one
does not work.

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

avfvideosrc: Frame durations as CTime to the API, not double

Newer iOS seems to automatically convert, older iOS/OSX just crashes.

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

nvenc: release the frames and  list when searching for an output buffer

Fixes a massive leak in:

videotestsrc ! nvh264enc ! fakesink

qt: use a static_cast instead of dynamic one

The dynamic_cast is a little but of overkill as the app will still crash if it
fails in the later g_assert.

Allows compilation with -fno-rtti

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