gl/cocoa: use NSAutoreleasePool to free resize data

Otherwise when resizing the window you will also get messages like:
class NSConcreteMapTable autoreleased with no pool in place - just leaking
class NSConcreteValue autoreleased with no pool in place - just leaking
class NSConcreteValue autoreleased with no pool in place - just leaking
class __NSCFDictionary autoreleased with no pool in place - just leaking

gl/cocoa: include gl3.h to have GL_CONTEXT_PROFILE_MASK

gl/cocoa: make sure to turn on frame rectangle changes notifications

Default value of property postsFrameChangedNotifications is YES
but it is worth to explicitly enable it.

gl/cocoa: fix uncaught exception 'unrecognized selector sent to instance'

Need to set the ':' as the reshape method now takes one parameter.
For the story, the GstGLNSView was previously inheriting from
NSOpenGLView which has a reshape function without any parameter.
Now the GstGLNSView inherits from NSView and we re-use the reshape
function manually.

gl/cocoa: fix compiler warning

Use the reshape function after being defined. The other way
would have been to declare the reshape function in the header.

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.)

intervideosrc: Negotiate framerate with downstream

Prefer the closest framerate to the input framerate though.

inter*src: No need to set caps manually on the pad

basesrc does that for us already.

videoaggregator: Swap source/destination parameters of gst_video_converter_frame()

inter: fix order of arguments

interaudio: Set all required fields on the template caps

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

gl: fix Since marker for gst_gl_window_run_navigation()

inter: don't leak surface name

mpegdemux: start pushing data again when a pad gets linked later

The whole not_linked optimisation is really a bit dodgy here, but
let's leave it in place for now and at least start pushing data
again when a pad got linked later, in which case we should get a
RECONFIGURE event.

mpegdemux: namespace fixes

audiovisualizer: post QoS messages when dropping frames due to QoS

audiovisualizer: fix boilerplate macros

Sprinkle some G_PARAM_DEPRECATED and #ifndef GST_REMOVE_DEPRECATED

hls: fix indentation

glshader: Fix memory leak

Memory is only freed in the TRUE clause of the if conditional. Free in the else
clause as well.
Also, consolidate g_malloc + sprintf into a g_strdup_printf().

CID #1212171

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

rtpopus: Use OPUS encoding name

Both Firefox and Chrome uses OPUS as the encoding in their SDP.
Adding this now defacto standard name remove the need for special
case in SDP parsing code.

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

mssdemux: add type of stream to debug log

Makes it faster to debug why certain streams are not showing

mssdemux: do not unlock not locked mutex

glcolorconvert: fix comments that confuse gtk-doc

GTK-Doc uses a special syntax for code documentation. A multiline comment that
starts with an additional '*' marks a documentation block that will be processed
by the GTK-Doc tools. So GTK-Doc is confused if a comment block starts with that
additional '*' but isn't meant to be processed. Removing this additional '*'.

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

glimagesink: resize the viewport correctly on a caps change

with force-aspect-ratio=true, if the width or height changed, the
viewport wasn't being updated to respect the new video width and height
until a resize occured.

glmixer: advertise support for changing input caps mid-stream

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

gl: sprinkle some Since markers

mpegts: remove storage of never used values

Both _parse_atsc_mgt() and  _parse_atsc_vct () change the value of the variable
data just before returning. The new value is never used since data is a pointer
declared at the beginning of the function and going out of scope just after the
new value is stored.

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

gaudieffects: remove < 0 comparison on guint32

Current CLAMP checks both if the value is below 0 or above 255. Considering it
is an unsigned value it can never be less than zero, so that comparison is
unnecessary. Switching to using if just for the upper bound.

CID #1139796

gaudieffects: declare floor and ceiling as constants

gaudieffects: stored value is overwritten

Value from left_luminance is assigned to out_luminance here, but that stored
value is not used before it is overwritten in the next cycle of the loop.
Removing assignation.

CID #1226473

mpegtsbase: do not remove programs on EOS

As a consequence, tsdemux won't remove its pads anymore on EOS.

Fixes the case when mpegtsbase is not able to process new packets
after EOS as the corresponding pids aren't known anymore because
the programs were removed and the pes/psi were kept, preventing the
PAT to be parsed again.

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

siren: fix sample rate list

