inter: Use 64-bit safe scaling functions

dvbsrc: reflow switch case statement a bit

Make it clear there's no 'break' missing at the end
of the case SYS_DVBT, and pacify coverity (CID 1249689).

curlsshsink: use the locally defined types

Just a matter of coding style, makes the code a bit tidier...

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

Parallelise 'make check-valgrind'

configure: check for more openh264 API that we need

tests: fix audiomixer test on big endian systems

dvbsrc: correctly capitalize DiSEqC

dvbsrc: rework _output_frontend_stats()

Retry stat reporting ioctls on EINTR

dvbsrc: add checks on allowed DVB-T bandwidths

dvbsrc: add fixme note on DTV_LNA setting

Rephrase another comment while at it

dvbsrc: add missing logic for additional bandwidths

New 1.712, 5 and 10 MHz bandwidths were been ignored
at set/get prop.

configure.ac: Use libsrtp.pc if present

rawparse: fix handling of segment event in sink event handler

glmixer: override the caps query in order to 'convert' capsfeatures

Otherwise, it is only possible for the sink pads and the src pads to
have the exact same caps features.  We can convert from any feature
to another feature so support that.

glmixer: override the accept caps query in order to 'convert' capsfeatures

Otherwise, it is only possible for the sink pads and the src pads to
have the exact same caps features.  We can convert from any feature
to another feature so support that.

gl: propogate other-context using GstContext

dash: Remove obsolete README.

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

interaudio: Add support for arbitrary raw audio formats

intervideo: Add support for arbitrary video formats

inter: Truncate the caps as part of the fixating

interaudio: Fix timestamp, latency and period handling

interaudio: Use GST_DEBUG_OBJECT() instead of GST_DEBUG()

interaudio: Properly pass through caps from the sink to the source

Otherwise a magic capsfilter after the source is required with
exactly the same caps as the input.

inter: Clean up surfaces after the last user is gone

inter: Whitespace cleanup

intervideo: Properly pass through caps from the sink to the source

Otherwise a magic capsfilter after the source is required with
exactly the same caps as the input.

This would've failed before with invalid buffer sizes:
gst-launch-1.0 videotestsrc ! intervideosink  intervideosrc ! "video/x-raw,width=640,height=480" ! xvimagesink

rawparse: Check for 0-size after adjusting the size for the frame size

If we don't have a complete raw audio frame we would otherwise still
ask for a 0 sized buffer from the adapter.

rawparse: Don't try to retrieve 0 byte buffers from the adapter in multi-frame mode

configure: improve openh264 check

The openh264 API is still unstable, so check for
structure member that we need to prevent building
against an older incompatible version.

configure: simplify openh264 check

Of course PTHREADS_CFLAGS is not used anywhere now
for the check, but that was the case before as well.

gl/cocoa: Fix compiler warning

gstglwindow_cocoa.m: In function '-[GstGLNSView drawRect:]':
gstglwindow_cocoa.m:555: warning: 'GstGLNSView' may not respond to '-reshape'
gstglwindow_cocoa.m:555: warning: (Messages without a matching method signature
gstglwindow_cocoa.m:555: warning: will be assumed to return 'id' and accept
gstglwindow_cocoa.m:555: warning: '...' as arguments.)

vtenc: Fix locking

The object lock only protects the session, as we modify
the session from other threads when the bitrate property
is changed. Don't hold it much longer than for session
related things.

And we need to release the video decoder stream lock before
enqueueing a frames. It might wait for our callback to dequeue
a frame from another thread, which will then take the stream
lock too and deadlock.

Automatic update of common submodule

From a8c8939 to 84d06cd

configure: use a macro to check for pthread

Replace the hardcoded -lpthread in most of the places with $PTHREAD_LIBS. For
openh264 also add $PTHREAD_LIBS to OPENH264_LIBS until upstream ships a .pc
file.

Automatic update of common submodule

From 36388a1 to a8c8939

hlsdemux: reset end_of_playlist attribute when we receive a seek

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

audiomixer: critical error for blocksize, timeout min/max values

Audiomixer blocksize, cant be 0, hence adjusting the minimum value to 1
timeout value of aggregator is defined with MAX of MAXINT64,
but it cannot cross G_MAXLONG * GST_SECOND - 1
Hence changed the max value of the same

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

glmixer: fixup eglimage include path

videoaggregator: fixate the parts of the caps we don't know how to deal with

fixes glvideomixer with video/x-raw,width=foo i.e. no format field.

glbufferpool: add the GstVideoGLTextureUploadMeta buffer pool option

videoaggregator: operate on caps rather than video info

Otherwise the CapsFeatures will be lost along with the possibility
of multiple output types and formats.

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

vc1parse: introduce a helper to make sequence-layer

It will be useful to implement stream-format conversion.

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

gl: Check for GLU before trying to use it in configure.ac

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

glcolorconvert: support RGB16/BGR16 video format upload

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

vc1parser: fix expected level in sequence-layer parsing unit test

Sequence-layer used for unit test have a level set to 2 which should
match the medium level, not the high.

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

