rtspsrc: Use new bin suppressed flags API for managing the element flags

ext, gst: fix indentation

tests: fix indentation

rtpjitterbuffer: Fix calculating next_seqnum when dropping old buffers from a full queue.

Fixes calculating the next sequence number when a ITEM_TYPE_LOST with more than one
definitely lost packets is encountered.

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

rtpjitterbuffer: improved rtx-rtt averaging

The basic idea is this:
1. For *larger* rtx-rtt, weigh a new measurement as before
2. For *smaller* rtx-rtt, be a bit more conservative and weigh a bit less
3. For very large measurements, consider them "outliers"
   and count them a lot less

The idea being that reducing the rtx-rtt is much more harmful then
increasing it, since we don't want to be underestimating the rtt of the
network, and when using this number to estimate the latency you need for
you jitterbuffer, you would rather want it to be a bit larger then a bit
smaller, potentially losing rtx-packets. The "outlier-detector" is there
to prevent a single skewed measurement to affect the outcome too much.
On wireless networks, these are surprisingly common.

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

rtpjitterbuffer: Detect whether to assume equidistant spacing when loss

Assuming equidistant packet spacing when that's not true leads to more
loss than necessary in the case of reordering and jitter. Typically this
is true for video where one frame often consists of multiple packets
with the same rtp timestamp. In this case it's better to assume that the
missing packets have the same timestamp as the last received packet, so
that the scheduled lost timer does not time out too early causing the
packets to be considered lost even though they may arrive in time.

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

rtpjitterbuffer: Don't request rtx if 'now' is past retry period

There is no need to schedule another EXPECTED timer if we're already
past the retry period. Under normal operation this won't happen, but if
there are more timers than the jitterbuffer is able to process in
real-time, scheduling more timers will just make the situation worse.
Instead, consider this packet as lost and move on. This scenario can
occur with high loss rate, low rtt and high configured latency.

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

rtpjitterbuffer: Fix lost duration when gap after lost timer

This patch fixes an issue with the estimated gap duration when there is
a gap immediately after a lost timer has been processed. Previously
there was a discrepancy beteen the gap in seqnum and gap in dts which
would cause wrong calculated duration. The issue would only be seen with
retranmission enabled since when it's disabled lost timers are only
created when a packet is received and the actual gap length and last dts
is known.

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

rtpjitterbuffer: Expose rtx-deadline as a property

The default -1 gives the old behavior.

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

rtpjitterbuffer: Improved expected-timer handling when gap > 0

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

rtpjitterbuffer: Major improvements for RTX stats

Stats should also be collected for unsuccessful packets.

rtx-rtt is very important for determining the necessary configured
latency on the jitterbuffer. It's especially important to be able to
increase the latency when retransmitted packets arrive too late and are
considered lost. This patch includes these late packets in the
calculation of the various rtx stats, making them more correct and
useful.

Also in the case where the original packet arrives after a NACK is sent,
the received RTX packet should update the stats since it provides useful
information about RTT.

The RTT is only updated if and only if all requested retranmissions are
received. That way the RTT is guaranteed to make sense. If not we don't
know which request the packet is a response to and the RTT may be bogus.
A consequence of this patch is that RTT is not updated for a request
when one of the RTX packets for that seqnum is lost, but that since
measured RTT will be more accurate.

The implementation store the RTX information from the timed out timers
and use this when the retransmitted packet arrives. For performance
these timers are stored separately from the "normal" timers in order to
not impact performance (see attached performance test).

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

rtpjitterbuffer: Add and expose more stats and increase testing of it

Add num-pushed and num-lost.
Expose num-late, num-duplicates and avg-jitter.

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

rtxreceive: Set buffer flag for retransmitted packets

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

rtpjitterbuffer: Option to disable rtx-delay-reorder

When disabled we can save some iterations over timers.

There is probably an argument for rtx-delay-reorder to exist, but
for normal operations, handling jitter (reordering) is something a
jitterbuffer should do, and this variable feels like functionality that
is not "in-sync" with what the jitterbuffer is trying to achieve.

Example: You have 50ms jitter on your network, and are receiving
audio packets with 10ms durations. An audio packet should not be
considered late until its rtx-timeout has expired (and hence a rtx-event
is sent), but with rtx-delay-reorder, events will be sent pretty much
all the time due to the jitter on the network.

