resampler: limit max number of taps

Don't use more taps than the input size.

video-converter: add scaling support

Add scaling support for the video-converter object

video-scaler: add video scaler helper object

Add a video scaler object build on top of the resampler. It has
implementation to deal with interlaced video as well as horizontal and
vertical scaling functions.

video: add generic resampler

Add an object that can generate a set of resample filter coefficients.

video-converter: rework the generic converter function

Use a LineCache object to track and process lines between unpack,
upsample, convert, downsample and pack stages. This simplifies the
main core processing function a lot and allows for future additions
easily.

Add support for interlaced formats in chroma up and downsampling.

video-convert: swap src and dest

It is more natural and consistent with other uses.

video-chroma: fix typo

Automatic update of common submodule

From 84d06cd to 7bb2bce

video-blend: segfault when xpos >= video size

When the xpos is given as greater than or equal to the video size,
we get a segfault, due to improper condition.
Hence adding proper conditions.

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

basetextoverlay: segfault when xpos >= video size

When the xpos is given as greater than or equal to the video size,
we get a segfault, due to improper condition.
Hence adding proper conditions.

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

examples: add new appsink example to .gitignore

Revert "decodebin: fix the autoplugging of parser elements"

This reverts commit 2b0d3927410ae24e6b0fce100bd4ebbbe805a66f.

This breaks cases where an actual second parser is required after the parser,
e.g. to do timestamp corrections.

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

Revert "decodebin: Fix locking"

This reverts commit aa94d5dc9aa6ef381da6b60a67f218117c662958.

tests: fix playbin-complex test on big endian

tests: fix expected GstRTSPTimeRange structure size for ABI test for ppc32

Also see https://bugzilla.gnome.org/show_bug.cgi?id=695276

tests: fix adder check on big-endian

rtsp: use generic marshaller

basetextoverlay: Make GstBaseTextOverlay::font-desc readable

Automatic update of common submodule

From a8c8939 to 84d06cd

decodebin: Fix locking

The chain mutex needs to be locked when looking at chain->elements. Move code
around a bit to require only one lock() and unlock().

decodebin: fix the autoplugging of parser elements

If there are two parser elements available for the same media format,
then decodebin is autoplugging an extra capsfilter and parser irrespective
of caps and rank. So restrict the decodebin from autoplugging multiple parser
elements back to back in adjacent positions with in a single DecodeChain
for the same media format.

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

Automatic update of common submodule

From 6e75498 to a8c8939

videotestsrc: assertion error

timestamp_offset is being declared as an int64 variable,
for which the min
value of G_MININT64 is -9223372036854775808
Changing the minimum and maximum limit for the offset variable.

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

decodebin: optimize the code a bit by avoiding unnecessary string comparisons

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

decodebin: Fix typo in comment

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

rtspconnection: call watch notify before freeing any watch resources

This gives control to the notify function allowing it to finish other
watch related functionality.

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

appsink: Fix gst_app_sink_pull() docs to transfer full for the return value

Also we get a GstSample, not a GstBuffer here.

typefind: use gslice for typefine data

Also use our free function in the failure case.

encodebin: fix some leaks in error code path

Fixes test_encodebin_sink_pads_nopreset_static
running under valgrind.

tests: parallelise 'make valgrind'

Use $(MAKE) instead of 'make' inside the Makefile,
otherwise the make will run as if -j1 had been
specified and complain about the job server not
being available, and with $(MAKE) in inherits the
parent make's settings it seems.

Upgrade common submodule for parallel check-valgrind.

riff-media: allow more channel_masks

Allow partial valid channel masks.
Set channel mask to 0 for non-valid channel masks.

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

audio-channels: allow partially valid channel_mask

Since WAVEFORMATEXTENSIBLE allows to have more channels than
bits in the channel mask we should allow this, too, to avoid
loss of information.

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

audiodecoder: should post DECODE errors and not ENCODE

Fix error code for audio decoder

videoblend: Avoid assigning a negative value to a guint

There are some few but certain conditions where it is possible for the
dest_width to be smaller than x. So we check this before assigning a negative
value to src_width, which is a unsigned and would be promoted to a number that
can segfault videoblend.

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

basetextoverlay: Fix segfault when overlay outside the frame

When the textoverlay is set outside the video frame by deltax or deltay the
calculation segfaults, but it is also  unnecessary since it doesn't need to be
displayed. So we should clip the text.

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