vc1parser: fix level values for simple/main profile

In simple profile, level set to 0 or 2 indicate low and medium level
respectively. In main profile, level set to 0, 2 or 4 indicate low,
medium and high level respectively.

Level values are defined in Annex J.1.2 of the SMPTE 421M.

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

tsdemux: send initial GAP event on sparse streams for faster startup with playbin

Signal sparse streams properly in stream-start event and force sending
of pending sticky events which have been stored on the pad already and
which otherwise would only be sent on the first buffer or serialized
event (which means very late in case of subtitle streams). Playsink in
playbin waits for stream-start or another serialized event, and if we
don't do this it will wait for the multiqueue to run full before
starting playback, which might take a couple of seconds.

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

tsdemux: GAP detection

All pads of a stream are now added at the beginning. In order to cope with
streams that don't get any data (forever or for a long time) we detect gaps
and push out GAP events when needed.

Cleanups and commenting by Jan Schmidt <jan@centricular.com>

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

vc1parse: select caps according to wmv format at negotiation

Some VC1 decoder can have different caps according to wmv format, ie
WMV3 or WVC1.
So instead of keeping the first available caps, we interserct with
current WMV format.

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

configure: add pthread to the configure check for openh264

rtmpsink: Declare sink variable that was forgotten in last commit

rtmpsink: Free URI string in finalize()

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

gl/cocoa: Fix for building on Mac OS X 10.10

Using NSApp directly seems to confuse something, as the compiler
was expecting an id<NSFileManagerDelegate>. Switched to using
[NSApplication sharedApplication], and specified the delegate
protocol on the window class as well.

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

vtenc: Use kVTProfileLevel_H264_Baseline_AutoLevel only on iOS

It is not required on OSX apparently and was only added in 10.9.6 there.
Calculating the correct level from the configuration is not trivial, so let's
just not set a level at all here.

vtenc: VTCompressionSessionPrepareToEncodeFrames only exists since 10.9.6

Check with configure for it instead of using one of the availability macros
as those wouldn't work as expected with minor versions.

vtenc: Remove unused #define

rtpbad: Fix make dist by removing non-existing file from EXTRA_DIST

It's used by the rtp plugin in gst-plugins-good for VP8 but not needed
or used here.

rtpbad: include plugins base cflags (fix build on jenkins)

rtp: add h265 RTP payloader + depayloader

curlbasesink: strip newlines from curl debug messages

The messages we receive in the custom log handler might end with a newline and
are not \0 terminated. Copy the messages, trim and terminate them.

curlbasesink: small code cleanup

Use a local var and save a local var.

tests/x265enc: add simple unit test

x265enc: add x265 encoder element

gldeinterlace: ref the uploaded buffer

Instead of the possibly non-GL input buffer.

glfilter: add uploaded_buffer field

vc1parse: parse frame header when stream format is ASF/raw for simple/main profile

When stream-format is ASF or sequence-layer-raw-frame, we basically have
a raw frame so we can parse it to extract some information such the
keyframe flag. The only requirement is to have a valid sequence-header.

This commit parse the frame header and set the DELTA_UNIT buffer flag in
case the frame is not a keyframe.

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

libde265: Change rank to SECONDARY, gst-libav has PRIMARY rank

Integrate libde265 into gst-plugins-bad.

vc1parse: just assume none header-format when no codec_data is present

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

vtdec: Set reorder length to 0 if we can't calculate it

Instead of leaving it at whatever value it had before.

glimagesink: Only finalize the other context in finalize()

Otherwise we change a value of a property when going to READY state,
which is unexpected behaviour.

glmixer: Call the pad's parent class finalize method

glmixer: Only finalize the other context in finalize()

Otherwise we change a value of a property when going to READY state,
which is unexpected behaviour.

gltestsrc: Add other-context property

glimagesink: Don't leak other-context

glmixer: Add other-context property

dvb: make interleaving a prop and proxy on dvbbasebin

DTV_INTERLEAVING is currently used only for DTBM. This is
congruent with the v4l dvb API where the different interleaving
modes where added for v5.7

dvbsrc: add transmission mode checks for DVB-T/T2

These and every sanity check leading to a warning
(we have a few in place) should come handy when
debuging failed tunning scenarios.

dvbsrc: docs, drop v5 API change history comments < .5

We only support from minor 5 and up now. Also add a
note about the recently added preliminar DTMB support.

dvbsrc: add missing DVB-T2 bandwidth types

DVB-T2 supports 5, 10 and 1.712 MHz

Order of the enum values (new values after _AUTO)
has been kept congruent with the one in the v4l
API for consistency

waylandsink: do not render twice the same buffer

Do not try to render a buffer that is already being rendered.
This happens typically during the initial rendering stage as the first
buffer is rendered twice: first by preroll(), then by render().
This commit avoids this assertion failure:
  CRITICAL: gst_wayland_compositor_acquire_buffer: assertion
  'meta->used_by_compositor == FALSE' failed

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

Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

waylandsink : Allow surface to catch input events

