camerabin: use gst_object_ref to make it easier to track refs

Helps debugging

wrappercamerabinsrc: unref request pads

Do not forget to unref output-selector requested pads

jpegparse: port to baseparse

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

glwindow/11: Emit signals for mouse and key navigation events

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

glwindow: Constify string parameters to the send_*_event() functions

glimagesink: remove extra argument from debug call

glimagesink: Add navigation interface and callbacks for GstGLWindow mouse/key signals

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

GstGLWindow : Add mouse-event and key-event signals for navigation

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

gl: fix multi gl object leaks

1. fix FBO leaks in decide_allocation
2. fix texture leaks in decide_allocation and reset
3. fix texture leaks in FBO incomplete error path

wrappercamerabinsrc: only flush buffers if renegotiation is needed

This avoid extra overhead when taking sequential pictures that woudln't
need renegotiation

camerabin: handle EOS on the pipeline

Make camerabin handle EOS to the pipeline to allow standard pipeline
close where an EOS is sent to the whole pipeline before setting it to NULL.

tsdemux: Do not scan for keyframe when in push mode.

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

tsdemux: Do not scan for a keyframe in non-accurate mode.

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

gdppay: remove obsolete code

Buffers have no caps in 1.0

gdppay: put all sticky events in streamheader

Use the sticky events to compose the streamheader as they are the
ones that are persisted to config new pads linked. Instead of storing
them ourselves rely on the pad storage that already orders it for us

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

facedetect: improve enum GstFaceDetectUpdates descriptions

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

facedetect: make updates on_change works as expected

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

pnmenc: Port PNM Encoder to use GstVideoEncoder Class

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

pnmdec: Patch to handle max value

Convert the image values from 0-maxvalue to 0-255 when
'decoding' the pnm image

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

katedec: handle segment event earlier

Delaying the segment event to when caps are decided can cause issues as
the first thing katedec does on its chain function it doing a segment clip.
It will lead to an assertion if the segment format is undefined

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

katedec: handle streamheaders in caps

Properly handle the caps event by configuring the kate decoding lib using the
available streamheaders. This makes it possible to decode kate subtitles when
the stream is seeked before katedec gets the initial buffers that are usually
the streamheaders.

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

openjpeg: Don't remember -lopenjp2 as libs when falling back to openjpeg1

gl: Reorder CFLAGS to include in-source dirs first in examples

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

openjpeg: Win32-compatible OpenJPEG-2.0 detection

OpenJPEG 2.0 API uses stdcall on W32 by default. This prevents normal
autoconf library macros from finding its functions.
A more compatible check is to acutally link a program that includes a
real header.

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

openni2src: Fix unitialized variable compiler warning

gstopenni2src.cpp:721:14: error: variable 'oni_ts' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:752:26: note: uninitialized use occurs here
  GST_BUFFER_PTS (buf) = oni_ts - src->oni_start_ts;
                         ^~~~~~
gstopenni2src.cpp:721:10: note: remove the 'if' if its condition is always true
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:721:14: error: variable 'oni_ts' is used uninitialized whenever '&&' condition is false [-Werror,-Wsometimes-uninitialized]
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
             ^~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:752:26: note: uninitialized use occurs here
  GST_BUFFER_PTS (buf) = oni_ts - src->oni_start_ts;
                         ^~~~~~
gstopenni2src.cpp:721:14: note: remove the '&&' if its condition is always true
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
             ^~~~~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:642:18: note: initialize the variable 'oni_ts' to silence this warning
  uint64_t oni_ts;
                 ^
                  = 0

parser: mpeg4: fix vlc table used for sprite trajectory

The vlc table members cbits, cword and values were assigned in the wrong
order, causing the mpeg4 parser to fail when handling sprite
trajectories.

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

hlsdemux: Make statistics message more generic for other adaptive streaming demuxers to reuse

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

hlsdemux: Provide statistics about time to download playlists and fragments

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

