rtpjitterbuffer: schedule lost event differently

Schedule the lost event by placing it inside the jitterbuffer with the seqnum
that was lost so that the pushing thread can interleave and push it properly.

tests: remove timeouts from check

Timeouts make the test unreliable and are not needed.

rtpjitterbuffer: remove list debug

rtpjitterbuffer: add type to the item

So that the upper layer can know what data is contained in the item.

rtpjitterbuffer: fix flush

Pass function to flush to properly free the queue items.

rtpjitterbuffer: append seqnum -1 packets

rtpjitterbuffer: use structure to hold packet information

Make the jitterbuffer operate on a structure containing all the packet
information. This avoids mapping the buffer multiple times just to get the RTP
information. It will also make it possible to store other miniobjects such as
events later.

rtpjitterbuffer: update expected timer when possible

When we receive a packet and we have some missing packets, we can update their
estimated arrival times based on the timestamp difference.

rtpjitterbuffer: fix order of timeout events

Improve the order of the timeout events, if there are timers with the same
timeout, we want to trigger the lowest seqnum first. For this we need to loop
over the complete array of timers to find the best one before triggering the
timeout.

rtpjitterbuffer: send lost event before signaling next buffer

First send the lost event, then update the next_seqnum counter and then
send the signal to the pushing thread that it can retry to push a buffer. This
avoids pushing out buffers before the lost event is pushed.

jitterbuffer: configure clock-rate on jitterbuffer

Add a get and setter to configure the clock-rate in the jitterbuffer instead of
passing it as an argument to the insert method.

tests: add test for packet delay and retransmission

rtpjitterbuffer: add option to reset retransmission timers

rtpjitterbuffer: stop the timer thread

The timeout code could release the lock so we need to check if we are allowed to
wait for the clock some more.

rtpjitterbuffer: unlock only once

tests: check both PTS and DTS

tests: add unit-test for multiple missing packets

Check if multiple missing packets generate retransmission events and that the
retranmission requests are canceled when the missing packet arrives.

rtpjitterbuffer: improve flush and shutdown

There is no need to unschedule the timer in flush-start, flush-stop will remove
the timers and unschedule.
Unschedule the current timer before attempting to join the timer thread.

tests: improve debug

rtpjitterbuffer: set correct expected time

When we already have a timer for a packet, skip it but don't forget to adjust
the dts to the expected dts of the next packet.

jitterbuffer: improve debug

alpha: use POFFSET instead of OFFSET

Use the more correct POFFSET macro to get the offset of a component in its
plane. The offset macro gives the offset of the component relative to the start
of the frame.

goom: Fix MMX assembly compilation with clang

clang does not want or need a clobber list for emms:
error: clobbers must be last on the x87 stack

Patch taken from the FreeBSD ports, provided by
Dan McGregor <dan.mcgregor@usask.ca>

Automatic update of common submodule

From b613661 to 6b03ba7

matroska-demux: Make sure that subtitle buffers are \0-terminated

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

qtmux: handle issues correctly when downstream is not seekable

The streamable property only make sense for fragmented formats.
For regular MP4, when downstream is not seekable we can't rewrite
the headers, so qtmux can only work with fast-start=TRUE, where
the headers are written finishing the file.
For fragmented MP4, when streamable is not seekable and the streamable
property is FALSE, we must enforce streamable=TRUE warning the user
about this change

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

qtmux: make "streamable" TRUE as default

The most common use case for fragmented MP4 (Dash and Smooth Streaming)
is producing streamable content (even for VOD). streamable=FALSE would only
be used to generate fragmented MP4 with and index of MOOF's that could
be reproduced without a playlist/manifest
https://bugzilla.gnome.org/show_bug.cgi?id=707242

qtmux: deprecate the streamable property for non-fragmented MP4

The streamable property only makes sense for fragmented MP4.
https://bugzilla.gnome.org/show_bug.cgi?id=707242

v4l2: Remove commented out line

Automatic update of common submodule

From 74a6857 to b613661

Automatic update of common submodule

From 098c0d7 to 74a6857

alpha: don't assume planar formats have just 1 block

Don't assume planar formats have just one memory block with the data but use the
macros to access the right memory block where a component can be found.