If waylandsink is the owner of the display then it is in charge
of catching input events on the surface.

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

Signed-off-by: Tifaine Inguere <tifaine.inguere@st.com>
Reviewed-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

waylandsink: take into account the case where a pool may be destroyed together with GstWlDisplay

There are two cases covered here:
1) The GstWlDisplay forces the release of the last buffer and the pool
   gets destroyed in this context, which means it unregisters all the
   other buffers from the GstWlDisplay as well and the display->buffers
   hash table gets corrupted because it is iterating.
2) The pool and its buffers get destroyed concurrently from another
   thread while GstWlDisplay is finalizing and many things get corrupted.

waylandsink: stack the video subsurface into another subsurface that covers the whole render rectangle

The main reason behind this is that when the video caps change and the video
subsurface needs to resize and change position, the wl_subsurface.set_position
call needs a commit in its parent in order to take effect. Previously,
the parent was the application's surface, over which there is no control.
Now, the parent is inside the sink, so we can commit it and change size smoothly.

As a side effect, this also allows the sink to draw its black borders on
its own, without the need for the application to do that. And another side
effect is that this can now allow resizing the sink when it is in top-level
mode and have it respect the aspect ratio.

waylandsink: rename video format conversion functions to indicate they are about wl_shm

Needed to add linux_dmabuf format conversion functions later

waylandsink: remove the ugly gst_wl_display_stop() now that this mechanism is not needed anymore

Because we no longer have a custom buffer pool that holds a reference
to the display, there is no way for a cyclic reference to happen like
before, so we no longer need to explicitly call a function from the
display to release the wl_buffers.

However, the general mechanism of registering buffers to the display
and forcibly releasing them when the display is destroyed is still
needed to avoid potential memory leaks. The comment in wlbuffer.c
is updated to reflect the current situation.

waylandsink: replace the custom buffer pool with an allocator

This reduces the complexity of having a custom buffer pool, as
we don't really need it. We only need the custom allocation part.
And since the wl_buffer is no longer saved in a GstMeta, we can
create it and add it on the buffers in the sink's render()
function, which removes the reference cycle caused by the pool
holding a reference to the display and also allows more generic
scenarios (the allocator being used in another pool, or buffers
being allocated without a pool [if anything stupid does that]).

This commit also simplifies the propose_allocation() function,
which doesn't really need to do all these complicated checks,
since there is always a correct buffer pool available, created
in set_caps().

The other side effect of this commit is that a new wl_shm_pool
is now created for every GstMemory, which means that we use
as much shm memory as we actually need and no more. Previously,
the created wl_shm_pool would allocate space for 15 buffers, no
matter if they were being used or not.

waylandsink: rework the mechanism for keeping buffers out of the pool until wl_buffer::release

This also removes the GstWlMeta and adds a wrapper class for wl_buffer
which is saved in the GstBuffer qdata instead of being a GstMeta.

The motivation behind this is mainly to allow attaching wl_buffers on
GstBuffers that have not been allocated inside the GstWaylandBufferPool,
so that if for example an upstream element is sending us a buffer
from a different pool, which however does not need to be copied
to a buffer from our pool because it may be a hardware buffer
(hello dmabuf!), we can create a wl_buffer directly from it and first,
attach it on it so that we don't have to re-create a wl_buffer every
time the same GstBuffer arrives and second, force the whole mechanism
for keeping the buffer out of the pool until there is a wl_buffer::release
on that foreign GstBuffer.

schrodec: optimize parse logic

Header will be read each and everytime parse function will be called
which is not necessary since until we have complete data,
we need not parse the header again.

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

fluiddec: don't leak incoming caps event

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

vc1parse: fix framesize when input is frame-layer

frame-layer header is represented as a sequence of 32 bit unsigned
integer serialized in little-endian byte order, so framesize is on the
first 3 bytes.

SMPTE 421M Annex L.

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

midiparse: mention fluiddec instead of fluidsynth

The element name is actually fluiddec even if it uses fluidsynth.

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

fluiddec: fix some memory leaks

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

pcapparse: Add support for LINKTYPE_RAW

Also, strictly speaking, these numbers aren't DLT_*; they are LINKTYPE_* because
libpcap translates from internal OS-specific DLT_ numbering to the portable
LINKTYPE_ number space when writing files.

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

test: use G_GSIZE_FORMAT in audiomixer test

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

tests/aggregator: add timeout handling test for the timeout parameter

aggregator: add latency query handling

videoaggregator: support unresponsive pads

Render unresponsive pads with the last video frame received.

aggregator: add a timeout property determining buffer wait time

Determines the amount of time that a pad will wait for a buffer before
being marked unresponsive.

Network sources may fail to produce buffers for an extended period of time,
currently causing the pipeline to stall possibly indefinitely, waiting for
these buffers to appear.

Subclasses should render unresponsive pads with either silence (audio), the
last (video) frame or what makes the most sense in the given context.

videoaggregator: remove the use of the queued buffer on sink pads

That data is now held by the aggregator class