pbutils: Rename clock-base/seqnum-base to timestamp-offset/seqnum-offset

To match how they were renamed elsewhere.

playsink: Use correct property enum value for video-filter property installation

videoscale: remove FIXME about NV21 support

NV21 is already supported so removing FIXME about adding support for it.

videotestsrc: add gradient pattern

Makes a gradient between background and foreground color.

video-chroma: improve 4x downsampling coefficients

audioresample: remove unused variables

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

typefindfunctions: Remove leftover #define from 0.10

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

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

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

decodebin: Free factories array when delaying autoplugging due to non-final caps

videoconverter: Free the converter config in free()

decodebin: unref decode pad after usage

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

videoencoder: Stop storing if we received EOS

This was never reset when going from PAUSED->READY and resulted
in encoders being not reusable after EOS. They just rejected any
buffer because they received EOS in their previous life.

The flag wasn't used anywhere except for rejecting buffers after
EOS, and this is now handled by GstPad directly.

vorbisdec: don't reorder streams with channels count greater than eight

vorbis_reorder_map is defined for eight channels max. If we have more
than eight channels, it's the application which shall define the order.
Since we set audio position to none, we just interleave all the channels
without any particular reordering.

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

uridecodebin: Removed setting "iradio-mode" property in the source element

The "iradio-mode" property used to have a default FALSE value in HTTP
source elements but now it should default to TRUE or just do not exist
as a property so it is not really needed to set it any more in
uridecodebin.

Apart from that this code could've never worked as uridecodebin looks for a
string-typed iradio-mode property, but it's a boolean in all sources.

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

design: Add a proposal for handling stereoscopic 3D and multiview

videoencoder: release frame in finish_frame when no output state is configured

Otherwise, frame is leaked.

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

video-converter: add orc optimized matrix8 function

Add an ORC implementation of the matrix8 function.
Regenerate video-orc-dist.[ch]

audio: Fix up a comment in GstAudioBaseSink

Rewrote the comment to not be PulseAudio-specific.

video: Make sure to link against libm

xvimagesink: get rid of unnecessary private struct for pool

ximagesink: get rid of unnecessary private struct for pool

This is not exposed as API after all.

audio: Trivial comment for unhandled MPEG-2 payloading case

The spec mentions a version of the MPEG-2 frame with a base frame and
extension frame. I don't have IEC 13818-3 to figure out what that is,
and don't see any references in search results, so it's a FIXME for now.

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

audio: Fixes for MPEG-2 LSF IEC61937 payloading

The low sample frequency case for MPEG-2 is <=12kHz (the 32kHz number
applies to MPEG-1).

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

audio: correct condition for MPEG case.

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

video: improve YUV -> RGB conversion

Reorganize orc instructions to free up some registers.
We can reuse the ORC code to implement the generic AYUV->ARGB matrix.

videotestsrc: storel is better then copyl

It is better to use storel to splat the variable into the destination.
ORC doesn't know when a variable is last written to so it can't yet optimize
away the copy operation.

videoscale: avoid recalculating values

Avoid recalculating values used multiple times as base of index. Plus some style
fixes.

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

videoscale: support lanczos method for NV formats

Support lanczos scaling method for NV12 and NV21 formats.
Scale the 'Y' plane and scale 'NV' plane.
Implementation for submethods - int16, int32, float and double

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

video: update disted orc backup files

video: convertor -> converter

video: move videoconvert code to video library

Move the conversion code used in videoconvert to the video library
and expose a simple but generic API to do arbitrary conversion. It can
currently do colorspace conversion but the plan is to add videoscale to
it as well.

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

video-color: add gst_video_color_matrix_get_Kr_Kb()

Move the function to get the color matrix coefficients from
videoconvert to the video library.

audiosink: compensate for segment restart with clock's time_offset

When playing chained data the audio ringbuffer is released and
then acquired again. This makes it reset the segbase/segdone
variables, but the next sample will be scheduled to play in
the next position (right after the sample from the previous media)
and, as the segdone is at 0, the audiosink will wait the duration
of this previous media before it can write and play the new data.

What happens is this:
pointer at 0, write to 698-1564, diff 698, segtotal 20, segsize 1764, base 0

it will have to wait the length of 698 samples before being able to write.

In a regular sample playback it looks like:
pointer at 677, write to 696-1052, diff 19, segtotal 20, segsize 1764, base 0

In this case it will write to the next available position and it
doesn't need to wait or fill with silence.

