rtpsession: Add a warning if an empty RTCP packet is tried to be sent

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

vpxdec: Use GstMemory to avoid copies

With the VPX decoders it's not simple to use downstream buffer pool,
because we don't know the image size and alignment when buffers get
allocated. We can though use GstAllocator (for downstream, or the system
allocator) to avoid a copy before pushing if downstream supports
GstVideoMeta. This would still cause a copy for sink that requires
specialized memory and does not have a GstAllocator for that, though
it will greatly improve performance for sink like glimagesink and
cluttersink. To avoid allocating for every buffer, we also use a
internal buffer pool.

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

aacparse: Avoid over-skipping when checking LOAS config

There might be multiple LOAS config in a row in a full frame. The first
one might be a multi-layer config (which we can't properly parse yet)...
but then followed by a valid (single-layer) one.

The code was previously skipping whole frames (instead of just the LOAS
config we failed to read) resulting in multiple frames (seen up to 6s in
some situation) being dropped before finally getting the configuration.

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

avidemux: Properly set SPARSE stream flags for subpicture/subtitle

And while we're at it, also detect 'DXSA' as being a variant fourcc
of 'DXSB' for XSUB

tests: souphttpsrc: grammar fix

tests: souphttpsrc: switch shoutcast stream provider

Fixes failing ICY test. Previous provider has
streaming disabled outside UK.

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

avimux: don't crash if we never got audio caps before stopping

auds.blockalign is set once the first caps arrive. If
gst_avi_mux_stop_file() is called before this happens then auds.blockalign
is zero and gst_avi_mux_audsink_set_fields() cause a crash:
[...]
avipad->parent.hdr.rate = avipad->auds.av_bps / avipad->auds.blockalign;
[...]

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

v4l2bufferpool: don't block when resurecting a buffer

When we are resurecting a buffer, don't block. instead let us copy a
buffer.

wavparse: remove extra variable to improve readability

Makes it easier to see that the event is being replaced/unrefed

wavparse: respect seqnum in seek events

Propagate the original seek seqnum to events originated from
seeking to make sure they have the same value

wavparse: flush upstream when seeking in pull mode

Makes sure upstream will unblock and return the thread so that
seeking can continue

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

rtph264pay: add "send SPS/PPS with every key frame" mode

It's not enough to have timeout or event based SPS/PPS information sent
in RTP packets. There are some scenarios when key frames may appear
more frequently than once a second, in which case the minimum timeout
for "config-interval" of 1 second for sending SPS/PPS is not sufficient.
It might also be desirable in general to make sure the SPS/PPS is
available with every keyframe (packet loss aside), so receivers can
actually pick up decoding immediately from the first keyframe if
SPS/PPS is not signaled out of band.

This patch adds the possibility to send SPS/PPS with every key frame. This
mode can be enabled by setting "config-interval" property to -1. In this
case the payloader will add SPS and PPS before every key (IDR) frame.

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

rtph264pay: change config-interval property type from uint to int

This way we can use -1 as special value, which is nicer than MAXUINT.
This is backwards compatible even with the GValue API, as shown by
a unit test.

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

qtdemux: add support for Opus

Add support for demuxing Opus encapsulated in MP4 files, based on the
following spec: https://www.opus-codec.org/docs/opus_in_isobmff.html

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

qtdemux: use macro for codec_name

Use _codec() macro instead of duplicating code.

v4l2: videodec: choose format from caps

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

v4l2: add gst_v4l2_object_probe_caps

Add a variant of gst_v4l2_object_get_caps that bypasses the probed_caps cache.

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

v4l2-probe: Skip devices without supported formats

v4l2: Track /dev/video* to triggered required probe

If something in /dev/video* get added, removed or replaced, we need to
probe the devices again in order to ensure the dynamic devices are up to
date.

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

rtpmanager: rtpsession: don't send empty RTCP packets

generate_rtcp can produce empty packets when reduced size RTCP is turned on.
Skip them since it doesn't make sense to push them and they cause errors with
elements that expect RTCP packets to contain data (like srtpenc).

qtdemux: restore the segment on case of soft reset

When seeking back to restore the mdat position a flush is pushed
through and it resets downstream segment information. Make sure
that after the flush (that does a soft reset) a segment will
be pushed again

Fixes regressions spotted at
https://ci.gstreamer.net/job/GStreamer-master-validate/2100/

multifilesink: fix spelling of variable

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

qtdemux: unite duplicate FourCC

Unite in fourcc.h the FourCCs that are used twice or more in qtdemux

v4l2: Fix capture/output-io-mode properties

There was some miss-match in the implementation. This makes it
concistent, though functionally it worked, except the video decoder
output-io-mode getter.

atoms: remove unused argument of build_mov_wave_extension()

AtomTrak * trak argument of build_move_wave_extension() isn't used.
Removing it.

qtdemux: remove duplicate FourCC

Use the available FourCCs in fourcc.h instead of duplicating them.

isomp4: centralize all FourCC

10 FourCCs generated with GST_MAKE_FOURCC() in gstqtmux.c and atoms.c
already exist in fourcc.h. Don't duplicate these and use them directly.
Plus moving 6 to fourcc.h, to centralize them all.

matroska/webmmux: fix outdated example launch lines

Update gst-launch-0.10 lines to gst-launch-1.0

isomp4: add support for Opus in mp4mpux

Add support for muxing MP4 files containing Opus. Based on the spec
detailed here:
https://www.opus-codec.org/docs/opus_in_isobmff.html

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

qtdemux: Replace tabs with spaces

qtdemux: Cast to signed integers to prevent unsigned compare between negative and positive numbers

This fixes seeking if the first entries in the samples table are negative. The
binary search would always fail on this as the array would not be sorted if
interpreting the negative numbers as huge positive numbers. This caused us to
always output buffers from the beginning after a seek instead of close to the
seek position.

Also add a case to the comparison function for equality.

matroskamux: remove duplicate check

We want 1 or 2 streamheaders, the check  if (bufarr->len != 1 &&
bufarr->len != 2) is enough. Not need to check if bufarr->len is <= 0 or
> 255.

souphttpclientsink: Fix error leak and handle error

g_thread_try_new allows for possiblity of failures. In case it fails,
error is not handled and leaked.

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

rtpgstdepay: Properly handle backward compat for event deserialization

Actual code is checking for a NULL terminator and a ';' terminator,
for backward compat, in a chained way that cause all events being rejected.
The proper condition is to reject the events when terminator isn't
in ['\0', ';'] set.

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

tests: rtp-payloading: Test for handling of custom events in rtpgst

Add a simple test that checks proper serialization/deserialization
of custom events with rtpgstpay and rtpgstdepay.

vpxdec: Use threads on multi-core systems

This adds an automatic mode to the threads property of vpxdec in order to
use as many threads as there is CPU on the platform. This brings back
GStreamer VPX decoding performance closer to what is achieved by other
players, including Chromium.

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

qtdemux: only send initial gaps for non-fragmented streams

It would be unusual to have the header segment with an 'edts' atom
indicating gaps at the beginning when handling fragmented streams.

The header usually doesn't contain any timestamping information, this
should come from the playlist/manifest and the segments with media
in those scenarios.

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

Revert "Revert "qtdemux: respect qt segments in push-mode for empty starts""

This reverts commit d842ff288a9d01214a046becbfd9cbff3a4acea0.

This was reverted by accident

udpsrc: Add "loop" property for enabling/disabling multicast loopback

On POSIX, IP_MULTICAST_LOOP is a setting for the sender socket. On Windows it
is a setting for the receiver socket. As such we will need it on udpsrc too to
allow filtering out our own multicast packets.

Revert "qtdemux: respect qt segments in push-mode for empty starts"

This reverts commit 142d8e2d23e5602e7382977af1043d621625f8c8.

qtdemux: Fix string memory leak

The string got using g_strdup_printf will be allocated memory
and should be freed after use.

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

v4l2/object: remove unnecessary NULL check before g_free()

osssrc: remove unnecessary NULL check before g_free()

sunaudiosrc: remove unnecessary NULL checks before g_free()

wavparse: remove unnecessary NULL checks before g_free()

matroskamux: remove unnecessary NULL checks before g_free()

matroska/read-common: remove unnecessary NULL checks before g_free()

isomp4/atoms: remove unnecessary NULL checks before g_free()

rtp/theorapay: remove unnecessary NULL checks before g_free()

rtp/vorbispay: remove unnecessary NULL checks before g_free()

rtp/jpegpay: remove unnecessary NULL checks before g_free()

rtpgstpay: remove unnecessary NULL checks before g_free()

rtspsrc: remove unnecessary NULL checks before g_free()

flxdec: remove unnecessary NULL check before g_free()

effectv/optv: remove unnecessary NULL checks before g_free()

effectv/shagadelictv: remove unnecessary NULL checks before g_free()

effectv/ripple: remove unnecessary NULL checks before g_free()

effectv/radioac: remove unnecessary NULL checks before g_free()

effectv/streak: remove unnecessary NULL check before g_free()

shout2: remove unnecessary NULL checks before g_free()

vp9enc: remove unnecessary NULL check before g_free()

vp8enc: remove unnecessary NULL check before g_free()

souphttpsrc: remove unnecessary NULL checks before g_free()

v4l2object: add support of NV16, NV61 and NV24 formats

Mapped respectively to V4L2_PIX_FMT_NV16/V4L2_PIX_FMT_NV16M,
V4L2_PIX_FMT_NV61,V4L2_PIX_FMT_NV61M and V4L2_PIX_FMT_NV24 v4l2 formats.

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

splitmuxpartreader: Fix GCond leak

inactive_cond is not being cleared resulting in memory leak.

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

jpegdec: fix output state memory leak

When jpeg_finish_decompress is called, output state reference is being created.
But if there is any failures in finishing decompress, it jumps to setjmp,
and at that point state was not referenced. Resulting in leak of output state.
Hence adding another setjmp after output state is referenced.
Similarly adding another setjmp to unmap the frame in case error happens before
finish_decompress

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

qtdemux: respect qt segments in push-mode for empty starts

In push-mode it is hard to support qt segments overall but it is
possible to support when the file isn't heavily edited but just contain
a segment to indicate a gap at the beginning. This also allows properly
timestamping data that has negative DTS in push-mode.

It is relevant to support those for 2 scenarios:

1) fragmented streaming
2) HTTP playback of 'regular' mp4

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

