v4l2object: Fix colorimetry transfer lookup for 4K video

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

v4l2src: Check if caps have changed after try_fmt

try_fmt will update the caps colorimetry and interlace-mode. Before this
call, those field are missing. The caps equality check was always
failing when a spurious reconfigure event was received.

qtmux: Allow MPEG layer 1/2, AC3 and Opus in qtmux

qtmux is supposed to be the muxer that allows all formats,
with others (mp4mux and friends) being profile-restricted.

rtph265depay: fix keyunit detection

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

jpegdec: Fix decoding of streams that don't signal exactly twice the height

... and also progressive streams.

jpegdec: Handle interlaced MJPEG streams

These come with two JPEG images per buffer of half height than signalled
in the container.

Changes based on Tim-Philipp Müller's 0.10 branch:
https://cgit.freedesktop.org/~tpm/gst-plugins-good/log/?h=jpegdec-interlaced

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

dcaparse: Really fix "usage before unmap"

Previous patch would try to unref a buffer that was pushed downstream.

Instead only unref when/if needed and keep usage of the cleanup: goto
block

dcaparse: Don't unmap buffer before accessing data from it

The previous patch added a check for a substream header after
gst_buffer_unmap(), which is incorrect.

dcaparse: preserve DTS HD substream

rtpsbcpay: Fix some tabs that crept in somehow

rtpbin: Also log local and SR RTP running times when doing ntp-sync=true

rtpbin: also create session when creating the send_rtcp_src_%u pad

If one requests the send_rtcp_src_%u pad before a recv_rtcp_sink_%u pad,
the session/pad would never be created and NULL was returned.
Switching the request order would work.

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

tests: mpg123audiodec: add files needed by unit tests

tests: add basic unit test for twolame as well

tests: lame: fix build

tests: ignore another binary

twolame: hook up to build system

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

Moving twolame mp2 encoder plugin from -ugly

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

lame: hook up to build system

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

Moving lame mp3 encoder plugin from -ugly

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

meson: Link mpeg123audiodec test against gstfft

Fixing build error:

/run/build/gst-plugins-good/_flatpak_build/../tests/check/elements/mpg123audiodec.c:150: undefined reference to `gst_fft_s32_new'
/run/build/gst-plugins-good/_flatpak_build/../tests/check/elements/mpg123audiodec.c:151: undefined reference to `gst_fft_s32_window'
/run/build/gst-plugins-good/_flatpak_build/../tests/check/elements/mpg123audiodec.c:151: undefined reference to `gst_fft_s32_fft'
/run/build/gst-plugins-good/_flatpak_build/../tests/check/elements/mpg123audiodec.c:147: undefined reference to `gst_fft_s32_free'

tests: tagschecking: remove gst-check-xmp-* temp files when done

Also fix temp file creation a bit.

docs: update for changes in git

mpg123: add to docs

mpg123: hook up to build system

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

Moving mpg123 plugin from -ugly

Automatic update of common submodule

From 48a5d85 to 3f4aa96

ac3parse: fix not-linked handling causing glitches when selecting stream

Fix chain function not handling not-linked from baseparse.

When an input data is separated into 2 buffers, the second buffer
would not be pushed into the adapter if baseparse returns not-linked
for first buffer.

This caused glitches when switching streams and selecting
a stream that was previously unselected.

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

goom2k1: Convert source files to UTF-8

Causes problems with the new gtk-doc 1.26 otherwise,
but is a good idea in any case.

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

wavparse: Add support for growing WAV files

With some fixes by me.

rtpsbcpay: Fix compile error

rtpsbcpay: fix if buffer size exceeds MTU

The plugin queued buffer data if not all buffer data fit
into a single RTP packet. Now RTP packets are pushed as long
as enough data is available.

vpxenc: discard frames that have been dropped by libvpx

