rtpmanager: re-enable -Werror

rtpjitterbuffer: fix compiler warning

gstrtpjitterbuffer.c: In function 'gst_rtp_jitter_buffer_loop':
gstrtpjitterbuffer.c:2978:3: error: 'result' may be used uninitialized in this function
   while (result == GST_FLOW_OK);
   ^

Automatic update of common submodule

From 1a07da9 to fe1672e

rtpjitterbuffer: Fix uninitialized variable compiler warning

rtpjitterbuffer: Remove raw comparisons of RTP sequence numbers

Several conditional statements perform comparison on RTP sequence
numbers without taking the sequence number rollover into account.
Instead, use the gst_rtp_buffer_compare_seqnum function to perform the
comparison.

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

tests: Don't build disabled plugins' check tests

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

docs: install prebuilt plugin docs if gtk-doc is disabled

Sync to the Makefile.am from gst-plugin-base where it is done right.
Fixes #725034

v4l2object: do not emit "parsed" caps for vp8

VP8 doesn't require parsing (vp8parse doesn't exist, so negotiation with demux fails
if "parsed" is set in caps).

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

v4l2: Don't require parser for VP8

Until GStreamer has one (see bug722760), we should not require a parser for VP8.

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

v4l2: CAPTURE_MPLANE is well tested now

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

v4l2videodec: Create one element per device

For each videoCdevice probe it input/output capabilities
if it match with video decoder requirement register a new element.

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
https://bugzilla.gnome.org/show_bug.cgi?id=722128

v4l2videodec: Calculate latency from device information

Decoders or other devices that expose a minimum buffers required produce
an first output. We use this information to calculate latency.

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

v4l2videodec: Implement v4l2videodec

Implement an element that can driver V4L2 M2M decoder device.

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

rtph264pay: only update last_spspps time if all sps/pps got sent successfully

This fixes an issue with gst-rtsp-server where no sps and pps are
sent for the first intra frame, because the payloader starts working
already when receiving DESCRIBE but there is no transports so it tries
to send sps and pps, but that fails with a FLUSHING flow. But the time
for last sent sps and pps would still be set, so when PLAY arrives and
the first intra frame is to be sent there is no sps and pps sent due to
that time since last sps pps is less than spspps_interval.

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

rtspsrc: Fix deadlock when task creation is no successful

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

autodetect: demote candidate error to warning and plug fake{sink,src}

In the case where we have no suitable candidate we post a warning and plug a
fake-element. Do the same when non of the candidate work.

This is more consistent and plugin the fakesink as a fallback is probably
helpful for running unit tests without requiring hardware src/sink elements.

Fixes #722981

v4l2: make some more controls configurable

... at least if one tries hard enough using extra-controls property.

v4l2: Require mplanar support for now in configure

The code fails to compile without currently, see
https://bugzilla.gnome.org/show_bug.cgi?id=723446

It's better to disable it instead of failing compilation
until this is fixed properly.

jack: add some simple log handlers for jack

Add log handlers for jack that write to the gst debug log. This avoids spamming
the console when e.g. using autoaudiosink, having the jack elements installed,
but not running jack.

v4l2src: handle old and odd driver behaviour when listing controls

audiofxbaseiirfilter: check if coefficients are provided inside filter lock

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

v4l2src: also unset INTERLACED flag on buffers if frame is not interlaced

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

v4l2src: Flag interlaced buffers as interlaced.

We correctly indicate the field ordering on interlaced buffers, but fail to
flag them as containing interlaced video, which we need to do here because
we signal interlace-mode=mixed in our caps. This means that downstream
elements (like vaapipostproc from gstreamer-vaapi) don't recognise these
buffers as in need of deinterlacing.

Fix this by setting the interlaced flag on all interlaced buffers.

Signed-off-by: Simon Farnsworth <simon.farnsworth@onelan.co.uk>
https://bugzilla.gnome.org/show_bug.cgi?id=724899

aacparse: be more strict at ADTS header parsing

Adds two extra checks:

