gltransformation: requires graphene 1.0.0

gltransformation: graphene multiplication order has changed

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

glvideomixer: update element documentation

tests/glcolorconvert: do reorder checking for rgba formats

glcolorconvert: convert xRGB into ARGB properly

The alpha channel might not be the last component so check which one
it is in and clobber that one instead.

tests/glmemory: add simple transfer test

tests transferring to/from the GL with a 1x1 RGBA pixel.

glmemory: unset the opposite corresponding transfer flags when mapped with write

fixes consistency with consecutive gst_memory_map()'s with
flags & GST_MAP_WRITE

gl/cocoa: Update viewport according to the current clipping

We have to update the GL viewport if the NSView is only partially
visible. Otherwise the content of the frame will be visibly offset.

tsdemux: do not discard on discont if PES start

If a discontinuity in the stream is detected, data is discarded until
a new PES starts. If the first packet after the discontinuity is also
the start of a PES, there is no reason to discard the packets.

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

dash: Use GstUri to join URIs

gl/cocoa: Improve the NSApplication initialization

This is only for non-Cocoa apps but previously caused a 2 second
waiting during startup for Cocoa apps. This is unacceptable.

Instead we now check a bit more extensive if something actually
runs on the GLib default main context, and if not don't even
bother waiting for something to happen from there.

gl/cocoa: NSView::drawRect() should call into reshape too

We a) need to render again because part of the window was
obscured and b) might need to reshape because of clipping
changes.

gl/cocoa: Call UI related API from the application main thread

gl/cocoa: Switch to a plain NSView subclass instead of NSOpenGLView

We don't and can't use NSOpenGLView as it's supposed to be used and
it gets into our way by being to clever in various situations.

glcontext: Initialize variable

Avoids the following warning:

gstglcontext.c:647:58: error: 'prev' may be used uninitialized in
 this function [-Werror=maybe-uninitialized]

video: fix build in gst-uninstalled setup

Include the base video lib appropriately and resort all of them as
thaytan suggested.

examples: Fix Qt/GL qglwtextureshare example for X11

We need to pass the X11 display to GstGL or else it will
use its own X11 Display pointer, and the GL Context won't get shared
correctly on newer X servers

examples: port GL/Qt examples to Qt5

Update the GL Qt integration examples for Qt5 changes and for GstGL
signal signature changes.

glcontext: fixup error in sharegroup comment

Revert accidental push: "gltestsrc: implement checkers pattern with GLSL"

This reverts commit eae09179f0b4eb584c8fc7b6560d8fc8e86eed4c.

glcontext: detect the sharegroup case where the context are in different chains

In the end they will lead to the same root context in the tree so check that
for equality.

gltestsrc: implement checkers pattern with GLSL

glfilter: silence a warning with empty input caps

discovered with the doublecube example

gst_caps_get_features: assertion 'index < GST_CAPS_LEN (caps)' failed

examples/gl: update for signal signature change

videoaggregator: don't create converters when the subclass doesn't want them

fixes a critical with glvideomixer after
35bd1969f912cecda0fb5df9595a005e6ddf4e7a

CRITICAL **: gst_video_converter_new: assertion 'in_info->width == out_info->width' failed

gloverlay: fix zero output frames with YUY2/UYVY

also, be nice and disable blending for other elements

gloverlay: protect desktop gl specific tokens with #if

gloverlay: free the image memory every time the location changes

fixes a memory leak

gloverlay: Include into the build with GLES2 only too

gltestsrc: free the shader on stop

glvideomixer: fix blending with low-alpha sources

We also need to apply the blend paramaters to the alpha channel otherwise
the output of the blend will appear black at low alpha values (< 0.2).

gloverlay: reimplement everything

 - update for shaders
 - add alpha property
 - image placement properties shamelessly borrowed from gdkpixbufoverlay
 - image placement properties are GstController able
 - use GstGLMemory for the overlay image data
 - add support for gles2

style: remove unnecessary break statements

Code flow never arrives to these break statements due to the gotos preceding
them. So the breaks are unnecessary, as discussed in bug 736942.

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

mpeg2enc: break statement unnecessary

Code flow never arrives to the break statements due to the goto's. So the
breaks are unnecessary.

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

gl/cocoa: Switch from our custom main loop to a GMainLoop