tests: add retransmission jitterbuffer test

Store both DTS and PTS on buffers.
Make a queue for srcpad events.
Activate pads after linking so that we don't get RECONFIGURE events.
Add test for retransmission.

rtpjitterbuffer: keep delay as a separate variable in timer

Keep a separate delay in the timer so that we still know the original timestamp
of the packet that this timer refers to. We can then place the correct
running-time in the Retransmission event.

rtpjitterbuffer: fix writability of properties

Back to development

Release 1.1.90

Update .po files

po: Update translations

v4l2bufferpool: dmabuf is not a singleton anymore

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

souphttpsrc: do not do http requests in READY

HEAD requests to discover if the server is seekable shouldn't be done in
READY as it might lock the main thread that is doing the state change.

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

rtpjitterbuffer: reevaluate the current timer after timeout

When we trigger the timeout logic of a timer, reevaluate it because it is
possible that it still has the lowest timeout.

rtpjitterbuffer: don't update time when unscheduled

Don't try to estimate the current time when we got unscheduled.

rtpjitterbuffer: init packet spacing on first buffer

Already init the packet spacing variables on the first buffer so that we can
calculate the spacing on the second buffer already.

tests: fix comments

rtpjitterbuffer: push the lost event from the timer thread

Instead of pushing the lost event from the chain function, schedule a timeout
that will push the lost event from the timer thread. This avoid blocking the
upstream thread while we push and sync the event.

rtpjitterbuffer: add another test

The test is modified slightly because the late lost packets are only
generated now when a large gap is received.

rtpjitterbuffer: round gap duration to multiple of duration

Make sure the gap duration in the lost event is a multiple of the packet
duration.
Enable another test.

rtpjitterbuffer: keep track of duration

Keep track of the estimated duration of missing packets and use it in the lost
event.
Enable another unit test

rtpjitterbuffer: handle large gaps with one lost event

When we have a large number of missing packets, generate one lost event for all
the packets that have no chance of being pushed out in time.
Fix and activate unit test for large gaps.

rtpjitterbuffer: refactor lost event sending

Also make sure we only increment the expected seqnum and last
output timestamp.

jitterbuffer: refactor timeout triggers

jitterbuffer: simplify the timeout code

Keep track of the current time in the timeout loop.
Loop over all timers and trigger all the expired ones, we can do this in the
same loop that selects the new best timer.

jitterbuffer: rearrange timer update code

Also update the timers when retransmission is disabled. We need to
do this because when we added LOST timers when we detected missing packets and
we need to remove those timers when the packet finally arrives.

videomixer: link to libm for maths stuff

Fixes undefined references to rint and pow on ubuntu
build bot.

jitterbuffer: release lock on shutdown

check: change for videomixer renamed orc file

qtmux: remove MAX_TOLERATED_LATENESS

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

examples: we don't need the queue anymore

jitterbuffer: use separate thread for timeouts

Use a separate thread for scheduling the timeouts instead of using the
downstream streaming thread that might block at any time.

qtmux: set first_ts to DTS for streams that have DTS

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

qtmux: make sure duration is a valid number for last buffer

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

qtmux: use segment.start or last buffer end time in case of missing DTS

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

Revert qtmux: Use buffer PTS if DTS is not set"

This reverts commit f72c3cf71fde622067f41f31a53978ba4c94469d.

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

videomixer: Update orc generated files

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

rtpsession: Demux RTCP buffers from the RTP stream

If there are RTCP buffers in the RTP stream, process them as
RTCP. This way, we want receive streams following RFC 5761

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

rtp: Remove bogus extra caps from L24 template.

The extra caps entry in the template was making it sometimes
get plugged for any dynamically allocated payload type.

rtpbin: use PacketInfo for the sender

Avoid mapping the packet multiple times when sending RTP.

rtpbin: store more in the PacketInfo

Store all info in the PacketInfo so that we can avoid mapping the packet
multiple times.

session: store more in the PacketInfo structure

rtpbin: RTPArrivalStats -> RTPPacketInfo

Rename a structure because we are also going to use this for the sender
bits.

source: small cleanups

qtdemux: only update stop position if seek requests it