- Sampling frequency on header can't be 15.
- Frame size should be at least 9 or 7, depending
  on whether CRC protection is present.

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

aacparse: make sure we have enough ADTS data

We need at least 6 bytes to pass over to _get_frame_len()
but we were just checking for a minimum of 2 bytes for the
syncword.

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

autodetect: check if the kid has a sync property

previously autovideosrc did not have a sync property and v4l2src has none either.

autodetect: use a common baseclass

This makes the actual elements super simple. We're using the ELEMENT_FLAG to
configure source/sink and a string for the Audio/Video type.

rtspsrc: add tls-database property

Add support for a new property: tls-database. If the property is set,
the certificate database will be given to the rtsp connection if TLS
protocol is being used. If the server certificate can't be verified with
the default database, this additional database will be used.

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

osxaudio: remove unused variables

autodetect: extract common helper code

The function to generate the pretty names is basically the same. Use one and add
a parameter.

autodetect: improve the tests

Add fake audio/video sinks. Previously running the test might be flaky due to
the use of real elements (hardware in use), which we don't want to test here.
Add two more tests that check that the fakes are chosen.

souphttpsrc: do not emit error when connection with unknown size ends

Commit 46fd12ae5ec53200b16dfd7f17048d6bc60fbfbc introduced connection
recovery. But when server does not specify content-size,
souphttpsrc tries to reconnect even after regular end of stream.
Http server replies  with SOUP_STATUS_REQUESTED_RANGE_NOT_SATISFIABLE
but souphttpsrc still emits error instead of EOS.

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

Signed-off-by: Branislav Katreniak <bkatreniak@nuvotechnologies.com>

autodetect: fix the disabled test

Use a shared helper for both tests. It turns out that the valgrind variant is
fine (maybe due to picking up pulsesink though).

autodetect: remove cruft from the test

Remove the obsolete version check and use the ignore macro for the disabled test.

docs: use docbook markup for xi:include

It turns out that the change in gtk-doc-1.20 which wraps the |[]| content in
CDATA break xi:inlcude examples. As in a whole jhbuild checkout these where
the only 4, we're fixing them instead.

isomp4mux: fix copy and paste

This fixes doc warnings.

docs: use the gtk-doc syntax to link to properties

Don't use docbook unless needed. Also stip other docbook tags in the the files we fix.

pulsesink: fix crash when getting the current-device in NULL->READY

The "goto unlock" is wrong as in this code path we haven't take the lock yet.
Fixes #724619

soup: We need libsoup >= 2.40 for proper usage of the content decoder

Previous versions did not consider our chunk allocator and allocated
memory by themselves, which caused crashes and broken behaviour.

docs: fix mismatched para tags

newer gtkdoc is more sensitive to mismatched docbook tags.
This fixes the build in master.

rtpjitterbuffer: add support for serialized queries

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

tests: fix typecast to fix compilation

rtpsession: proxy caps and allocation on RTP pads

recv_rtp_sink: allow proxying of the allocation query.
send_rtp_sink: allow proxying of caps and allocation. This allows us to
query caps downstream as well as get an allocator from downstream.
send_rtp_src: allow proxy of caps, this makes the caps query do
upstream.

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

qtdemux: handle tags in mac encoding

Check the charset from (C)*** tags and set the charset
to convert from MAC encoding if suitable.

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

souphttpsrc: Use new automatic_eos API from basesrc

We want to notice ourselves that we're EOS. Otherwise we will
always cancel requests in the very end and confuse the server...
and also make it impossible to use persistent connections.

souphttpsrc: Consistently use have_size instead of content_size!=0

souphttpsrc: Free extra headers when finalizing the element

It's set as property by the application, we should not just reset
properties when going back to READY.

souphttpsrc: Properly close the session when going back to NULL

Don't wait for that until the element is disposed.

souphttpsrc: add support for keep-alive sessions

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

souphttpsrc: Add "compress" property to enable/disable automatic gzip/deflate content encoding handling

souphttpsrc: Retry connection if we're finished before the content size only if we actually have a content size

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