Point being: The jitterbuffer should adapt its size to the measured network
jitter, and then rtx-delay-reorder needs to adapt as well, or simply
get out of the way and let the other (better) rtx-mechanisms do their job.

Also change find_timer to only use seqnum as an argument, since there
will only ever be one timer per seqnum at any given time. In the
one case where the type matters, the caller simply checks the type.

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

rtph263pay: Fix double free from coverity

CID #1372887

rtph263pay: Indent as per gst-indent

configure: Depend on gstreamer 1.9.2.1

autodetect: Use gst_bin_set_suppressed_flags() API

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

jack: Fix pipeline hang when jack changes sample rate or buffer size

If jackd changes the buffer size or sample rate, jackaudiosink hangs
and can't be stopped. This also happens if jack is configured as slave
and a gstreamer pipeline is started on the slave machine while the jack
master isn't running yet. If the the jack master is started it changes
the buffer size / sample rate and jackaudiosink can't be stopped.

This fix calls jack_shutdown_cb when jack_sample_rate_cb or
jack_buffer_size_cb is called.

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

deinterlace: Fix field ordering for reverse playback

And actually calculate the field duration instead of a frame duration so
that we can properly timestamp output frames in fields=all mode.

This is probably still broken for reverse playback in telecine mode.

udpsrc: Fix compilation on NetBSD

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

Automatic update of common submodule

From b18d820 to f980fd9

qtdemux: offset is irrelevant when no crypto info

Cause later it will try to use the crypto info array to get an index and
attach on of the positions as buffer's crypto info.

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

Automatic update of common submodule

From f49c55e to b18d820

osxaudio: Distribute device provider files

Those where missing the the dev release tarballs for 1.9.2 which
prevented building from tarball on OSX platform

qtdemux: Fix crash with no cenc aux offset

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

aacparse: parse a bit more of the humongous LOAS data

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

aacparse: make it clear when a potential LOAS frame is not one

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

aacparse: add a few comments to anchor parsing to the spec

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

aacparse: improve channel/rate handling

Keep track of the last parsed channels/rate fields so they can be
used even if the element was not yet configured.

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

aacparse: fix varlength number reading as per spec

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

aacparse: strip uneeded static arrays slack

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

rtpmp4adepay: Only declare a stream to be framed once a marker bit has been seen

This may cause a few packets to be processed by the parser, but it's
better than never pushing out buffers from a slightly broken stream
where no marker bits are set.

dvdemux: Fix timestamping in reverse playback mode

This is only supported right now if after a demuxer that supports reverse
playback, e.g. with DV container inside AVI container.

meson: Bump version to 1.9.2

qtmux: Implement the preset interface.

+ And provide a "youtube" preset, which based on
https://support.google.com/youtube/answer/1722171 sets
faststart to True.

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

Back to development

Release 1.9.2

po: Update translations

tests/examples: #define GDK_DISABLE_DEPRECATION_WARNINGS

We use gdk_cairo_create() which is deprecated since 3.22.

osxvideo: Remove QuickTime references.

QuickTime.h is no longer available on OS X 10.12 (Sierra),
and both the header and the framework seem unnecessary
for compilation - at least as of 10.11 (El Capitan).

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

Use the new API to post flow ERROR messages on the bus

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

gitignore: ignore qtdemux, rtph261 and rtpvp9 tests

tests: use GST_NET_LIBS instead of hardcoded -lgstnet

Fixes build in OSX when running 'make check' in gst-uninstalled.

tests: remove a wrong 'const' specifier

Fixes "error: duplicate 'const' declaration specifier"

build: silence error about pthread for 'make check' in osx

Fixes "clang: error: argument unused during compilation: '-pthread'"

vp9enc: Fix build of unit test by letting it link to libgstvideo

Revert "rtpmux: fix PROP_TIMESTAMP_OFFSET range problems"

This broke API, so we need a better solution!

This reverts commit c7579d31a6e9d788e94b83258309063d0aae481e.

rtpvp9depay: Support flexible mode

vp9enc: Fix leak of vpx_image_t

rtph263pdepay: Don't try to push empty frame

If the result of depayloading is an empty frame, just drop it. This is
likely the result of a buggy payloader.

rtpmux: fix PROP_TIMESTAMP_OFFSET range problems