Back to development

Release 1.4.0

Update .po files

po: Update translations

wrappercamerabinsrc: set src to ready when there are no pending buffers

Setting to ready will block waiting for buffers to be reclaimed, so flush
before setting to null to make sure no buffers are pending

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

examples: fix compilation of gl fxtest example on win32 with Gtk+ 3.0

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

openslesringbuffer: Provide the size of our array to GetDestinationOutputDeviceIDs

Otherwise it does not and just fails. It needs to know the size of the
array to not write too much to it.

openslessink: Silence some error debug output to log output

These are not really errors, just unsupported features we don't
necessarily need.

aggregator: Reset flow_return *after* stopping the srcpad task.

Otherwise it might be set in an already running aggregate function.

aggregator: Flush sinkpads when stopping

All values are meaningless in that case, so we should make sure that
we clean everything

aggregator: Do not forget to reset the flow return when stoping

Setting it to FLUSHING when the element is not started, and to OK
when it starts.

aggregator: Handle event seqnum

mxfmux: Avoid taking unnecessary ref

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

gl: Include GL_CFLAGS while checking for GL typedefs

Otherwise those checks may fail at configure time if they contain extra
include paths, while at build time they are included, potentially causing
incompatible typedefs between system GL headers and gstreamer compatibility
prototypes.

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

gl: Always include EGL_CFLAGS in GL_CFLAGS when EGL_LIBS is added to GL_LIBS

They should be handled in tandem, in case any EGL provider could require some
CFLAGS and set them (possibly once moved to prefer pkg-config files),
such as for a custom header location.

po: update POTFILES

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

d3dvideosink: Open Direct3D devices in a threadsafe way

Otherwise we'll get crashes when using the device from multiple
threads, e.g. when using multiple sinks at once.

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

curlsshsink: Include gst.h and other stuff before anything else

wrappercamerabinsrc: Fix caps filter caps proxying

The notify signal is triggered when caps is changed. But instead of
proxying the fixed caps, we query for the caps. Hence, when we go to
READY state, we endup setting template caps on the proxied caps
filter instead of NULL, which leads to negoitation failure. Correctly
proxy NULL caps if this is the new caps. Fixes not negotiated error
when running in cheese. Also fix a leak of caps string in one of the
trace.

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

gl: Always build gstglmixer, not only when full OpenGL support is enabled

curl: Use winsock2 instead and the GLib macro for checking if we're on Windows

motioncells: Use no network API if not required

ext: Include winsock.h on Windows when required

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

Release 1.3.91

gl: Link to all required libraries but not more

video: Properly include headers in Makefile.am

and don't set noinst_HEADERS twice.

Update .po files

atdec: Change rank to MARGINAL until it can properly handle multichannel audio

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

gl: Move GstGLMixer to the plugin for now

It depends on GstAggregator and we don't want to install headers
for that yet.

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

libs: Don't install headers and pc files for libgstwayland/badvideo/badbase

These will disappear after 1.4.0 and it would be rather annoying if
people started depending on them.

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

opus: Fix a double-unref in the Opus header code

The headers were never getting reffed when being added to the headers
list, which is later unreffed-and-freed by the caller (e.g.
gst_opus_parse_parse_frame()).

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

po: Update translations

examples: Add CAT section and CA descriptor support

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

mpegts: Add parsing for CA descriptor (0x05)

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

camerabin2: Flush downstream after setting src to READY

v4l2src requires all buffers to come back in order to cleanly stop
streaming. Flushing the pipline should force all buffers to come
back.

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

tests/icles/dccp: Release reference to parent

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

glimagesink: Keep aspect ratio by default

The expected default behaviour for video sink is to maintain the
aspect ratio. Fix the default value to reflect this. The property
default was already TRUE, but the value was not initially TRUE.

mpegts: No need to check for NULL before calling g_free()

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

