directsoundsrc: Avoid leaking copy of caps object

gst_pad_get_pad_template_caps() returns a reference which is unreferenced,
so creating a copy using gst_caps_copy() results in a reference leak.

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

pvrvideosink: Avoid leaking bufferpool configuration

gst_buffer_pool_get_config() returns a copy to the bufferpool's
configuration, which must be passed to gst_structure_free() after
use if not given away to gst_buffer_pool_set_config().

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

amcvideoenc: Avoid leaking copy of caps object

gst_pad_get_pad_template_caps() returns a reference which is unreferenced,
so creating a copy using gst_caps_copy() results in a reference leak.
Also remove the incorrect comment to avoid confusion in the future.

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

gstgsmenc: Unref caps object after usage

gst_audio_encoder_set_output_format() does not consume caps, so
unreference that object instead.

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

interlace: fix caps refcounting

tcaps was unrefed when it shouldn't have been unrefed
in some cases.

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

dvbsuboverlay: Avoid leaking copy of caps object

gst_pad_get_pad_template_caps() returns a reference which is unreferenced,
so creating a copy using gst_caps_copy() results in a reference leak.

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

vtenc: Avoid leaking caps object and its copy

gst_pad_get_pad_template_caps() returns a reference which is unreferenced,
so creating a copy using gst_caps_copy() results in a reference leak. Also
the caps are pushed as an event downstream, but this doesn't consume the
caps so it must still be unreferenced.

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

element-maker: Avoid leaking copy of caps object

gst_pad_get_pad_template_caps() returns a reference which is unreferenced,
so creating a copy using gst_caps_copy() results in a reference leak.

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

videoaggregator: Unref allowed caps after usage

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

ladspa: Unref pad template caps after usage

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

decklink: 59.94fps is 60000/1001, not 30000/1001

And also change 30/1 to 60/1 for one 60fps mode.

codecparsers_h264: initialize some fields of pic_timing structure

Otherwise pic timing structure can have invalid cpb_removal_delay,
dpb_output_delay or pic_struct_present_flag which are blindly retrieved
in h264parse.

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

decklinksink: Initialize pixel format to bmdFormat8BitYUV

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

decklinksink: Catch errors during data flow and error out properly

Otherwise we will just deadlock the thread if e.g. scheduling a video
frame fails for whatever reason.

mpegtspacketizer: Store PCR time/byte offsets in 64 bit integers

32 bit integers are going to overflow, especially the PCR offset to
the first PCR will overflow after about 159 seconds. This makes playback
of streams stop at 159 seconds as suddenly the timestamps are starting
again from 0. Now we have a few more years time until it happens again
and 64 bits are too small.

dashdemux: support downloading segments in reverse order

When a seek with a negative rate is requested, find the target
segment where gstsegment.stop belongs in and then download from
this segment backwards until the first segment.

This allows proper reverse playback.

gl/docs: remove superflous 'the'

glupload: don't determine if frame needs upload by pointer compare

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

dashdemux: return the error to the source immediately to stop faster

When flushing, this will prevent dashdemux from trying to download more
fragments or more chunks of the same fragment before stopping.

Also improves the error handling to not transform everything non-ok into
an error.

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

mpegtspacketizer: avoid timestamp overflows

Cause timing to break in the pipeline that can lead to a stall

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

gl: document GST_GL_* environment variables

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

glimagesink: keep the uploaded buffer around on successful redisplay

We might need it later to perform a redisplay.  GstGLUpload will take
of releasing the previous buffer when it receives a new buffer.

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

glmemory: reenable the texture_rg support for !eagl

The GST_GL_HAVE_PLATFORM_EAGL is always defined we need to compare
against the value instead.

opencv templatematch: Set caps to BGR order

templatematch operates on BGR data. In fact, OpenCV's IplImage always
stores color image data in BGR order -- this isn't documented at all in
the OpenCV source code, but there are hints around the web (see for
example
http://www.cs.iit.edu/~agam/cs512/lect-notes/opencv-intro/opencv-intro.html#SECTION00041000000000000000
and http://www.comp.leeds.ac.uk/vision/opencv/iplimage.html ).

gst_templatematch_load_template loads the template (the image to find)
from disk using OpenCV's cvLoadImage, so it is stored in an IplImage in
BGR order. But in gst_templatematch_chain, no OpenCV conversion
functions are used: the imageData pointer of the IplImage for the video
frame (the image to search in) is just set to point to the raw buffer
data. Without this fix, that raw data is in RGB order, so the call to
cvMatchTemplate ends up comparing the template's Blue channel against
the frame's Red channel, producing very poor results.

templatematch: mark pads as proxy caps

Allows negotiation to happen properly

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

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

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

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

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