v4l2object: Always set the colorimetry in S_FMT

So far we were only setting colorimetry for OUTPUT devices (v4l2sink or
m2m sink pad). This prevented selecting through caps negotiation the
colorimetry for CAPTURE devices (v4l2src or m2m src pad). This is rarely
selectable, but trying is harmless.

monoscope: Only fixate pixel-aspect-ratio if the field exists

splitmuxsink: Don't send fragment-opened-closed message if the reference ctx is NULL

It can happen during teardown that the reference context becomes NULL.
In that case, trying to send the fragment-opened-closed message would
lead to a crash.

splitmuxsink: Run gst_iterator_foreach() as long as it returns GST_ITERATOR_RESYNC

CID 1434160

qtdemux: Fix comparision for extra caption atom

We want to make sure we have *enough* data for the potential 2nd
caption atom.

CID #1434161

qtdemux: Handle bogus caption samples

Corrupted files could potentially have multiple cdat/cdt2 atoms in
a sample entry, which is unclear how to handle.

Ignore repeated ones.

CID #1434162
CID #1434159

monoscope: Fixate pixel-aspect-ratio too and make sure the final caps are completely fixated

Otherwise e.g. this fails with assertions:
gst-launch-1.0 audiotestsrc ! audioconvert ! monoscope ! videoconvert ! \
    videoscale ! video/x-raw,width=800,height=600 ! ximagesink

qtmux: Add comments and doc about prefill mode

qtmux: Refactor pad re-negotiation code

It was similar for all pads

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

qtdemux: Detect and expose CEA 608/708 Closed Caption tracks

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

rtxsend: fix wrong memory layout assumption

The code responsible for creating retransmitted buffers
assumed the stored buffer had been created with
rtp_buffer_new_allocate when copying the extension data,
which isn't necessarily the case, for example when
the rtp buffers come from a udpsrc.

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

rtpbin: new signal "get-storage"

Similar to the get-session and get-internal-session signals,
we expose a get-storage signal in addition to the
get-internal-storage signal to give access to the actual
element for applications that need to set properties on the
element, in particular "size-time"

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

splitmuxsink: Add new reset-muxer property

With this the muxer is not set to NULL after each segment but instead
only flush events are sent to it to reset the EOS state.

As a result, the muxer will keep stream state and e.g. mpegtsmux will
keep the packet continuity counter continuous between segments as needed
by hlssink2.

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

tests: remove broken and now pointless v4l2src-test

This tests APIs that don't exist any longer and also doesn't
work at all, and was last touched in a meaningful way in 2006.

v4l2: Fix unknown type name ‘off_t’ error

Fix following build error

gstv4l2object.h:197:17: error: unknown type name ‘off_t’
       gint fd,  off_t offset);
                 ^

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

rtspsrc: reject segment seeks

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

qtdemux: Handle variant of vorbis in mp4

Comes from gpac apparently. The codec_data uses the same packing
mechanism as matroska.

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

qtdemux: Check sample count is valid in PIFF parsing

The value stored in cenc_aux_sample_count wasn't in sync with the
parsing code that followed which checks whether all entries are
valid and present.

Only write the actual sample count when we know for sure.

CID #1427087

qt: Get EGL native display from QPA if platform header is available

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

udpsrc: switch to using a buffer pool

This exposes a new property, mtu, which is used to determine the
initial size of buffers from the buffer pool. If received data
exceeds this, the element gracefully handles that in a manner similar
to what we had previously: a large memory gets filled and reallocated
at the next call to "fill".

The default size is set to 1500, which should cover most use cases.

With contributions from Mathieu Duponchelle <mathieu@centricular.com>

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

udpsrc: optimize GstUdpSrc object for cache performance

Optimize GstUdpSrc for cache performance.

Move the hot properties, which are used by the read function, to the top:
@used_socket, @addr, @cancellable, @skip_first_bytes, @timeout,
@retrieve_sender_address.

Remove the unused property @ttl.

Where needed reorder so that holes are avoided (the 64-bit @timeout)

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

qtdemux: Fix seeking on streams with frame reordering

The samples table is sorted by DTS, not PTS. As such we can only get the
correct result when using a binary search on it, if we search for the
DTS.
Also if we only ever search for the frame, where the following frame is
the first one with a PTS after the search position, we will generally
stop searching too early if frames are reordered.