mpegts: Don't confuse slice allocator with regular one

Previously selector_bytes and private_data_bytes were sometimes allocated and
free using the normal allocator and sometimes using the slice allocator.
Additionally prefer g_strdup() to g_memdup() for strings.

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

mpegts: Fix dvb linkage leaks

CID #1224130

mpegts: use getter for egde linkage descriptor type

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

mpegts: docs: add missed *_free methods

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

mpegts: fix annotation

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

mpegts: Add padding to public structures

Allows use to add API in the future without breaking ABI. We broke the API/ABI
once between 1.2 and 1.4, let's try to avoid this in the future even if this
is an unstable library.

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

mpegts: atsc: fix leak of short name

mpegts: fix leak of language code

videoaggregator: Fix some more the locking logic in update_src_caps

We need the GST_OBJECT_LOCK only to iterate the sinkpads, nothing else.

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

videoaggregator: fix broken locking in update_src_caps function

We would unlock an already-unlocked mutex that we never re-locked.

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

aggregator: fix locking

We would unlock a mutex we never locked on SEGMENT
events.

facedetect: Make cascades dir a little more portable

handdetect: Use already mapped image

No need to map again the image, it's already handled by the base class.

handdetect: Move size warning in set_caps so it's called once

faceblur: Port to OpencvVideoFilter base class

This fixes issue whit black frames when special memory, like GlMemory is
in used.

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

cvvideofilter: Don't try to make buffer writable

First this is handle by base transform, hence this is a no-op, and if it wasn't it
would lead to a buffer copy being leaked, and then an unreffed buffer being
pushed downstream.

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

cvvideofilter: Check buffer_map return value

Check the resturn value and cleanly fail if we could not mapped the buffers.

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

mimdec: Print invalid fourcc in error message in hex

Previously this was printed as characters which caused later processing
of the error message to sometimes warn about non-UTF-8 characters.

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

aiffparse: Print invalid fourcc in error message in hex

Previously this was printed as characters which caused later processing
of the error message to sometimes warn about non-UTF-8 characters.

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

configure: fix uninitialized variables with --disable-external

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

openni2src: Add proper clean up of OpenNI2 objects

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

openni2src: Don't embed C++ objects in our GObject

Since C++ objects shoudl be properly constructed, we keep only pointers
to them and manually construct them on the heap.

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

openni2src: Close device when stopping the stream

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

openni2src: Fix timestamping

OpenNI2 makes no guarantees of timestamp starting from zero, just that
it will be a millisecond timestamp. Make timestamps start from zero
manually so things work correctly.

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

openni2src: Make the location property not be mandatory

Our calls to device open already handle the unset location case (by
opening any available device).

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

openni2src: Open device on NULL->READY

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

openni2src: Fix deadlock when _get_caps() is called before READY

The object lock was not being dropped in the empty case. Restructured
the code a bit to make this sort of error less likely.

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

openni2src: Mark element as a live source

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

d3dvideosink: Always lock the D3D surfaces in write mode

Locking them in readonly mode can give different stride to mapping
in write mode, which then causes rendering to be broken.

Happened on all (many?) NVIDIA GPUs.

Thanks to voskater15@gmail.com for hinting at the problem.

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

d3dhelpers: Swap UV planes properly for YV12 as compared to I420

If we only do it in one place colors will look funny.

amc: Properly integrate COLOR_QCOM_FormatYVU420SemiPlanar32m again

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

amcaudiodec: audio/mpeg is always MPEG 1 layer 3

Configuring e.g. layer 2 on such a decoder usually fails unless
it additionally also accepts audio/mpeg-L2.

mpegtspacketizer: Don't dereference NULL if we have no PCR yet

Can happen sometimes if the duration is requested before we
received enough data with a PCR.

tsdemux: Add locking around mpegtspacketiser groups.

Otherwise occasionally DURATION queries can crash, if they happen at
the wrong moment.