souputils: Fix compiler warning

gstsouputils.c:35:25: error: comparison of constant 9 with expression of type
      'SoupLoggerLogLevel' is always false
      [-Werror,-Wtautological-constant-out-of-range-compare]

souphttp*: add ability to do HTTP session logging

This changeset adds the loggin infrastructure and
mods both souphttpsrc and souphttclientsink to use it.

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

matroska: add support for GRAY8, BGR and RGB video colourspaces in V_UNCOMPRESSED codec

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

souphttpsrc: Add mapping for NOT_FOUND and NOT_AUTHORIZED errors

souphttpsrc: Don't duplicate status_code to GStreamer error mapping

goom: Remove unused functions

matroskaparse: Comment out some unused functions used only from the commented out pull-mode code

id3v2mux: Fix another compiler warning

souphttpsrc: Fix implicit enum conversion compiler warning

error: implicit conversion from enumeration type
'SoupStatus' to different enumeration type 'SoupKnownStatusCode'

interleave: Fix unitialized variable compiler warning in test

error: variable 'mask' is used uninitialized
whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]

id3v2mux: Fix unitialized variable compiler warning

error: variable 'image_type' is used uninitialized
whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]

oss4: Fix typo in header include guard

error: 'GST_OSS4_AUDIO_H' is used as a header guard here,
followed by #define of a different macro [-Werror,-Wheader-guard]

rtprtxsend: Fix unitialized variable compiler warning

variable 'rtx_ssrc' is used uninitialized whenever
'if' condition is false [-Werror,-Wsometimes-uninitialized]

rtpac3depay: Remove unused variable

flx: Fix typo in header include guard

error: '__GST_FLX_FMT__H__' is used as a header guard here,
followed by #define of a different macro [-Werror,-Wheader-guard]

qtmux: remove have_dts flag from pads

It was used in the past in 0.10 when there was no explicit DTS
field in buffers, now we have it in 1.x series and we can
check it directly with GST_BUFFER_DTS_IS_VALID

qtmux: improve support for sparse streams

Do not try to use subsequent buffer timestamps to calculate
sparse streams durations because the stream is sparse and
the buffers might not be 'time adjacent'. So rely on the
duration and give the option to the pad to provide
custom 'empty' buffers to represent the gaps in the
stream, this can vary on how the data is represented.

Right now, the only sparse stream supported is tx3g subtitles.

qtmux: add support for text/x-raw subtitles

Adds it to mp4mux, qtmux and gppmux.

Buffers need to be prefixed with 2 bytes for the text length before
being muxed.

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

qtmux: add support for the TX3G atoms

Adds functions for creating and setting values related to the
tx3g atom for raw text subtitle support.

QTFF spec has information on those atoms

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

qtmux: add subtitle support to qtmuxmap structures

adds basic stubs for subtitle support around the qtmux and
qtmuxmap structures. Still no real subtitle implemented, but
basic functions in place

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

matroska: factor out read context init/reset

While at this, move _track_reset() to track-ids
so it can be called from the common read context
reset routine.

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

effectv: fix doc section of revtv element

osxvideo: Fix libtool usage

--tag=CC is needed for static build

deinterlace: do not try set deinterlace method if passthrough is enabled

Fixes an issue with progressive content and unsupported video formats
for the deinterlace method.

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

flacenc: order format in template caps by preference

To minimise risk of bad fixation, though audioconvert
at least should be smart enough to avoid it.

v4l2: Remove obsolete definition GST_V4L2_MISSING_BUFDECL

The only use was removed by 9edc0c0365f79ab07ff2e65461c6696e3931a3f0

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

gst: Don't use endianness-specific S8 audio format

It does not exist.

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

cairooverlay: add support for RGB16

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

matroskamux: Fix constantly growing used uid list

Moves the used uid list to the class to avoid having it grow forever.

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

Automatic update of common submodule

From d48bed3 to 1a07da9

wavparse: Ignore Broadcast Wave Format (BWF) tags when searching for 'fmt' chunk

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