Simplifies code a lot and makes it more similar to the other backends.

gl/cocoa: Clear the current GL context when it should happen

curlsmtpsink: Email with multipart content-type should end with a final boundary

Add final boundary for each of the sending out emails from smtpsink.

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

curl*sink: add more logging

curlbasesink: add a debug function

If gstreamer is build with logging enabled, add a debug function and output
verbose messages to the log instead of stderr.

curlhttpsink: only add content-disposition header if a filename is set

The header is optional and therefore we better omit it if we don't have a filename.

curlbasesink: don't replace specific error messages with a generic one

Only set an error if we haven't set one already.

curltlssink: swap check

Lets check for NULL first, as that is the default anyway.

h264parse: Return flushing if we get chained while being set to READY

Or not negotiated in the case we would be actually not negotiated

Currently we are getting assertions from
gst_pb_utils_add_codec_description_to_tag_list because of NULL
caps.

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

dashdemux: mark first buffer as discont after restarting a download task

Mark first buffer as discont after restarting a download task, so
downstream can reset its state to properly handle the new fragment.

Related issue: https://bugzilla.gnome.org/show_bug.cgi?id=736318

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

dashdemux: push a time segment when restarting streaming

This should inform demuxers/decoders downstream that the stream is
about to start from a new position and they should flush accordingly

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

video: use video lib conversion code instead of copy

mplex: break statement unnecessary

One more code flow issue where it never arrives to the break statement due to
the goto. So the break is unnecessary.
Plus two coding style fixes.

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

chopmydata: Transfer timestamps to output buffers

Allows using chopmydata as a simple repacketizer

androidmedia: break statements unnecessary

Code flow never arrives to the break statements due to the goto's. So the
breaks are unecessary. Second part to commit a3958ae65b

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

androidmedia: break statement unnecessary

Code flow never arrives to the break statements due to the goto's. So the
breaks are unecessary.

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

dshowsrcwrapper: avoid error when set_caps called twice

If set_caps is called in a running state, return immediately if the caps
haven't changed. If the pins are already connected, disconnect them.

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

glvideomixer: skip input frames with an alpha of 0

gl: download whenever we have sysmem capsfeatures

Otherwise we could pass on a RGBA formatted buffer and downstream would
misinterpret that as some other video format.

Fixes pipelines of the form

    gleffects ! tee ! xvimagesink

gsettings: Fix typo in description

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

mpegpsmux: refactor duplicate else branches

Unknown case should assert if reached instead of just assuming the max buffer
size.

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

vc1parse: initialize sent_codec_tag before using it

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

videofilters: Add support for more YUV formats

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

gltestsrc: add Mandelbrot fractal pattern.

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

gltestsrc: implement snow pattern with GLSL.

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

glupload: fallback to upload if contexts cannot share GL resources

Fixes pipelines of the form

    gleffects ! tee ! glimagesink

as tee does not pass on the allocation query.

glcontext: add gst_gl_context_can_share

Which determines whether two GstGLContext's can share sharable
OpenGL resources.

gl/tests: update for upload API changes

gl: Let gst_gl_context_get_thread() return a new reference to the GThread

Otherwise it might go away in the meantime and the caller has some random pointer.

gl: consecutive return, break statement

Signed-off-by: Anuj Jaiswal <anuj.jaiswal@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=736939

gl/debug: desktop GL does not have the KHR suffixes

examples/gl/gtk: use the sync bus callback to set the window handle

And output error messages.

dvbsrc: add missing FEC rates

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

gl: add since markers for new API

glmemory: add debug line for freeing textures

glupload: provide the output buffer that is rendered into

Allows callers to properly reference count the buffers used for
rendering.

Fixes a redraw race in glimagesink where the previous buffer
(the one used for redraw operations) is freed as soon as the next
buffer is uploaded.

1. glimagesink uploads in _prepare() to texture n
1.1 glupload holds buffer n
2. glimagesink _render()s texture n
3. glimagesink uploads texture n+1
3.1 glupload free previous buffer which deletes texture n
3.2 glupload holds buffer n+1
4. glwindow resize/expose
5. glimagesink redraws with texture n

The race is that the buffer n (the one used for redrawing) is freed as soon as
the buffer n+1 arrives.  There could be any amount of time and number of
redraws between this event and when buffer n+1 is actually rendered and thus
replaces buffer n as the redraw source.

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