This fixes a memory leak. When dropframe-threshold has been set,
libvpx may output less frames than the input ones, which causes
some GstVideoCodecFrames to queue up in GstVideoEncoder's internal
frame queue with no chance of ever being all released. And because
the frames keep references to the input buffers, the input buffer
pool keeps allocating new buffers and memory usage grows very fast.
For example the following pipeline's memory usage grows at a rate
of about 1GB per minute!

videotestsrc ! capsfilter caps=video/x-raw,width=1920,height=1080,framerate=30/1,format=I420 ! \
  vp8enc target-bitrate=1000000 end-usage=cbr dropframe-threshold=95 ! fakesink

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

rtpstats: fix unsigned integer comparisons.

Callers of the API (rtpsource, rtpjitterbuffer) pass clock_rate
as a signed integer, and the comparison "<= 0" is used against
it, leading me to think the intention was to have the field
be typed as gint32, not guint32.

This led to situations where we could call scale_int with
a MAX_UINT32 (-1) guint32 as the denom, thus raising an
assertion.

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

taglib: use -fvisibility=hidden with this C++ plugin in meson too

Also pass args as cpp_args.

qtdemux: allow larger files

For really long files such as contiguous recordings of a whole day, the
50MB limit is not sufficient.

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

qtdemux: Fix offsets for reading lpcm specific fields

We were reading at the completely wrong positions, 16 bytes later in the
data.

Also add support for high-aligned samples.

meson: don't export symbols by default

Only plugin entry points should be exported.

Currently plugins might export more symbols with
the meson build, as we don't have the exports
regexp there that we pass to libtool.

wavparse: Fix memory leak in wavparse element

Fixing of leaking the text field of the GstWavParseNote and
GstWavParseLabl structure.

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

v4l2bufferpool: Don't mark jpeg frames as deltas

JPEG formats are encoded, but they never have keyframe flag. But in
fact they are keyframes

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

osxvideo: rename library according to the plugin name

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

v4l2bufferpool: Don't drop buffer ref on qbuf

This function no longer take ownership of the buffer.

CID 1414800

v4l2: Enable VP9 format

This was missing, preventing the encoder and decoder to work
properly. This also adds support for camera that would produce
VP9 (if that exists).

v4l2: Remove spurious CATEGORY_EXTERN

These have been copy pasted all over the place and are not used anymore.
All object have it's own category now. This fixes build warning since
the VP9 decoder had vp8 category declared.

v4l2videoenc: Move the profile/level negotation in the base class

This removes duplicated code across different codec.

v4l2videoenc: Turn gst_v4l2_is_video_enc into a helper

This reduces the amount of code needed in each codec class.

v4l2: Add VP8/9 encoder support

v4l2object: Use mmap64 to match libv4l2 signature

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

v4l2bufferpool: Copy flags and timestamp when importing

Whenever we import from downstream pool (userptr or dmabuf-import), we
should copy over the flags and timestamp, otherwise downstream will not
get proper synchronization or will not be able to notice frames that has
corruption in it.

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

v4l2: Add H263 Encoder support

v4l2: Add missing no-inst header

v4l2: Add interface for MPEG4 encoding

v4l2: Ignore register issue and keep probing

Don't stop registering the other dynamic plugins if one registration
fails.

mulawdec: Unmap input buffer if failing to map the output buffer

alawdec: Fix Memory leak in error case

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

v4l2: Update external files with latest

This is copied from the linux kernel with only some include changes so
it works outside the kernel headers.

matroskamux: For audio tracks, take the default duration from the first buffer

... if we don't have any better idea from the caps. This allows writing
SimpleBlocks for a majority of audio streams where the duration of
frames is usually fixed. And as a side effect, allows VLC to play
streams with Opus as it only works with SimpleBlocks currently:
  https://trac.videolan.org/vlc/ticket/18545

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

v4l2: Fix compilation without libv4l2

v4l2: Keep ref to element in allocator/pool

Removes the FIXME/Question in the buffer pool and add a ref to the
element in the GstAllocator too. This ref is strictly required to keep
the GstV4l2Object structure around.

v4l2object: Removed unused members

v4l2: Add run-time environment to enable libv4l2