It was using a 24000/24000/48000, but I think it meant to use
24000/32000/48000. Not 100% sure...

https://en.wikipedia.org/wiki/G.722.1 has the list of supported
bitrates. It's not clear whether the "flag" code maps to this,
however.

Coverity 206072

videoaggregator: remove storage of never used values

These two values are stored just before the function returns and they go out of
scope.

opusenc: update output segment stop time to match clipped samples

This will let oggmux generate a granpos on the last page that properly
represents the clipped samples at the end of the stream.

srtpdec: Trivial documentation fix

glmixer: don't get the current caps from GstVideoInfo for the srcpad

It's missing the caps features needed.

gl/examples: update for other-context property removal

glcontext: add more functionality to wrapped contexts

Implements get_current_context() and get_proc_address() for wrapped
contexts.

glshader: advertise the default vertex and fragment shaders on desktop GL

codecparsers: h264: recognize SVC NAL units.

Identify SVC NAL units and tag them as such. This is necessary for
gst_h264_parser_parse_slice_hdr() to fail gracefully, if the user
did not perform the check himself.

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

codecparsers: h264: fix number of list0 {,non-}anchor refs.

Fix copy-paste error in gst_h264_sps_mvc_copy() where num_anchor_refs_l0
and num_non_anchor_refs_l0 were incorrectly initialized from list1.

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

Update .def for new API

codecparsers: sprinkle some gtk-doc Since: markers for new API

satisfy gst-indent

glfiltercube: Fix typo in property description

codecparsers: Initialise nalu extension type to NONE

Always set a default NALU extension type, and override it
when we find a supported extension, to avoid having it unset/random
for unsupported NALU extensions

h264parser: Fix frame packing SEI parsing

codecparsers: h264: add support for Frame Packing Arrangement SEI message.

This parses the frame_packing_arragement() payload in SEI message.
This information can be used by decoders to appropriately rearrange the
samples which belong to Stereoscopic and Multiview High profiles.

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

Signed-off-by: Sreerenj Balachandran <sreerenj.balachandran@intel.com>
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: add support for Stereo Video Information SEI message.

Add the necessary payload parsing support for stereo_video_info.

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

Signed-off-by: Sreerenj Balachandran <sreerenj.balachandran@intel.com>

codecparsers: h264: parse seq_parameter_set_mvc_extension().

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

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

codecparsers: h264: parse MVC syntax elements.

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

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

tsparse: Handle backward and discont timestamps better.

Assume that small backward PCR jumps are just from upstream packet
mis-ordering and don't reset timestamp tracking state - assuming that
things will be OK again shortly.

Make the threshold for detecting discont between sequential buffers
configurable and match the smoothing-latency setting on tsparse
to better cope with data bursts.

tsparse: Implement timestamp/retimestamping of output buffers

When the set-timestamps property is set, use PCRs on the provided
(or autodetected) pcr-pid to apply (or replace) timestamps on the
output buffers, using piece-wise linear interpolation.

This allows tsparse to be used to stream an arbitrary mpeg-ts file,
or to smooth jittery reception timestamps from a network stream.

The reported latency is increased to match the smoothing latency if
necessary.

configure.ac: auto decision to include GL library fails

The part of the configure.ac that consist to check if we
can include both GL and GLES2 at the same time is failing.
Indeed, in the case NEED_GLES2=yes and NEED_OPENGL=auto,
HAVE_OPENGL variable is updated whereas it should be HAVE_GL
variable that has to be updated (HAVE_OPENGL variable is not
used in the rest of the configure.ac).

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

Signed-off-by: Vincent Abriou <vincent.abriou@st.com>
Reviewed-by: Benjamin GAIGNARD <benjamin.gaignard@linaro.org>

codecparsers: remove ignored increment of return

'return val++;' returns the value before it is incremented because the post
increment happens after the statement. Removing the unused increment.

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

glmixer:fix incorrect parameter passed to handle_set_context

Add some missing G_END_DECLS

faad: fix wrong unrefs in set_format error code path

glfilter: get the config from the correct pool

glfilter: avoid uninitialized variable size when chaining multiple glfilters

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

glfilter: gst-indent file

glcontext: add api for retreiving the current context and api

that is current in the calling thread.

glprototypes: add vertex attribute array definitions

Automatic update of common submodule

From 84d06cd to 7bb2bce

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