In forwards playback this is not really a problem (after the decoder
reordered the frames, clipping is happening), in reverse playback
it means that we can output one or more frames too few as we stop too
early and the decoder would never receive it.

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

rtp: Fix compilation with non-C99 compilers

By moving variable declarations out of loop headers.

Back to development

Release 1.14.0

Update docs

rtpulpfecdec: fix build with older gcc

As on Ubuntu Trusty.

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

splitmuxsink: Allow splitting at exactly the time/bytes threshold

76e458a119926424e9dd5acf3210a592a314d713 changed the conditions from
"queued > threshold" to "queued >= threshold", which broke hlssink2 and
resulting in too small fragments being created although keyframes would
be at *exactly* the configured threshold.

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

rtpulpfec: fix unconditional use of __attribute__ ((packed))

Fix compilation with MSVC. We still assume that attribute
is supported by all other relevant compilers, which seems
to be the case since we haven't had any complaints about
similar code in rtpsbcpay.

rtpulpfec: don't use non-portable notation for 64-bit int constants

Use GLib macro instead, even if it's a bit unwieldy.

rtpulpfecdec: don't use __builtin_ctzll unconditionally

Fixes build with MSVC, and possibly other compilers too.

Release 1.13.91

Update docs

docs: rtpbin: add some Since markers for new properties

meson: Add deviceprovider changes to directsoundsink

These were missed when they were added to Makefile.am

configure.ac: enable largefile support if possible

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

v4l2: Fix support for 32bit mmap

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

Release 1.13.90

Update docs

flvmux: Duration & unit tests

The muxed buffers will not carry the duration of the
incoming buffers.

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

flvmux: Set PTS based on running time

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

rtspsrc: Ignore sendonly/recvonly attributes unless a backchannel is configured

This works around a bug in various ONVIF cameras that implement the
attributes the wrong way around. They still won't work with a
backchannel but at least normal playback will work for the time being.
It restores pre-1.14 behaviour where we would fail to preroll on any SDP
that lists a recvonly stream. For 1.16 a better solution should be
found.

The problem here is that the ONVIF spec has the meaning of the two
attributes the wrong way around in the examples, compared to RFC4566.

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

meson: enable more warnings

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

Port to latest GstAggregator segment API

The aggregator segment is now exposed on the src pad

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

directsoundsink: Downgrade rank to match directsoundsrc in -bad

As stated in commit c2956036b8da4b8f22a63a4f5a254be03e870aa6 in -bad,
the wasapi elements are now better than directsound, and should be
preferred if they are available.

For a later release, once the elements have more testing, we can
consider moving them to -good.

matroskamux: Only mark new clusters as keyframe if they start on a keyframe or we're muxing only audio

Based on a patch by Nicola Murino <nicola.murino@gmail.com>

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

matroskamux: Clip maximum cluster duration to the maximum possible value

Only up to timescale * G_MAXINT16 is possible as cluster duration, which
is already higher than our default value. Using higher values would
cause overflows and broken files.

Based on the investigation by Nicola Murino <nicola.murino@gmail.com>

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

matroska-mux: Refuse caps changes after starting to write headers

Matroska does not support changing the stream type and stream properties
after the headers were started to be written, and for example H264
codec_data changes can't be supported.

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

tests: fix redenc tests

The default of the allow-no-red-blocks property was changed in a
previous commit, thus breaking the test assumptions

rtp: fix another debug log printf format warning on 32-bit systems

rtpulpfeccommon.c:432:27: error: format ‘%lx’ expects argument of type
‘long unsigned int’, but argument 10 has type ‘guint64 {aka long long unsigned int}’

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

rtpptdemux: provide example usage for ignored-payload-types

rtpbin, rtpptdemux: Add missing Since markers

FEC elements: document, remove irrelevant properties

The ulpfecenc "mux-seq" and "ssrc" properties were initially added
because the element did more than implement ULPFEC. As it was
decided that FLEXFEC would be implemented in a separate element,
both properties are now unneeded and confusing.

Change the default for the ulpfecenc multi-packet property,
as it is expected that most users of this element will be protecting video
streams.

Change the default property for the rtpredenc allow-no-red-blocks
property, as it should also be its default mode of operation.

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

rtpgstdepay: do not warn when caps were not yet received

