video: Add support for BT2020 colorspace (UHD)

video-scaler: remove useless debug

video-converter: add options to control chroma resampling

Add an option to disable chroma resampling.
Improve the matrix option values so that you can choose to use the input
or output matrix or disable conversion.

oggmux: remove unused enum

oggmux: fix silly GQueue iteration code

Fix documentation that incorrectly says a return value should be freed

The gst_discoverer_info_get_missing_elements_installer_details()
documentation and annotation says that the return value should be freed
with g_strfreev(), but actually it's owned by the GstDiscovereInfo
object and should definitely not get freed by the caller as well.

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

audiobasesrc: Explicitly document that buffer-time and latency-time may be ignored

oggmux: only clip by duration if end of buffer is ahead of segment

It might happen that the timestamp is before the segment and the
check would succeed. In this case reducing the duration makes no
sense and would lead to broken results.

videotestsrc: Report our latency properly in live mode

While we have no latency at all in theory, any other live source has the
duration of one buffer as minimum latency. Do the same in videotestsrc.

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

audiotestsrc: Report our latency properly in live mode

While we have no latency at all in theory, any other live source has the
duration of one buffer as minimum latency. Do the same in audiotestsrc.

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

videopool: update video alignment after applying

Video buffer pool will update video alignment to respect stride alignment
requirement. But haven't updated it to video alignment in configure.
Which will cause user get wrong video alignment.

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

audiobasesink: get the internal time before the clock reset

Otherwise calls to get the clock time might change its internal state
and the internal/external time for calibration get unbalanced leading to
a clock jump

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

MAINTAINERS: Update my mail address

video{en,de}coder: Call reset() before the start() vfunc

This makes sure that the element is in the same state before start() is called
the very first time and every future call after the element was used already.

Also it ensure that we always have a clean state before start(), cleaned the
same way in every case.

audioencoder: Call reset() before the start() vfunc to guarantee a clean state

The same was done already in the decoder, and we cleaned some state just above
manually that would also be taken care of by reset().

This makes sure that the element is in the same state before start() is called
the very first time and every future call after the element was used already.

video{en,de}coder: Reset the codec after calling the stop() vfunc

The stop() vfunc might mess with some of our fields we have just
reset, which could cause memory leaks or invalid state taken over
to later.

Also the stop() vfunc, or anything called until it from another thread,
might want to be able to use the fields that were just resetted and
become confused because of that.

In the decoder we already had a workaround for things like this happening,
this workaround is not needed anymore.

audiobase{sink,src}: Don't hold the object lock while calling create_ringbuffer() vfunc

The implementation of that vfunc might want to use the object lock for
something too. It's generally not a good idea to keep the object lock while
calling any function implemented elsewhere.

Also the ringbuffer can only be NULL at this point, remove a useless if block.

And in the sink actually hold the object lock while setting the ringbuffer on
the instance. Code accessing this is expected to use the object lock, so do it
here ourselves too.

riff-media: Error out early if we observe an invalid audio format

riff: Also handle invalid block aligns for raw audio

Fixes audio playback of
http://demo.archermind.com/Test%20Sample/Video/MPEG%204/Divx3/Low-Motion/576-320.avi

Audio and video together is still broken because of other issues.

audio: Fix private header include/dist

We want to dist it, but we don't want to install it.

Fixes make dist/distcheck

Automatic update of common submodule

From ef1ffdc to f2c6b95

video: audio: fix GI annotations for proxy caps function

Add the annotations to parameters that can be null and also for stating
the ownership of the returned caps

tests: audiodecoder: tests for caps query implementation

Copied from videodecoder tests and updated to audio features

audiodecoder: expose getcaps virtual function

Allows subclasses to do custom caps query replies.

Also exposes the standard caps query handler so subclasses can just
extend on top of it instead of reimplementing the caps query proxying.

audiodecoder: implement caps and accept-caps queries

Allows decoders to proxy downstream restrictions on caps.