pulse: Don't leak caps and structures in the device provider

rtpmanager: Document properties that are expressed in bits per second

This changed in 928cd110bcea5d143cab3ea747991851d52ecbad and
73c0c2920f9aca96982a4de0c20b3417aa148b81 but was not documented.

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

rtpmanager: Trivial gst-indent fixes

qtdemux: support for cenc auxiliary info parsing outside of moof box

When the cenc aux info index is out of moof boundaries, keep track of
it and parse the beginning of the mdat box, before the first sample.

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

matroskademux: Use codecutils helpers for creating Opus caps

Also fix up codec data with values from the container.

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

matroskademux: There is no multistream field for Opus anymore

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

matroska/webmmux: Support Opus in webmmux and VP9 in matroskamux

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

matroskademux: Parse and handle CodecDelay, SeekPreroll and DiscardPadding

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

matroskamux: Write CodecDelay, DiscardPadding and SeekPreroll for Opus

And also adjust timestamps and durations according to the codec delay, both
should include it for whatever reason.

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

matroskamux: Opus headers are not in-band

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

v4l2: Set O_CLOEXEC on the device fd

This is needed to make sure that child processes don't inherit the video
device fd which can cause problems with some drivers.

rtpmanager: switch G_GINT64_FORMAT for GST_STIME_ARGS

