aiffparse: bail out early when bytes per sample is not set

This should always be set for valid files when we get there,
and checking this avoids having ad hoc checks further down
in several places.

Coverity 1139698

codecparsers: h264: fix quantization matrix conversion routine names.

Fix routine names for zigzag/raster scan order conversion routines for
quantization matrices. This ought to use the gst_h264_quant_matrix_*()
naming convention instead of gst_h264_video_quant_matrix_*(), which
derived from the MPEG-2 function names.

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

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

codecparsers: add C++ guards for MPEG-4 and VP8 parsers.

Fix MPEG-4 and VP8 APIs to export their external symbols as pure C
symbols, i.e. un-mangled for C++.

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

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

glmixer: send the stream-start event

and the caps event after.

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

gl/examples: add GST_PLUGINS_BASE_CFLAGS

gl/examples: remove spurious include for doublecube example

gl/examples: add the srcdir and builddir includes

And remove references to $(GST_PLUGINS_GL_*)

glfilterapp: remove the reshape/draw properties

The reshape property was never used.
Replace the draw property with a signal.

Based on patch by Mathieu Duponchelle <mathieu.duponchelle@epitech.eu>

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

glmixer: lock the size of mix->frames to the number of pads

Fixes a segfault with decodebin ! glmixer where the request pads on
both sides were being requested after the state change to PAUSED.

Also fixes dynamically adding and removing pads while glmixer is
in a state >= PAUSED.

gl/examples: Port to using signals instead of properties

Based on patch by Mathieu Duponchelle <mathieu.duponchelle@epitech.eu>

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

glimagesink: replace pointer properties with signals

Based on patch by Mathieu Duponchelle <mathieu.duponchelle@epitech.eu>

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

mpegtspacketizer: Use first group for handling reference pcr

If _set_current_pcr_offset gets called after a flushing seek, we ended
up using the current group for delta calculation ... whereas we should
be using the first group to calculate shifts.

Also add an early exit if there are no changes to apply

tsdemux: Remove unused variable

The seeksegment was no longer used since the switch to calculating segments
when we see data.

mpegtspacketizer: Fix debug statement

mpegtspacketizer: Fix duration evaluation in push mode

When working in push mode, we need to be able to evaluate the duration
based on a single group of observations.

To do that we use the current group values

mpegtspacketizer: Handle early PTS conversion

When handling the PTS/DTS conversion in new groups, there's a possibility
that the PTS might be smaller than the first PCR value observed, due to
re-ordering.

When using the current group, only apply the wraparound correction when we
are certain it is one (i.e. differs by more than a second) and not when it's
just a small difference (like out-of-order PTS).

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

gdp: don't print already-freed string

CID 1221987

speed: Fix position query some more

Forgot to remove code in 16fd917632c0992349ca28cf9108e9562c493ad7

CID #1139677

mpegtsdemux: Fix sticky event handling

When we receive sticky events from upstream, always return TRUE.

Fixes the issue where we receive custom sticky events (such as "uri")
and no pads are created yet.

hlsdemux: Improve parsing quoted key URIs and apply it for I-frame-based stream URI

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

hlsdemux: Don't store the current key in the playlist

It's per fragment and applying to all following fragments
until the next key is specified.

hlsdemux: Fix compiler warnings

hlsdemux: Reload the variant playlist if refreshing a playlist or downloading a fragment fails

This can happen if the playlists have moved due to the variant playlist
now being redirected to another target. This currently only works as long
as the referenced playlists don't change in relation to the variant
playlist, and the new location is purely due to a new path triggered by a
new redirection target of the variant playlist, or a new redirection
target of the playlist itself.

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

schroenc: Use an arbitrary framerate if none if given

The schro encoder always requires a framerate, but some source material,
such as RTP doesn't have a fixed one, so just fake it.

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

dvbsrc: drop redundant assignment

dvbsrc: improve timeout handing at locking loop

New approach attempts to be more accurate by measuring
the elapsed time by iteration. Also:

* Use a 10 seconds default timeout and a half a second
  polling step. New values should better match the tuning
  process on real-life scenarios.
* Correct elapsed_time computation.
* Add _retry_ioctl() to avoid bailing out on temporary
  ioctl EINTR failures (no need to check for EAGAIN cause
  we are opening the frontend on blocking mode)
* Small corrections to fail condition handling

gl/wayland: create a dummy display instead of an EGLDisplay

Currently, GstGLWindowWaylandEGL holds the wayland display connection
If we create the EGLDisplay at the GstDisplay creation time, then
libEGL will internally open another connection to the wayland server.
These two display connections are unable to communicate resulting in
no window output/display and hangs inside libEGL.