It is expected that when connecting to a stream that has
already started, the caps will only arrive at the interval
specified on rtpgstpay, we shouldn't be warning as this is
a normal mode of operation.

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

rtpulpfec: fix debug log printf format warning on 32-bit platforms

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

docs: hook up new RTP FEC elements

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

docs: update for git master

.gitignore more test binaries

tests: also dist new fec test header file

rtp: dist new header files

Fixes make distcheck

rtp: fec: fix build with gstreamer debug log system disabled

rtpptdemux: do no assume sink caps are non NULL

check: Fix ulpfec test build

The test name was updated but not the build definition

rtpbin: Expose FEC support signals

Also slightly refactor complete_session_src

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

rtp: Implement ULPFEC (RFC 5109)

We expose a set of new elements:

* ULPFEC encoder / decoder
* A storage element, which should be placed before jitterbuffers,
  and is used to store packets in order to attempt reconstruction
  after the jitterbuffer has sent PacketLost events
* RED encoder / decoder (RFC 2198), these are necessary to
  use FEC in webrtc, as browsers will propose and expect ulpfec
  packets to be wrapped in red packets

With contributions from:

Mathieu Duponchelle <mathieu@centricular.com>
Sebastian Dröge <sebastian@centricular.com>

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

rtpptdemux: Add ignored-payload-types property

Packets with these payload types will be dropped. A use case
for this is FEC, where we want FEC packets to go through the
jitterbuffer, but not be output by rtpbin.

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

rtpptdemux: Add ssrc to output caps

It may be useful downstream

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

gtk: fix compiler warning with recent glib

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

qt: don't use libEGL functions when we don't link to libEGL

Use the provided wrapper available from libgstgl.

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

monoscope: Forward the SEGMENT event from the chain function

Otherwise we'll break the event order and forward the SEGMENT event
before sending a CAPS event.

rtspsrc: Fix missing read property of backchannel

Add missing read property code for backchannel

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

examples: rtsp: fix meson build take 2

rtsp: Fix meson.build of the example

rtspsrc: Implement ONVIF backchannel support via TCP

rtspsrc: Implement ONVIF backchannel support

Set backchannel=onvif to enable, and use the 'push-backchannel-sample'
action signal with the correct stream id.

splitmuxsrc: Improve not-linked handling.

Don't report not-linked unless all pads have
returned not-linked.

Back to development

Release 1.13.1

docs: update plugin docs

po: update translations

rtpjitterbuffer: allow timestamps to move backwards

The original solution for #784002 incorrectly assumed that timestamps
may not move backwards and changed timestamps that did so.

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

docs: remove pointless Since: 0.10.x markers

rtpvorbisdepay: fix unbounded memory usage

All received configurations are parsed and added to a list, this lead
to an unbounded memory usage. As the configuration is resent every
second this quickly lead to a large memory usage.

Add a check to only add the config if it is not already available in
the list. This fix only handle the typical case of a well behaved
stream, a malicious server could still send many useless
configurations to raise the client memory usage.

docs: add qt plugin

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

qt: hook up to build

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

Move qt plugin from -bad

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

configure: fix build with --disable-external

docs: add moved gtk plugin to docs

docs: update for git master

gtk: hook up to meson build

gtk: hook up to autotools build

Move gtk plugin from -bad

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

qtdemux: Fix comment typo in previous commit

qtdemux: More 'meta' atom parsing fixes

Turns out everybody is doing it their own way, so peek into the
meta atom itself to figure out which spec it is following

qtmux: Add support for muxing svmi atom for stereoscopic video information

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

qtdemux: Minor cleanup

Just move variables to the blocks where they are used.

That function is massive, could do with some splitting up for
readability :(

qtdemux: Cope with difference between QTFF and ISO BMFF specs

The 'meta' atom is defined differently in QTFF and BMFF, so try
to guess which spec the current stream applies to by looking
at the major file type.

isomp4: Make 'hdlr' atom dump more flexible

The smallest possible is 24 (and not 25) bytes.

The last "name" field can according to QTFF specifications not be present
at all. The parser will handle this fine and so will the rest of
the qtdemux code.

Update ORC files

meson: make version numbers ints and fix int/string comparison

WARNING: Trying to compare values of different types (str, int).
The result of this is undefined and will become a hard error
in a future Meson release.