It could not set the offset for the full guint32 range.

rtpbin: introduce max-streams property

To be able to cap the number of allowed streams for one session.

This is useful for preventing DoS attacks, where a sender can change
SSRC for every buffer, effectively bringing rtpbin to a halt.

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

rtpsource: reordered packets are very normal, and should not be a warning

rtpsession: degrade g_warning to GST_ERROR

So we don't blow up while investigating

rtph263pdepay: Fix picture header for non-writable payload

Under certain conditions gst_rtp_buffer_get_payload() returns a copy of
the payload. In this case the payload modifications will not affect the
rtp buffer. So instead of modifying the payload buffer directly we
should modify the buffer that actually gets pushed on the adapter.

rtph261depay: Fix check of valid payload length

Packets with no H.261 payload should be dropped to avoid invalid
write/reads.

rtph263pay: Fix double free, invalid reads and leak

rtpsession: sanity check RTT before ignoring PLI/FIR

rtpsession: handle sdes messages with non-utf8 more gracefully

rtph263pay: change log level on bitstream parsing messages

tests/rtprtx: refactor the tests to use gstharness

The functionality of all the tests was kept exactly the same. Some tests
were renamed:
test_push_forward_seq -> test_rtxsend_rtxreceive
test_drop_one_sender -> test_rtxsend_rtxreceive_with_packet_loss
test_drop_multiple_sender -> test_multi_rtxsend_rtxreceive_with_packet_loss

test_rtxreceive_data_reconstruction was testing that retransmitted
buffer produced by rtxsend was correctly transformed to the original
buffer by rtxreceive. Now we are checking for this in all the tests
where both rtxsend & rtxreceive are involved. That's why the test was
removed.

rtph265pay: Set RTP marker bit

Set the RTP marker bit on the last RTP packet of an H.265 access unit.

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

videoflip: added GstVideoDirection interface

It implements now this interface with its video-direction
property. Values are changed to GstVideoOrientationMethod but they have
the same value than the originals.

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

gstrtpsession: refactor duplicate code into a function

Less code, easier to read, more consistent.

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

rtpbin: fix typo in max-misorder-time property name

splitmuxsink: fix printf format compiler warning in debug message

On 32-bit x86: gstsplitmuxsink.c:966:31: warning: format ‘%u’ expects
argument of type ‘unsigned int’, but argument 9 has type
‘guint64 {aka long long unsigned int}’

Add support for Meson as alternative/parallel build system

https://github.com/mesonbuild/meson

With contributions from:

Tim-Philipp Müller <tim@centricular.com>
Jussi Pakkanen <jpakkane@gmail.com> (original port)

Highlights of the features provided are:
* Faster builds on Linux (~40-50% faster)
* The ability to build with MSVC on Windows
* Generate Visual Studio project files
* Generate XCode project files
* Much faster builds on Windows (on-par with Linux)
* Seriously fast configure and building on embedded

... and many more. For more details see:

http://blog.nirbheek.in/2016/05/gstreamer-and-meson-new-hope.html
http://blog.nirbheek.in/2016/07/building-and-developing-gstreamer-using.html

Building with Meson should work on both Linux and Windows, but may
need a few more tweaks on other operating systems.

Fixed splitmuxsink 32-bit overflow bug

Extend the byte tracking counters to 64-bit on
all platforms, instead of using gsize, which overflows
after 4GB.

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

isomp4: Fix coverity warning

If atom_copy_data fails to write anything, return 0

CID #1371458

v4l2: consistently check #ifdef HAVE_GUDEV instead of #if

Both work with autotools but they definitely don't mean the same thing, cause
problems with other build systems, and are bad form. Existence should always be
checked with #ifdef or #if defined.

directsound: port away from old DirectX API

D3DX has been deprecated for the last 4 years and latest versions of
Windows no longer ship headers for it. This is fine as long as you're
building with Cerbero's Wine-based DirectX headers, but sucks if you
want to build against the actual Windows SDK.

We were just using it to get error strings anyway, so just use the
generic error string API.

Revert "flacparse: Add maximum bitrate tag"

This reverts commit c703ab69f526092bb26cce41ca691a896c8383d8.

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

rtpjitterbuffer: Fix unit test by disabling adaptive misorder/dropout calculations