Also implements accept-caps query to prevent regressions caused by the
new fields on the return of a caps query that would cause the accept-caps
to fail as it uses subset caps comparisons

audioencoder: refactor getcaps proxy function to be reusable

Makes the audioencoder's getcaps function that proxies downstream
restriction available to other elements in the audio module to use it

videodecoder: expose getcaps virtual function

Allows subclasses to do custom caps query replies.

Also exposes the standard caps query handler so subclasses can just
extend on top of it instead of reimplementing the caps query proxying.

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

videodecoder: accept-caps should only require fields from the template

With the new caps query results the caps returned might have extra fields
that are not required by the decoder (framerate for image decoders) and it
causes a regression making, for example, jpegdec reject caps that don't
have framerates.

The accept-caps implementation will do 2 checks:

1) Do subset check with the template caps, making sure all the required
fields that are present on the template are present on the received caps.
2) Do a intersection check with the result of a caps query, making sure
that downstream can accept the fields in the received caps.

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

videoutils: proxy filter when doing a caps query downstream

Allows downstream to use the filter and possibly reduce caps complexity
to speed up negotiation

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

videoutils: return empty if the element has no possible allowed caps

Instead of returning the template caps and having a failure happen
later because there are no possible caps

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

videodecoder: implement caps query

Refactor the encoder's caps query proxying function to a common place
and use it in the videodecoder to proxy downstream restrictions.

The new function is private to the gstvideo lib.

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

configure: require release version of orc now that there is one

ximagesink: clear src and dest rectangles

Now that the center function also takes into account the x and y
coordinates of the dest rectangle, better clear all the fields before
using them.

videopool: update buffer size after video alignment

Update the new buffer size after alignment in the pool configuration
before calling the parent set_config. This ensures that the parent knows
about the buffer size that we will allocate and makes the size check
work in the release_buffer method.

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

audiobasesrc/sink: Add _CAST macros

video: Fix non-default usage of gst_video_sink_center_rect

Make sure we take into account non-0 x/y destination rectangles

examples: improve playback-test help text a little

And allow pipeline type to be specified as string.

pango: Add license/copyright header to header file

Revert "decodebin: Only emit the drain signal for the main decode chain, not any subchains"

This reverts commit a391dfe17f1a325f60e1d51a6d40c1a68eb196de.

It breaks gapless playback: https://bugzilla.gnome.org/show_bug.cgi?id=740045

audiorate: Fill gap events

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

audio: Add error handling to gst_audio_decoder_drain()

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

audioclock: Fix redundant definitions compiler warning

gstaudioclock.c:51:31: error: redundant redeclaration of 'gst_audio_clock_init' [-Werror=redundant-decls]
 G_DEFINE_TYPE (GstAudioClock, gst_audio_clock, GST_TYPE_SYSTEM_CLOCK);

gstaudioclock.c:51:31: error: redundant redeclaration of 'gst_audio_clock_class_init' [-Werror=redundant-decls]
 G_DEFINE_TYPE (GstAudioClock, gst_audio_clock, GST_TYPE_SYSTEM_CLOCK);

audioclock: No need to get the parent class in class_init, G_DEFINE_TYPE does that for us

audioclock: Use G_DEFINE_TYPE instead of a custom get_type() function

rtcpbuffer: fix spelling of word in comment

tests: rtpbasepayload: fix indentation

tests: audiodecoder: fix indentation

tests: audiodecoder: fix broken refcounting in unit test

The set_format vfunc does not pass ownership of the caps
to the decoder, so we mustn't unref the caps there.

gst_event_new_caps() does not take ownership of the caps
passed, so we must unref the caps afterwards.

Fixes leaks when running test in valgrind in 1.4 branch.

video: Update disted orc source files

Revert "video-converter: Fix compiler warning because of missing prototype of non-static function"

This reverts commit 406f32a9468c837a4d71f988de10dc2198a8edc9.

The problem was apparently that my video-orc.h was not updated and did not
include the prototype for that function. Only a "make clean" caused it to
be regenerated.