check: Use fakesink sync=True instead of an audio sink

Ensures the test can run on systems without alsa (or any audio output for
that matter), and will avoid people running build slaves wondering what
the hell was beeping during the night :)

ac3parse: custom get_sink_caps handling for private stream caps

... now that those are transformed rather than parsed, some transforming
of caps is required as well to make auto-plugging succeed.

v4l2: guard use of ENODATA with #ifdef

Not all systems with v4l have ENODATA defined, so check that we have it
before attempting to use it.

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

Revert "rtspsrc: Proxy rtpjitterbuffer do-retransmission property"

This reverts commit 9f7b1128b1f00a2b87a232ff890867549ab95ba5.

This should be handled automatically be rtspsrc if the AVPF profile
is used, and manual enabling of it can be done with the new-manager
signal.

rtspsrc: add signal to notify of new manager

So that you can configure and connect to signals on the rtpbin.

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

rtspsrc: Proxy rtpjitterbuffer do-retransmission property

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

rtpjitterbuffer: handle expected packet being an RTX packet

If the expected packet (do_next_seqnum is TRUE) is the one we requested
for retranmission earlier, do the logic to update the retransmission
statistics as well before setting up the timers for the next expected
packet.
Also reset the retransmission counter if the timer is reused for another
seqnum.

rtpbin: add a caps accumulator for the request-pt-map signal

Add an accumulator that stops the signal emission as soon as a caps has
been retrieved. Otherwise the default handler would continue emitting
the signal and possibly overwrite the result with NULL again.

rtxreceive: copy flags and timestamps from original buffer

rtpjitterbuffer: ignore invalid timestamps in rtt calculation

When the input buffer does not have a valid timestamp, don't try to
calculate the round-trip-time.

matroskaparse: better default caps when none set

Uses information gathered during EBML parsing to
forge a more suitable set of caps instead of blindly
assuming everything is video/x-matroska.

For consistency, stream type reset was added to
matroska-demux too.

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

tests: rtprtx::test_rtxreceive_data_reconstruction: remove useless code for triggering retransmission

There is no need anymore to push yet another buffer in rtxsend
in order to trigger the previously requested retransmissions
to actually happen.

tests: rtprtx::test_rtxreceive_data_reconstruction: fix race condition

Now with rtprtxsend pushing rtx buffers from a different thread,
this is necessary to ensure that the result of the test is deterministic.

This code makes use of GstCheck's global GMutex and GCond that are
being used inside GstCheck's sink pad chain() function in order
to synchronize with it.

tests: rtprtx::test_rtxsender_packet_retention: fix race condition

Now with rtprtxsend pushing rtx buffers from a different thread,
this is necessary to ensure that the result of the test is deterministic.

This code makes use of GstCheck's global GMutex and GCond that are
being used inside GstCheck's sink pad chain() function in order
to synchronize with it.

tests: rtprtx::test_push_forward_seq: fix race condition

Now with rtprtxsend pushing rtx buffers from a different thread,
this is necessary to ensure that the result of the test is deterministic.

This code makes use of GstCheck's global GMutex and GCond that are
being used inside GstCheck's sink pad chain() function in order
to synchronize with it.

tests: rtprtx::test_push_forward_seq: fix buffer refcounting

rtprtxsend: ensure that no rtx buffers are sent after EOS

To do that, enqueue the EOS event to be sent from the srcpad task
thread and flush the queue right afterwards, so that no more rtx
buffers can be sent, even if there are more requests coming in.

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

rtprtxsend: run a new GstTask on the src pad

The reason behind this is to minimize the retransmission delay.
Previously, when a NACK was received, rtprtxsend would put a
retransmission packet in a queue and it would send it from chain(),
i.e. only after a new buffer would arrive.

This unfortunately was causing big delays, in the order of 60-100 ms,
which can be critical for the receiver side.

By having a separate GstTask for pushing buffers out of rtxsend,
we can push buffers out right after receiving the event, without
waiting for chain() to get called.