The library has started preventing a lot of interesting use cases,
like CREATE_BUFS, DMABuf, usage of TRY_FMT. As the libv4l2 is totally
inactive and not maintained, we decided to disable it. As a convenience
we added a run-time environment that let you enable it for testing.

  GST_V4L2_USE_LIBV4L2=1

This of course only works if you have enabled libv4l2 at build time.

jpegenc: declare quality property changeable in PLAYING state

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

v4l2object: Fix colorimetry validation

While not documented, gst_video_colorimetry_matches() only accepts well
known names. Looking at the code and unit test, this seems to be on
purpose, so fixing by parsing the string and compating the colorimetry
structures.

v4l2encoder: Fix negotiation error handling

The subclass negotiated function will call set_format, if that fails the
pool will not be created. We ended up with an assertion.

GStreamer-CRITICAL **: gst_buffer_pool_set_active: assertion 'GST_IS_BUFFER_POOL (pool)' failed

v4l2src: Speedup camera startup by skipping try_fmt

In this commit, we enabled skip_try_fmt_probes quirk in order to speed
up the start which is known to be disastrously slow with certain USB
cameras.

This has the side effect that we needed to rewrite the entire
negotiation process in a way that we iterate over the possible caps
until we find one that works.

The new negotiation method consist of extracting a preferred structure
from the peer caps and using this to fixate and sort the caps. To
reflect the old behaviour, we sort all resolution strictly bigger
to the preferred one with the closes one first. The rest is appended,
keeping the same order. We then normalize the caps in case there was
some list of interlace-mode or colorimetry left. We finally iterate
over all fixed caps and try it. 99% of the time, the first or the
second one should work, whit the result of a single S_FMT being issues.

From there, it will be relatively easy to introduce new negotiation
algorithm. The current algorithm is made for optimal image quality
with a scaling sink that sets it's window resolution as preference.
This the case if for:

  v4l2src ! videoconvert ! videoscale ! ximagesink

Other strategy would be needed to optimize for non-scaling sink like
ximagesink or kmssink when the driver does not scale.

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

v4l2object: Introduce quirk to skip slow probes

skip_try_fmt_probes quirk is set, V4L2 object will not probe for
interlace-mode and colorimetry to avoid relying on try_fmt. This quirk
will be used by v4l2src to avoid desastrous startup time with slow
USB webcams.

When this quirk is enabled, caller will have to iterate over the
negotiated caps as it may contains unsupported formats. If the peer
didn't choose a specific interlace-mode, or colorimetry, the value
chosen by the driver is set into the caps. For this reason, when this
mode is enabled, gst_v4l2_object_set_format() will require writable
caps.

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

v4l2object: always set the GstV4l2Error on error

Some of the error case were conditional to using try_fmt or not.
This is slightly unexpected, always set the error so the caller
can decide.

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

v4l2object: Minor style fix and useful trace

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

v4l2object: Fix try/s_fmt errors

According to the spec,TRY_FMT cannot return EBUSY, though it can
return EINVAL if it was not possible to update the format to
something supported.

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

v4l2object: Validate colorimetry in S/TRY_FMT

This is in preparation for removing slow TRY_FMT probes for
colorimetry. As we won't have tried that colorimetry we cannot
assume the driver will accept it.

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

v4l2object: Validate field in S/TRY_FMT

This is in preparation from removing the slow TRY_FMT probes for
interlacing. As we won't have tried that interlace-mode already
we need to validate that the driver isn't refusing it.

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

tests: icles: fix build

Can't do additions/subtractions on void* pointers.

tests:icles: Fix previous patch by implementing our memmem

Using the string version of it will fail on '\0'.

tests:icles: Do not use memmem GNU extension function

As it is not avalaible on windows/msvc and we can use pure GLib for that

directsound: Fix .c file name in Makefile

This was broken by accident, bad search and replace.

waveform: Fix DLL name to match plugin name

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

directsound: Fix DLL name to match plugin name

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

qtdemux: preferably send open-ended segment rather than repeated segment events