This solution is borrowed from pulsesink that resets the clock to
start again from 0, which makes it reset the time_offset to the time
of the last played sample. This is used to correct the place of
writing in the ringbuffer to the new start (0 again)

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

videopool: add missing annotation for gst_video_buffer_pool_new()

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

videoscale Use stride instead of width in more places

videoscale: Use width instead of stride in buffer offset calculation

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

audioencoder: reshuffle code in error handling

Move the assert to the error handling block at the end of the function so the
the logging is still triggered. Reword the logging slightly and add another
comment to hint what went wrong.

Fixes #737138

videoencoder: log the timestamps if we are unhappy about them

When complaining about the DTS!=PTS on keyframes log the actualy timestamps.

tests: add orc test for videoconvert

gst-play: Fix format string compiler warning

gst-play.c:92:28: error: format string is not a string literal
      [-Werror,-Wformat-nonliteral]
  len = g_vasprintf (&str, format, args);
                           ^~~~~~

example/overlay: Specify minimum gdk version

Avoids deprecation warnings (such as for gtk_widget_set_double_buffered()
which became deprecated from 3.14)

gst-play: add --quiet option to suppress output

basetextoverlay: Do not fail the negotiation if query fails

The allocation query failure doesn't mean that the negotiation
has failed as the element can allocate buffers itself.

Instead, only fail if the pads are flushing and the allocation
query failed.

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

videoscale: Added NV support for 4Tap resize

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

playbin: Don't leak input-selector sinkpads

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

audiodecoder: Simplify code a bit

streamsplitter: do not leak events when flushing them

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

audioencoder: do not leak events when flushing them

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

videodecoder: do not leak events when flushing them

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

videoencoder: do not leak events when flushing them

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

audiodecoder: extend flush_events test to check for event leaks

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

audiodecoder: Don't leak events

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

audiocdsrc: do not leak uid after parsing TOC select event

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

typefind: correct the condition for irap flag

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

playsink: Add audio/videoconvert in front of the audio/video-filters

audioresample and videoscale is something the application will have to do if
required, but we can at least help here by adding the
audioconvert/videoconvert elements.

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

video-frame: Don't ref buffers twice when mapping

app: Add FIXME comment for making the instance/class structs private

appsrc: fix recent ABI breakage caused by GstAppSrc structure size increase

Also fixes 'make check'.

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

videodecoder: do not leak pool and allocator in error case

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

videofilter: Use new GST_VIDEO_FRAME_MAP_FLAG_NO_REF

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

video-frame: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF

This makes sure that the buffer is not reffed another time when
storing it in the GstVideoFrame, keeping it writable if it was
writable.

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

videofilter: Unref buffers before calling the transform_frame functions

GstVideoFrame has another reference, so the buffer looks unwriteable,
meaning that we can't attach any metas or anything to it

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

audiobasesink: Fix deadlock caused by holding object lock while calling clock functions

Issue:
During a PAUSED->PLAYING transition when we are rendering an audio buffer in AudioBaseSink
we make adjustments to the sink's provided clock i.e. fix clock calibration using the external
pipeline clock, within "gst_audio_base_sink_sync_latency function inside gstaudiobasesink.c".
For the calibration adjustment we need to get the sink clock time using "gst_audio_clock_get_time".
But before calling "gst_audio_clock_get_time" we acquire the Object Lock on the Sink. If sink is
a pulsesink, "gst_audio_clock_get_time" internally calls "gst_pulsesink_get_time" which needs to
acquire Pulse Audio Main Loop Lock before querying Pulse Audio for its stream time using
"pa_stream_get_time". Please see "gst_pulsesink_get_time in pulsesink.c".

So the situation here is we have acquired the Object lock on Sink and need PA Main Loop Lock.
Now Pulse Audio Main Thread itself might be in the process of posting a stream status
message after Paused to Playing transition which in turn acquires the PA Main loop lock and
needs the Object Lock on Pulse Sink. This causes a deadlock with the earlier render thread.

Fix:
Do not acquire the object Lock on Sink before querying the time on PulseSink clock. This is
similar to the way we have used get_time at other places in the code. Acquire it after the
get_time call. This way PA Main loop will be able to post its stream status message by
acquiring the Sink Object lock and will eventually release its Main Loop lock needed for
gst_pulsesink_get_time to continue.

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

appsrc: Add example that shows gst_app_src_push_sample() usage

appsrc: Add push_sample() convenience function for easy appsink -> appsrc use

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