Need to set max-misorder-time and max-dropout-time to 0 so the
jitterbuffer does not base them on packet rate calculations.
If it does, out gap is big enough to be considered a new stream and
we wait for a few consecutive packets just to be sure

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

splitmuxsink: Add option to split at exactly max-size-time

Will try to request a keyframe from the encoder to be sent at the target
running time.

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

splitmuxsink: Allow time and bytes to reach their respective thresholds

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

rtspsrc: Allow mimetypes with properties as long as they're application/sdp

Some servers add properties like charset, e.g.
  application/sdp; charset=utf8

Ideally we should also parse the charset and do conversion of all messages,
but that's for a later time.

qtmux: Added support for writing timecode track

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

multiudpsink: Initialize bytes_sent field.

This fixes endpoints not receiving any data intermittently.

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

rtpjitterbuffer: Actually calculate the packet rate for max-dropout and max-misorder calculations.

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

rtpjitterbuffer: Don't warn for duplicate packets

This is a normal scenario and should not be a warning.  This can
happen frequently when re-transmits of lost packets are enabled.

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

splitmux: Fix typo converting to running time.

Use the correct collected timestamp.

Revert "splitmuxsink: Use GstBin async-handling instead of our own."

This reverts commit fa008f271a52f82dededc28bd81b020ca7939b47.

async-handling in GstBin causes the pipeline to spin at 100%
CPU as the top-level pipeline tries to change that state
to PLAYING constantly. This is a workaround for a core
problem, essentially, but an improvement in this case for now.

splitmux: Recheck state after unlocking mutex.

After dropping the splitmux lock, re-check the state,
don't just fall through and sleep unconditionally,
as we may have already missed the wakeup.

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

splitmuxsrc: Don't stop and error on EOS flow return

Don't immediately halt on EOS flow return from downstream
due to out of segment. Let the demuxer handle it and send
EOS.

rtpjitterbuffer: avoid unref of null buffer

The current 'l' pointer will be NULL when the loop
is interrupted with a 'break' statement. Need to have
it advance to the next list item before interrupting.

wavparse: Add tags for container format and bitrate for uncompressed PCM

The PCM bitrate is added to help downstream elements (like uridecodebin)
figure out a proper network buffer size

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

flacparse: Add maximum bitrate tag

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

qtdemux: When receiving a DISCONT buffer that does not point to a sample, remember the offset

And don't just reset everything. This makes sure that we can continue to
handle data in the following scenario:

moov: discont
moof: discont
mdat: continuous

Previously this would fail because the offset would be the accumulated offset
from moov and moof at the mdat position, while the buffer offset might be
something completely different.

rtp: Filter with the filter caps in the payloader's getcaps

souphttpsrc: include http-status-code in error message details

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

splitmuxsink: Fix debug statement signedness.

The ts variable is a GstClockTime, don't print it
as a GstClockTimeDiff.

splitmuxsink: Handle negative running time

Use signed clock times for running time everywhere
so that we handle negative running times without
going haywire, similar to what queue and multiqueue
do these days.

splitmuxsink: Drop lock when sending dummy event

When pushing the dummy event into the multiqueue,
drop the splitmux lock or else we might deadlock.

rtph264pay: Intersect with filter caps in getcaps function.

Always intersect with the filter caps in the getcaps function
to make sure we return a subset of what was requested.

Other payloaders also have this problem and need fixing
in future commits.

tests: qtdemux: fix element and pad leak

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

tests: fix bus leaks

gst_bus_add_signal_watch() takes a ref on the bus which should be
released using gst_bus_remove_signal_watch().

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

configure: Call AG_GST_PKG_CONFIG_PATH to set GST_PKG_CONFIG_PATH

GST_PKG_CONFIG_PATH is used in docs/plugins directory, so
AG_GST_PKG_CONFIG_PATH must be called to set it.

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

souphttpsrc: Don't drop final bytes of a range request

At the end of a range request, we don't want to return GST_FLOW_EOS otherwise
the last bytes we just read will be dropped by basesrc.

Instead just return GST_FLOW_OK (which was set just before) and let basesrc
handle the fact we are at the end of the segment.

v4l2provider: Fix device type detection

The type detection would lead to assertion as it would try
to create a device without having found any type for it. It
also didn't detect MPLANE devices properly.