video-converter: Fix compiler warning because of missing prototype of non-static function

video-converter.c:838:1: error: no previous prototype for function
      '_custom_video_orc_matrix8' [-Werror,-Wmissing-prototypes]

audiodecoder: do not use fixed caps on source pad

decoders can change the caps on their source pads, so they don't
use fixed caps. Having fixed caps can cause renegotiation issues.

videodecoder: do not use fixed caps on source pad

decoders can change the caps on their source pads, so they don't
use fixed caps. Having fixed caps can cause renegotiation issues.

playbin: Do not mix up stream type when getting stream combiner element

We were always returning the video stream combiner whatever stream type
combiner was wanted.

playbin2: always unref the combiner sinkpad when removing the srcpad

Create a function to do the pad cleanup of the GstSourceCombine struct
and use it to not forget to also cleanup the sink pad and fix a memory
leak.

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

video-orc: make RGB pack/unpack faster

Avoid all the merging and splitting and use a pair of shifts and or

videodecoder: Add GST_VIDEO_DECODER_CAST macro

It's used in some macros already, so let's make it exist.

rtspconnection: No remove child if destroyed.

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

reverse-play: fix seek to end when starting reverse

Start reverse playback by actually seeking to the end of
the file.

video-converter: set bits and format after conversion

Update the current format, bits and pstride.

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

video-converter: free dither_lines

Avoid a memory leak

Bump ORC requirement to 4.22.1

We now depend on git commit f1cfa5, "orcc: allow setting custom
backup function"

video-converter: use custom backup function

Use the new orc feature to set a custom backup function.

video-converter: improve matrix8 function

Avoid using a constant.
Avoid doing saturated adds, results are not supposed to overflow here.
Rework the C backup function a little in preparation for custom backup
functions in ORC.

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

audiodecoder: Push pending events before sending EOS.

Segments are added to the pending events, and pushing a segment
is mandatory before sending EOS.

+ Adds a test.

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

oggdemux: Fix seeking before the first frame.

The previous code was setting keytarget to target
to make sure the keyframe found for each pad was
indeed before the target.

Then if target == keytarget, it assumed a keyframe had been
found, which was not the case if target was before the first frame
in the file.

This patch checks that a keyframe was indeed found, and if not
seeks to 0, without bisecting again.

Assuming default gst qa assets in $HOME/gst-validate

seek_before_first_frame.scenario:

description, seek=true, handles-states=true
pause, playback-time=0.0
seek, playback-time=0.0, start=0.0, flags=accurate+flush
seek, playback-time=0.0, start=0.01, flags=accurate+flush
seek, playback-time=0.0, start=0.1, flags=accurate+flush

GST_DEBUG=*theoradec*:2 gst-validate-1.0 playbin \
uri=file://$HOME/gst-validate/gst-qa-assets/medias/ogg/vorbis_theora.0.ogg \
--set-scenario seek_before_first_frame.scenario

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

playbin: Only check sinks which are in >= GST_STATE_READY

Otherwise we endup with bogus caps intersection (from the pad template
caps and not from what the actual hardware/device supports)

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

video-converter: fix chroma resampling check

Decide if we need chroma resampling by checking if we have a progressive
or interlaced chroma resampler.

video-converter: only do dithering when needed

Only do dithering when one of the quantizers is > 1.

audiorate: Use gst_util_uint64_scale_int_round()

Using gst_util_uint64_scale_int() causes slight drift
which accumulates over time.

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

defs: update defs file

videoconvert: add dither-bits option

Fix the dither option.
Add a new option to set the quantizer

video-scaler: add where orc functions could go

Add the disabled orc functions in #if 0 lines for when we can enable
them.

video-converter: add dithering

Use the new dither object to perform dithering.
Add option to select dithering method.
Add option to quantize to a specific value

video-converter: add palette when needed

video-dither: add video dither helper object

Add a new object that implements various dithering methods.

gst-play: do not set system's volume to 100% by default