No need to use G_GINT64_FORMAT for potentially negative values of
GstClockTimeDiff. Since 1.6 these can be handled with GST_STIME_ARGS.
Plus it creates more readable values in the logs.

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

rtpmanager: use GST_STIME_ARGS for GstClockTimeDiff

No need to manually handle negative values of diff, GST_STIME_ARGS does
exactly this.

videomixer: use GST_STIME_ARGS for GstClockTimeDiff

No need to manually handle negative values of diff, GST_STIME_ARGS does
exactly this.

deinterlace: use GST_STIME_ARGS for GstClockTimeDiff

No need to manually handle negative values of diff, GST_STIME_ARGS is
available for this.

audiochebband: Fix typo in example pipeline

Fix typo in example pipeline.

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

v4l2: fix double-unref in the v4l2 device provider

matroskamux: don't drop JPEG frames that only have PTS but no DTS set

For the MS/VfW codec ids, we want to write DTS timestamps instead
of PTS because that's what everyone else seems to do (and it's also
how it is in AVI). So for those input formats we use the buffer DTS
instead of the PTS. However, if there's no DTS set but only the PTS
then just take the PTS instead of dropping the input buffer. This
is useful especially for I-frame only codecs like JPEG and huffyuv,
but should also be fine as fallback in general.

Fixes regression with input JPEG frames that only have PTS set on them.

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

tests/check/splitmux: test that the release_pad vfunc of splitmuxsink actually releases pads

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

splitmuxsink: do not destroy the multiqueue & muxer when going to NULL

Instead, delay it until all request pads have been released. This is
because the release_pad() vfunc requires the multiqueue and muxer to
be there in order to release their request pads as well. If those
elements are destroyed earlier, release_pad() does not work, no
pads are released and some resources are leaked.

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

matroskademux: Read buffer timestamp *after* actually setting it

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

scaletempo: Fix handling of rate < 0

We have to reverse all samples in a buffer before processing them to properly
have continuous data from one buffer to another. As a result we will have a
negative applied rate and a rate of 1.0.

Also make sure that input buffers are correctly clipped to the segment,
otherwise our calculations are going to go wrong.

Also copy over the segment event's sequence number to the output segment while
we're at it.

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

deinterlace: break as soon as non-interlaced if found

It looks for a non-interlaced entry on the filter caps, break
as soon as one is found to avoid wasting cpu

deinterlace: implement accept-caps

Implement accept-caps handler to avoid doing a full caps query
downstream to handle it.

This commit implements accept-caps as a simplification of the _getcaps
function, so it exposes the same limitations that getcaps would.
For example, not accepting renegotiation to caps with capsfeatures when
it was last configured to a caps that it has to deinterlace.

tests: deinterlace: fix small typo in comment

check: Dist splitvideo0[012].ogg test files.

scaletempo: Add support for F64

docs: Minor fixes in various places

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

goom: remove compiler trick

After commit 2cb6cfed22166b262ae50cb58f3ff11dd8ba91f9 there is no need to
trick the compiler anymore about the usage of variable cpuFlavour.

Automatic update of common submodule

From b99800a to b319909

audiofx: remove unused variable

Remove unsued variable have_coeffs in audiofxbaseiirfilter

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

Use new GST_ENABLE_EXTRA_CHECKS #define

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

Automatic update of common submodule

From 9aed1d7 to b99800a