Eventually we will move the wl_display from GstGLWindow to GstGLDisplay.

rtmp: proxy logging from librtmp

Helps with debugging various librtmp issues

rtmpsrc: Fix position querying

It's the position we're querying, not the duration :)

tsdemux: Remove extraneous extra parameter from check_pending_buffers function

mpegtspacketiser: Always initialise packet AFC flags and PCR fields

Also fix a slight typo in a comment about the smoothing algorithm

tsdemux: Reset pending_ts on a discont, to re-measure initial timestamps

Since all the other timestamp tracking now gets reset on a discont,
it makes sense to wait for a PCR and timestamp buffers like when
playback first starts

srtp: add support for rollover counters and replay protection window size

We add a new signal, get-rollover-counter, to the SRTP encoder. Given a
ssrc the signal will return the currently internal SRTP rollover counter
for the given stream.

For the SRTP decoder we have a new SRTP caps parameter "roc" that needs
to be set when a new SRTP stream is created for a given SSRC.

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

liveadder: handle gap buffer flag

Remove GAP flag unless both sides have gap data, ignore
incoming data if it is gap data.

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

glimagesink: improve state change debug message

glupload: always release the previous buffer

Fixes the case where _perform_with_buffer() is called without
intervening calls to _release_buffer() as is the case on start up
with glimagesink.

Also release the buffer when reseting the upload.

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

tests: Take account of memory alignment in shm test

gdp: Fail gracefully if event can't be parsed

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

shmsink: Allocate enough memory to do alignment

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

androidmedia: Actually print the stacktrace into the error string

androidmedia: Clear the exception before trying to call Java methods to convert it to a string

androidmedia: Fix crashes when loading the plugin in a standalone application on Android >= 4.4

Check if libnativehelper is loaded in the process and if
it has these awful wrappers for JNI_CreateJavaVM and
JNI_GetCreatedJavaVMs that crash the app if you don't
create a JniInvocation instance first. If it isn't we
just fail here and don't initialize anything.

See this code for reference:
https://android.googlesource.com/platform/libnativehelper/+/master/JniInvocation.cpp

speed: make position query able to convert bytes to time

(same as 744c58d71b21475d2edd5b9334e277cfa4c53260 but for the
position query)

It was only querying in time, but then trying to use dead bytes
to time conversion code.

Coverity 1139677

mpegdemux: avoid crashing in pathological case

(Identical to commit 612cdeec80da95b12f9fad928419c7de2be20d68 which
was for resindvd)

When we'd see an unknown stream type, then a SDDS stream.
Then we'd get to the end of the switch with a NULL temp stream
pointer, and dereference it.

Coverity 1139708

glcontext: Handle (unlikely) NULL return of gl->GetString (GL_VERSION) more gracefully

CID 1219858

mpegts: Remove unneeded NULL check

Allocating those structures is done in all code paths leading there

CID #1219860
CID #1219861
CID #1219862
CID #1219863
CID #1219864

mpeg2enc: increase max. allowed value for bitrate property

For DVD the limit is around 10Mbit/s, but for e.g. ATSC
it could be much higher.

dvbbasebin: better error handling in case tsparse is missing

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

dvbbasebin: add prop setup code for guint64

Needed since we are now proxying tuning-timeout
from dvbsrc.

motioncells: improve logging

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

motioncells: make framedrop work at 30 fps too

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

motioncells: use standard macro for boolean

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

opencv: make the plugin compile with all 2.4.x versions

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

dvb/cam*: use g_strerror and other few nits

dvbbasebin: fix dvbsrc signal proxying

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

dvbbasebin: proxy new props/signals from dvbsrc

Proxy tuning start/done/fail signals and tuning-timeout
property.

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

dvbsrc: smarten up tuning logic

* Drop remaining sleep() logic in favor of polling
* Use best guess delivery system if none is set
* Make tuning/locking timeout configurable
* Add signals for tuning start, done and fail
* Drop gst_dvbsrc_frontend_status(). It was used only
  for signal LOCK checking. This is now part of the
  tuning/locking loop
* Break up frontend configuration and tuning
  on separate functions

Plus:

* Add some more useful DEBUG/TRACE messages
* Move over misplaced DVB API message
* Fix wrong comment for default DVB buffer size (http://linuxtv.org/downloads/v4l-dvb-apis/dmx_fcalls.html#DMX_SET_BUFFER_SIZE)

This patch builds up on previous work done by
Fabrizio (Misto) Milo <mistobaan@gmail.com>

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

androidmedia: Add exceptions from the Java API to error messages, and post more error/warning messages overall

androidmedia: Add a GError argument to all amc methods and catch exceptions there

androidmedia: Describe exceptions happening when scanning all codecs

androidmedia: Add API to convert an exception into a string

Release the codec before freeing it

On Samsung Galaxy S4 it is impossible to have more than one
hardware decoder at the same time. If we do not release it
explicitly the GC only releases it whenever the whole application
is finished not whenever the activity is finished and thus a player
will not be able to work correctly

camerabin: fix structure handling in preview message

Avoid trying to modify the message structure as it isn't mutable.
Use a copy and post a new message if necessary.

Fixes failing tests.

glmemory: remove uneeded gl api ifdefs

glwindow_x11: allow expose events to redraw our window

otherwise we will not update the window contents on low framerate
streams until the next buffer

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

glimagesink: unref the old buffer outside the lock

it could very well deadlock

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

Revert "[880/906] glimagesink: remove unused stored_buffer field"

This reverts commit af3a68db7dc473fb6903c18966b39e4c3f1464d7.

Conflicts:
ext/gl/gstglimagesink.c

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

glcontext: fix up assertion

error: 'return' with no value, in function returning non-void

glfeature: remove GST_GL_API_GLES3

instead check the gl version using gst_gl_context_check_gl_version()

glcontext: check for GLES versions

gl/context: add check_gl_version

glcontext: Try GLX support before EGL support

If available, glx has got a better chance of being the Gl platform
we want to use compared to EGL

glcontext: Try to open lib{EGL|GL|GLESv2}.so.1 before lib{EGL|GL|GLESv2}

Fixes issues with .so (without numbering) being installed for development
(such as from mesa-dev) but actual driver (with numbering) coming from
some other place (like nvidia drivers)

Update spec file with latest changes

atsc-section: Fix annotation

dvb-descriptor: Fix multilingual service name parsing

the provider and service name was inverted

example: ts-parser: add parential rating descriptor

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

example: ts-parser: add ca iddentifier descriptor

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

example: ts-parser: add stream identifier descriptor

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

example: ts-parser: add content descriptor

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

example: ts-parser: add component descriptor

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

examples: factor dumping code

We can just re-use the new function instead of having two duplicates

example: ts-parser: add linkage descriptor

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

mpegts: bugfix mobile hand over linkage parse missmatch

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

mpegts: atsc: add missing field to ETT table

Set the subtable_extension as ett_table_id_extension
for ETT tables as it is used by it

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

mpegts: atsc: add atsc's EIT table parsing

ATSC has its own version of the EIT table (DVB also has one).

This patch adds parsing for the ATSC EIT table and also fixed
the section identification to mark it as the ATSC one.

The implementation aws refactored to reuse some common internal
structures from ETT.

Also adds its dumping function to ts-parser example

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

tests: mpegts: add test for STT parsing

Specially for the time to UTC datetime conversion

Example taken from the A65 spec

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

mpegts: atsc: add STT table parsing

Adds the system time table structure and functions for convenient parsing of
it and for getting the UTC datetime that it represents. Also adds its
information dumping to the ts-parser example

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

mpegts: atsc: add encoding conversion for UTF-16 ETT strings

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

examples: mpegts: add function to dump ETT tables

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

mpegts: atsc: add ETT structures and parsing

ETT (extended text table) contains ATSC text information with descriptions
of virtual channels and events. The text can be internationalized and also
compressed.

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

tsbase: parse the mgt and add listed EIT/ETT pids to the known psi

This will make tsbase also parse the EITs and ETTs from ATSC streams that have
their pids reported on the MGT and post to the bus

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

mpegts: add atsc MGT section parsing

Add a parsing function for MGT and also detect the EIT tables
for ATSC, the EIT pids are reported inside the MGT and we are still
only relying only on the table id for detecting it. In the future we
would want to also check the pid and compare with whatever the MGT
previously reported to confirm that it is indeed the EIT.

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

mpegts: atsc: rename TVCT to VCT as it is the same as CVCT

Make the ATSC section parse handle both TVCT and CVCT as they are
nearly the same struct (CVCT uses 2 reserved bits that are ignored
in TVCT).

This is changing the glib type and the struct name but TVCT wasn't
released in a stable package yet so there should be no problem.

Also includes some parsing fixes and changes short_name to be
directly stored as utf8 rather than utf16

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

examples: ts-parser: add TVCT and CVCT tables dumping

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

glupload: Fix assertion after API changes

gl/colorconvert: keep our private pointer

again, avoid to much trigger

tests: update for glupload changes

gl/colorconvert: use GstObject's lock instead of our own

gl/colorconvert: remove unnecassary pointer inderection