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

video-resampler: remove print

video-resampler: improve variable taps

Improve quality of variable taps on all methods by reusing the lanczos
parameters where possible.

video-resampler: Fix lanczos parameters for variable taps

when using variable taps and when we are limiting the number of taps,
recalculate the lanczos parameters to match the clamped value.
Set the max number of taps to 128

playsink: Reset mute property of the sink to playsink's value when setting up the audio chain

Otherwise the following can happen:
1. set mute=true
2. play media1 (Ok)
3. play media without audio (audiochain removed)
4. play media2 (audiochain created, mute=*false*)

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

discoverer: fix typo in header file

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

pbutils: add description for audio/x-audible

typefind: improve 'audible' audio typefinder a little

Don't return NEARLY_CERTAIN just based on 4 bytes.
Also change media type to audio/x-audible.

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

typefindfunctions: add audio/audible typefinder

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

alsa: Change the log messages in xrun_recovery() from DEBUG to WARNING

xrun_recovery() runs when there is an error

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

video-converter: keep track of required temp lines

Make a small object to hold a pool of allocated temp lines.
Keep track of how many temp lines each conversion stage needs and use
this to allocate just enough temp lines from the temp lines object. from
the temp lines object.

video-converter: use err line in fastpath

Use the error line for temporary storage in the fastpath so that we
don't have to allocate any other temp lines.

videoencoder: don't complain about PTS != DTS on keyframes

It is valid for streams with b-frames

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

video-converter: handle mixed interlaced

When dealing with mixed interlaced, setup a scaler and chroma-resampler
for both interlaced and progressive frames and switch between them
depending on the interlace mode of the input frame.

video-converter: Cleanup options parsing

Cleanup option parsing
Add some debug

video-converter: there is no need to apply x offset to temp lines

video-scaler: ensure both fields have the same number of taps

video-converter: rework the options a little

Rework the options a little to make it nicer to set defaults.

video-resampler: add option to limits taps

Add an option to limit the number of taps to use in automatic mode. The
problem is that for lanczos, we might use more taps than what we can
handle with the current precision.
Rework the other options a little to make it nicer to set defaults.

video: update orc files

win32: Update defs file

rtspconnection: fix warning on param name mismatch

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

tests: Add reverse playback verification test

Plays a requested URI forward to EOS, then backward and
checks that the same timestamp range(s) are covered.

videorate: Operate in a zero-latency mode if drop-only is set to TRUE

There's no reason why we would have to wait for the next buffer to decide
whether to output the current one or not. We just have to check if the
current one is earlier than our expected next time, which is the previous
frame timestamp plus the expected frame duration.

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

video-converter: Use correct enum, GstVideoFormat instead of GstFormat

video-converter: fix size check

Add some debug, fix size check that decides what scaling to do first and
when to do conversion.

video-converter: avoid primaries conversion when asked

Don't do conversion between primaries when the option is disabled.
Only do some matrix code when needed.

video-info: add a note about subsampled formats

Add a note about gst_video_info_set_format() and interlaced formats.

video-info: handle interlaced size correctly

Refactor GstVideoInfo init, make function to set default colorimetry.
Call fill_planes after we configure the GstVideoInfo with parameters
from the caps.
The size of the chroma planes for interlaced vertically subsampled
formats needs to be rounded up to 2, we have 2 fields with each
the same anount of chroma lines.

video-color: return FALSE on unparsable colorimetry

video-format: handle unpack interlaced subsampled formats

For interlaced vertically subsampled formats the check for even lines
needs to take into account the two fields.

video-scaler: fix interlaced shift

video-converter: keep a small backlog of lines

Allow lines to jump backwards slightly, usefull for interlaced content.