Check for GST_SEEK_TYPE_NONE for stop poistion and only update
the stop time if it is requested. Otherwise just maintain whatever
was stored at the segment

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

rtp: Add missing headers tp fix make dist

In addition to a956a6ceb2deb87cc1361aee1d6626449f46dab2

flacparse: Make sure we have enough data to read image tags

Thanks to iputinei for reporting this on IRC.

jitterbuffer: handle segments with non-0 start

We keep the DTS and PTS in running-time inside the jitterbuffer. Make sure to
transform it back to a buffer timestamp before pushing out the buffer.

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

matroskademux: Fix off-by-one in validation of UTF-8

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

videomixer: Do not check if caps are empty when they are NULL

In the case the caps are actually NULL, we should just concider it the
same way as empty caps in that case.

videomixer: fix build if orc is not installed

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

matroskademux: Preserve seqnum when pushing seek upstream

After converting a seek from time to bytes, use the same seqnum
on the event that goes upstream

qtdemux: track streams that are EOS on push mode to finish earlier

When the segment has a defined stop position, qtdemux should check
when streams reach this position and mark those as EOS. When all
streams are EOS it will return GST_FLOW_EOS to upstream to allow
the pipeline to finish instead of continuously consume buffers
from upstream that are not useful for the segment.

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

qtdemux: preserve stop of segment when doing seeks in push mode

When handling seeks in push mode, qtdemux converts the seek to bytes
and pushes upstream. It needs to keep track of the seek and the
subsequent segment to be able to map them back to the requested
seek time and properly preserve the segment stop of the seek.

This is done by using the start offset in bytes of the seek,
that should be the same of the segment from upstream. And this
is also backwards compatible with what qtdemux already was using.

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

videomixer: Add colorspace conversion

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

videomixer: Don't send reconfigure event when formats or PAR are different

It is racy with multiple pads.

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

videomixer: Bundle private copies of videoconvert code

Ideally, this would be part of libgstvideo.
Prefixes videoconvert symbols with videomixer_.

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

v4l2: Use newly #defined metadata names.

rtspsrc: only wait if we flushed

Only wait for the STREAM_LOCK when we flushed something when sending
a command for PAUSED or PLAYING.

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

rtspsrc: return when a flush was issued

Make gst_rtspsrc_loop_send_cmd() return TRUE when the current
action has been flushed

rtp: add L24 pay and depayloader

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

v4l2bufferpool: Fix missing condition in previous commit

v4l2bufferpool: Also fix strides for other semi-planar video formats

v4l2bufferpool: Fix stride for NV12/NV21

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

matroskademux: fix leaking buffer and caps

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

udpsrc: fix build on win32

gstudpsrc.c:855:15: error: #if with no expression

avidemux: handle unseekable streams

Handle streams that we can't seek in and ignore them in the
seek logic.

avidemux: only check video compression for video streams

Or else we might deref a stream with a NULL strf.vids and segfault

qtdemux: Add support for the avc3 sample entry format of the AVC file format

Amendment 2 of ISO/IEC 14496-15 (AVC file format) is defining a new
structure for fragmented MP4 called "avc3". The principal difference
between AVC1 and AVC3 is the location of the codec initialisation
data (e.g. SPS, PPS). In AVC1 this data is placed in the initial
MOOV box (moov.trak.mdia.minf.stbl.stsd.avc1) but in AVC3 this data
goes in the first sample of every fragment (i.e. the first sample in
each mdat box).  The principal reason for avc3 is to make it easier
for client implementations, because it removes the requirement to
insert the SPS+PPS in to the decoder pipeline every time there is a
representation change.

This commit adds support for the "avc3" atom, which is almost identical
to the "avc1" atom, except it does not contain any SPS or PPS data.

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

videomixer: Don't set EOS to FALSE when the collectpad *is* EOS

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

flacparse: cleanup on error after state change

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

udpsrc: Bind to multicast addresses on non-Windows systems

On Windows it's not possible to bind to a multicast address
but the OS will make sure to filter out all packets that
arrive not for the multicast address the socket joined.

On Linux and others it is necessary to bind to a multicast
address to let the OS filter out all packets that are received
on the same port but for different addresses than the multicast
address

And deprecate the multicast-group property and replace it with the
address property.

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