qtdemux: fix seeking in fragmented file without mfra random access info

... which no longer worked due to unconditionally clearing sample info and
ending up in inconsistent state.  Let's tread a bit more carefully and also
allow for the old seek handling that resorts to scanning if no mfra info
is available.

v4l2videodec: add some useful debug messages

Add a couple of useful debug traces , they happened to be useful to
debug/investigate a 4K video playback issue with v4l2, so let's make these
changes more permanent.

Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>
https://bugzilla.gnome.org/show_bug.cgi?id=785109

v4l2: Fix 4K colorimetry

Since 1.6, the transfer function for BT2020 has been changed from BT709
to BT2020_12. It's the same function, but with more precision. As a side
effect, the V4L2 colorpsace didn't match GStreamer colorspace. When
GStreamer ended up making a guess, it would not match anything supported
by V4L2 anymore. This this by using BT2020_12 for BT2020 colorspace and
BT2020 transfer function in replacement of BT709 whenever a 4K
resolution is detected.

v4l2object: Only check CROPCAP for par once

The pixel aspect ratio is documented to not change unless the TV
Standard is changed. So this mean that this will be uniform across all
possible format and resolutions.

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

Revert "matroskamux: adjust unit test to modified behaviour"

This reverts commit 8fe478c8a7746cd2c63f20d23e97e26e1a0e6192.

We're back to previous behaviour

matroskamux: add properties to control cluster duration

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

v4l2: UVC driver is named uvcvideo these days

The quirk to avoid probing interlacing didn't work anymore as the driver
is now name uvcvideo. This should slightly speed up camera startup.

v4l2object: Remove unused defines

v4l2: Make gst_v4l2_get_capabilities static

It's not used outside of v4l2_calls.c

v4l2: Merge v4l2_calls.h into gstv4l2object.h

First step of a larger cleanup, all function from v4l2_calls are in fact
methods on GstV4l2Object. This split makes the code really confusing.
This also remove no longer unused macros.

mpg123audiodec: fix caps leak

The pad template takes its own ref, so we should unref the caps.

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

meson: po: use glib preset and read language list from LINGUAS

Supported since meson 0.37, so we can use it now.

v4l2object: Trace unknown fourcc as text

This makes it easier to find out what is not supported.

v4l2: Don't probe for unneeded format

For v4l2videodec/enc, we generate elements per formats, and in
this case we can speed up the start up by only probing the format
we care about.

v4l2videodec: Implement stable element names

Before that, each m2m node would be wrapped as a single, multi-format
decoder element. As a unique name was needed, we where using the device
name, which changes between re-boots. This led to unpredictable element
names. In this patch, we generate an element per codec, using
v4l2<codec>dec name. If there is multiple decoder for the same format,
the following elements will be named v4l2<node><codec>dec.

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

souphttpsrc: Post an element message with the HTTP headers on the bus too

Instead of just sending a sticky event with them downstream. This allows
getting the HTTP headers easily in the application, and especially also
on errors.

qtdemux: Fix parsing of RLE depth

Regression introduced by 86b427dc70562f891a551ffc9f96cefe1cafcddd

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

shout2: use gint and guint in place of int and uint

this fixes a compilation error with gcc 7.1.0 on mys2 where uint is not defined

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

rtpgsmpay: fix accidental garbage data before actual payload

Do not allocate payload size outbuf if appending payload buffer.

The commit 137672ff1824948bda4b1b1967de8c24a0055b67 attached payload
to the output buffer but forgot to remove payload allocation.  That
effectively doubled payload size and add zero'ed or random bytes.

Makes the following pipeline work again:

gst-launch-1.0 -v audiotestsrc wave=2 ! gsmenc ! rtpgsmpay ! rtpgsmdepay ! gsmdec ! autoaudiosink

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

matroskademux: segment seek position is expressed in buffer time

... so it need not be corrected again for stream start

avidemux: provide average bitrate tag

examples: v4l2: fix wrong initializations brought by 4e8ad583022671c5

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