vtenc: enable the hw encoder on OSX

decklinksink: Check the correct variable for failure

If we can't get an output, we would otherwise just use it later and crash.

interaudiosrc: Make sure we have a writable buffer before we start changing metadata

intervideosrc: Set the GAP flag on repeated buffers

videorate does the same and it allows optimizations downstream.

interaudiosrc: Set the GAP flag on buffers that only contain silence

gl: remove the width/height fields from the caps to support frame resizing

It was previously only occuring with sysmem caps features

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

h264parser: fix GstH264ParserResult documentation typo

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

tests: Add x265enc unit test to .gitignore

pngparse: optimise reading of png files

Read PNG data chunk in one go by letting the parser
base class know the size we need, so that it doesn't
drip-feed us small chunks of data (causing a lot of
reallocs and memcpy in the process) until we have
everything.

Improves parsing performance of very large PNG files
(65MB) from ~13 seconds to a couple of millisecs.

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

examples: remove pointless mpegtsmux example

Serves no purpose, is not even hooked up to the
build system, has hard coded file names and paths,
and can easily be replaced with a gst-launch line.

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

avfvideosrc: Don't pretend to support reconfiguration

We will run into an assertion in set_caps() if we try to change
caps while the source is already running. Don't try to find new
caps in GstBaseSrc::negotiate() to prevent caps changes.

vc1parse: set seq_layer_sent to FALSE on reset()

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

vc1parse: implement asf to *-frame-layer stream-format

This commit add an helper to convert a frame to frame-layer format and
use it to implement these two stream-format conversion:
- asf --> sequence-layer-frame-layer
- asf --> frame-layer

In simple/main profile, we basically have a raw frame, so building a
frame layer isn't too complicated. But in advanced profile, the first
frame-layer should contain sequence-header, entrypoint, and frame and
each keyframe should contain entrypoint, so we have to handle these
carefully.

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

vc1parse: check output format at negotiation time

Add an helper to check that output stream-format is coherent with
profile and header-format. It also check if we know how to do the
conversion if the input stream-format differs from selected
output-format.
So, in case output stream-format is not allowed, it will now fail at
negotiation rather than in pre_push_frame.

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

vc1parse: add some asf related stream-format conversions

This commit introduces an helper to convert an ASF frame to BDUs format with
startcodes and use this helper to implements following stream-format
conversions:
- asf --> bdu
- asf --> sequence-layer-bdu
- asf --> sequence-layer-raw-frame

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

vc1parse: add some simple stream-format conversion

It add the support of following stream-format conversion:
- bdu --> sequence-layer-bdu
- bdu-frame --> sequence-layer-bdu-frame
- frame-layer --> sequence-layer-frame-layer

For these conversion, the only requirements is to push a sequence-layer
buffer prior to data.

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

vc1parse: prepare the stream-format conversion code and add the simplest ones

It prepares the template for stream-format conversion and it implements
the following conversion:
- sequence-layer-bdu --> bdu
- sequence-layer-bdu-frame --> bdu-frame
- sequence-layer-frame-layer --> frame-layer

Work is done in the pre_push_frame() method.

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

dashdemux: do not override last known stream error

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

mssdemux: seeks in the manifest never fail

Set void as the return and remove error handling as it could
never fail

dashdemux: make sure flushes have the same seqnum as the seek event

They are part of the same operation and should preserve the seqnum

dashdemux: simplify seek to eos handling

Remove redundant flush stop pushing and remove a goto to make
the code flow simpler

dashdemux: send missing FLUSH_STOP/EOS when no seeking period is found

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

h264parse: set the HEADER flag on buffers containing SPS or PPS

h265parse: set the HEADER flag on buffers containing VPS, SPS or PPS

gstglconfig: Put gstglconfig.h into $(libdir)/gstreamer-1.0/include

It's architecture dependent and should not be placed into the include
directory as the assumption is that all those headers are architecture
independent.

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

smoothstreaming: initialise new GstH264NalUnit fields

CID #1251107.

glimagesink: don't make gst_glimage_sink_handle_events call _ensure_gl_setup

gst_glimage_sink_handle_events can be called from the overlay interface and from
the main thread before GL is setup. Before this change, that would call
_ensure_gl_setup() and deadlock on OSX.

Change things so that it's always safe to call gst_glimage_sink_handle_events()
without stuff deadlocking.

glimagesink: fix possible deadlock on osx

Remove gst_glimage_sink_handle_events call in gst_glimage_sink_init. It was
unnecessary and when the element was instantiated from the main thread, caused a
deadlock in OSX creating the context (thread).

vtenc: add realtime and allow-frame-reordering properties

glimagesink: clamp the resize width/height to >= 1 to avoid a GL error

intervideosrc: Print a warning into the debug log on generating black frames

intervideosink: Add some debug output in the render function

h265parse: don't unnecesarily set src_caps

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

h264parse: don't unnecesarily set src_caps

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

glutils: only attempt getting the app context when we don't already have a display

avoids querying/messaging the world on each frame

glcontext: fail context creation if glGetString returns NULL

glimagesink: implement gst_video_overlay_handle_events

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

interaudiosrc: Fix compiler warning about unused variables

gstinteraudiosrc.c: In function 'gst_inter_audio_src_create':
gstinteraudiosrc.c:339:27: error: variable 'buffer_samples' set but not used [-Werror=unused-but-set-variable]
   guint64 period_samples, buffer_samples;
                           ^

interaudio: Make buffer size and latency handling more explicit and add properties for them

This now makes audio work more reliable without disconts.

interaudiosink: Use the bpf from the audio info instead of hardcoding 4

interaudio: Only flush the ringbuffer on the sink side

interaudiosink: Flush the adapter when we get new caps

Ideally we would drain the source but that would require more coordination
between sink and source than what we currently have.

gl/cocoa: include gl3.h in configure too for consistency with gstglapi.h

insertbin: Add doc for the ignored user-data in action signals

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