glcontext: add API to retreive the thread that context is active in

gl: Removed unreachable break, unused variable

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

vtenc: Fix encoder to work on iOS

iOS has special stride requirements that we don't know yet, so copy
input buffers into buffers allocated by iOS for now.

Later we should check the stride and probably provide a buffer pool for these
buffers so upstream can directly write in there.

applemedia: We need to map the coremedia buffers in non-readonly mode

Our buffers claim to be writable so other elements expect to be able
to write there without losing their changes.

hlsdemux: lock client mutex before entering the retry_failover block

This is consistent with the case where the block execution is
triggered by the goto invoked after the current_variant update.

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

mssdemux: synchronize with the download loop thread to signal it to continue

If EOS or ERROR happens before the download loop thread has reached its
g_cond_wait() call, then the g_cond_signal doesn't have any effect and
the download loop thread stucks later.

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

hlsdemux: synchronize with the download loop thread to signal it to continue

If EOS or ERROR happens before the download loop thread has reached its
g_cond_wait() call, then the g_cond_signal doesn't have any effect and
the download loop thread stucks later.

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

dashdemux: synchronize with the download loop thread to signal it to continue

If EOS or ERROR happens before the download loop thread has reached its
g_cond_wait() call, then the g_cond_signal doesn't have any effect and
the download loop thread stucks later.

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

mssdemux: fix clearing of eos state in pads

The internal pad still keeps its EOS flag and event as it can be assigned
after the flush-start/stop pair is sent. The EOS is assigned from the streaming
thread so this is racy.

To be sure to clear it, it has to be done after setting the source to READY to
be sure that its streaming thread isn't running.

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

hlsdemux: fix clearing of eos state in pads

The internal pad still keeps its EOS flag and event as it can be assigned
after the flush-start/stop pair is sent. The EOS is assigned from the streaming
thread so this is racy.

To be sure to clear it, it has to be done after setting the source to READY to
be sure that its streaming thread isn't running.

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

dashdemux: fix clearing of eos state in pads

The internal pad still keeps its EOS flag and event as it can be assigned
after the flush-start/stop pair is sent. The EOS is assigned from the streaming
thread so this is racy.

To be sure to clear it, it has to be done after setting the source to READY to
be sure that its streaming thread isn't running.

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

vtenc: Let the encoder automatically choose a h264 level

We should negotiate these things via caps...

vtenc: Use 0 instead of G_MAXDOUBLE as the max keyframe interval

0 means no limit, which was meant here with G_MAXDOUBLE probably.

vtenc: Expected duration is supposed to be the duration of the stream, not a frame

Just don't set it for now, it isn't really needed.

vc1parser: add unit test for sequence-layer parsing

Check that a sequence-layer header is successfully parsed.

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

vc1parser: take care of endianness when parsing sequence-layer

sequence-layer is serialized in little-endian byte order except for
STRUCT_C which is serialized in big-endian byte order.

But since STRUCT_A and STRUCT_B fields are defined as unsigned int msb
first, we have to pass them as big-endian to their parsing function. So
we basically use temporary buffers to convert them in big-endian.

See SMPTE 421M Annex J and L.

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

vtenc: Properly handle keyframes

Especially set the SYNC_POINT flag on keyframes.

mpegdemux: removed an unwanted initialization and a variable

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

dshowsrcwrapper: Port to 1.0

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

camerabin2: removed redundant initialization

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

faad: logical disjunction consecutive return, break unnecessary

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

vc1parse: enable header-format conversion

In fact we support header-format conversion which is done in
update_caps() method.

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

vtdec: Fix compilation

vtdec: Handle 0/1 framerates correctly by not calculating their frame duration

atdec: Fix some compiler warnings with newer clang

vtenc: Prepare encoder after setup

This will allow encoding to happen faster on the first frame.

vtenc: Only drain the encoder in ::finish(), not on every frame

Otherwise quality and bitrate will be bad.

vtenc: Error out if encoding returned an error

Otherwise we will just continue consuming frames until all memory is filled
up and the app crashes.

vtenc: Check for errors from VTCompressionSessionCompleteFrames()

vtenc: Properly scale timestamps for the API and set invalid values