Only change the volume if requested

alsa: Use EPIPE instead of ESTRPIPE if the latter does not exist

NetBSD does not have ESTRPIPE.

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

Don't compare booleans for equality to TRUE and FALSE

TRUE is 1, but every other non-zero value is also considered true. Comparing
for equality with TRUE would only consider 1 but not the others.

encodebin: Add a way to disable caps renegotiation for output stream format

In some cases, the user might want the stream outputted by encodebin to
be in the exact same format during all the stream. We should let the
user specify when this is the case. This commit add some API in the
GstEncodingProfile to determine whether the format can be renegotiated
after the encoding started or not.

API:
    gst_encoding_profile_set_allow_dynamic_output
    gst_encoding_profile_get_allow_dynamic_output

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

tests: remove libs/video and videoconvert test from valgrind blacklist

Seem to work fine.

tests: don't run orc/* tests under valgrind

They just seem to blow up for some reason that needs investigating.

tagmux: fix criticals when there are no tags at all

test-reverseplay: Use uridecodebin for input

Work with any installed URI handler
Add some more debug output

video-frame: Mapping a frame with inconsistent values between GstVideoMeta and GstVideoInfo is a bug

It will cause the frame to be initialized with inconsistent values that then
later can cause crashes or any other kind of interesting and hard to debug
bugs.

Automatic update of common submodule

From 7bb2bce to ef1ffdc

video-blend: make use of x offset when unpacking overlay image pixels

Now that it's implemented we can use it, which is a minor
optimisation when the image to overlay gets cropped on the
left.

video-format: sprinkle some 'restrict' keywords in pack/unpack functions

In cases where we just call orc directly this is somewhat
superfluous, but let's do it anyway for consistency. In
other cases the compiler can hopefully use this to optimise
memory access a little.

video-format: handle x offset in unpack

Add support for x offset in almost all unpack methods.
Fix naming of source and dest pixels.
Add const to source pixels.

video-format: improve unpack i420

unpack_i420 does not need extra code to handle odd widths, the orc code
already handles it fine.

videoscale: use old property name

Unbreak ABI by changing to the old property name again.

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

decodebin: Analyze source pad before setting to PAUSED for 'simple demuxers'

Before we were setting them to PAUSED and (much) later connecting to
their source pad caps notify signal.

There was a race where that demuxer was pushing a caps and later a buffer
on its source pad when we were not even connected to its source pad caps notify
signal leading to decodebin missing the information and not keeping on
building the pipeline on CAPS event thus the demuxer was posting an ERROR
(not linked) message on the bus. This need to be done for 'simple
demuxers' because those have one ALWAYS source pad, not like usual demuxers
that have several dynamic source pads.

A "simple demuxer" is a demuxer that has one and only one ALWAYS source
pad.

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

decodebin2: Take STREAM_LOCK before sending sticky events.

There was a race where:

1) we would put the element to PAUSED
2) It would get data sent to it from upstream
3) It would thus send caps
3) caps_notify_cb would continue autoplugging
4) caps would flow downstream, the last pad would get exposed
5) we were still not done sending the sticky events

Taking the stream lock on the new element's sinkpad and only
releasing it when sticky events have all been sent prevents
the caps from reaching the source pad of the element before
we're all set.

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

typefindfunctions: detect mp4 common file format variant

Used e.g. by UltraViolet.

alsasrc: debug message fixes

In the same vein as 74e9640a.

video-scale: combine adds when max_taps equals combine size

When the amount of pixels/lines matches the amount we can combine,
combine the adds and multiplies and do the scale as a separate
operation.

video-scaler: combine scaling operations

Combine add and scale of multiple lines/pixels to reduce the amount of
read and writes to temporary memory.

timeoverlay: add "time-line" property

So we can also show running time or stream time, not just the
buffer time stamps.

videoscale: add property to do scaling after gamma-decode

videoscale: add more scaling filters

Adjust the filter parameters so that they use the same number of taps
and method as the old ones.
Add some new filters