mpg123: fix compiler warning and simplify checks in set_caps

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

mpg123: rework set_format code so mpg123audiodec works with decodebin/playbin

The old code was using gst_caps_normalize() and was generally overly
complex. Simplify by picking sample rate and number of channels from
upstream and the sample format from the allowed caps. If the format caps
is a list of strins, just pick the first one. And if the srcpad isn't
linked yet, use the default format (S16).

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

gl/cocoa: Disable hack for NSApp iteration with a special #define

The hack causes deadlocks and other interesting problems and it really
can only be fixed properly inside GLib. We will include a patch for
GLib in our builds for now that handles this, and hopefully at some
point GLib will also merge a proper solution.

A proper solution would first require to refactor the polling in
GMainContext to only provide a single fd, e.g. via epoll/kqueue
or a thread like the one added by our patch. Then this single
fd could be retrieved from the GMainContext and directly integrated
into a NSRunLoop.

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

inter: interaudiosrc - fixate layout for default caps

Makes interaudiosrc produce valid audio caps in the absence of any other
guidance by adding to the pad template and fixate function

aggregator: Remove pointless atomic

It is only modified from the streaming thread

aggregator: Fix query leak

audiovisualizer: remove check for below zero for unsigned value

CLAMP checks both if value is '< 0' and '> max'. Value will never be a negative
number since it is an unsigned integer. Removing that check and only checking if
it is bigger than max and setting it appropriately.

Also converting the previous instance of this into MIN() for consistency.

CID 1139793

h264parse: expose compatible profiles to downstream

Some video bitstreams report a too restrictive set of profiles. If a video
decoder was to strictly follow the indicated profile, it wouldn't support that
stream, whereas it could in theory and in practice. So we should relax the
profile restriction for allowing the decoder to get connected with parser.

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

aggregator: Print jitter from clock waiting in the debug logs

audiovisualizer: remove check if below zero for unsigned value

CLAMP checks both if y is '< 0' and '> h1'. y will never be a negative number
since it is an unsigned integer. Removing that check and only checking if it
bigger than h1 and setting it to that max approprietaly.

CID 1139792

hlsdemux: cache current file position in the list

Avoids iterating the list everytime to look for the next segment
to be used (or to advance to the next one).

hlsdemux: simplify next segment checking functions

Optimize loop by moving condition outside of it and reuse the
find_next_fragment function to check if there is next instead of
replicating the same loop

hlsdemux: cache total duration to avoid iterating at every query

Duration queries can be done a few times per second and would cause
the segment list to be traversed for every one. Caching the duration
prevents that.

decklinkvideosrc: Post LATENCY message when the mode changes

Different modes have different framerates, and thus different latencies. We
might need to reconfigure the latency of the pipeline.

decklinkvideosrc: Fix compiler warning

decklinkvideosrc: Add automatic mode detection

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

hlsdemux: g_return_if_fail in function with return type

Need to use g_return_val_if_fail() when the function returns a type, in this
case a gboolean

rtp: value truncated too short creates dead code

type is truncated to 0-31 with "& 0x1f", but right after that it is checks if
the value is equivalent to GST_H265_NAL_VPS, GST_H265_NAL_SPS, and
GST_H265_NAL_PPS (which are 32, 33, and 34 respectively). Obviously, this will
never be True if the value is maximum 31 after the truncation.
The intention of the code was to truncate to 0-63.

rtp: fix nal unit type check

After further investigation the previous commit is wrong. The code intended to
check if the type is 39 or the ranges 41-44 and 48-55. Just like gsth265parse.c
does. Type 40 would not be complete.

rtp: fix dead code and check for impossible values

nal_type is the index for a GstH265NalUnitType enum. There are two types of dead
code here:
First, after checking if nal_type is >= 39 there are two OR conditionals that
check if the value is in ranges higher than that number, so if nal_type >= 39
falls in the True branch those other conditions aren't checked and if it falls
in the False branch and they are checked, they will always also be False. They
are redundant.
Second, the enum has a range of 0 to 40. So the checks for ranges higher than 41
should never be True.
Removing this redundant checks.

CID 1249684

vtenc: Unlock the stream lock while waiting for pending frames to be completed in finish()

We might otherwise deadlock because gst_vtenc_enqueue_buffer() can be
called again and will take the stream lock again from another thread.

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

hlsdemux: implement _has_next_fragment to avoid busy looping

It will allow the demuxer to wait for a fragment to be available instead
of busy looping polling the playlist for a new fragment

decklink: Add a dummy value to the video connection mapping table for auto

decklinkaudiosrc: Add property to select the audio input connection

decklink: Add XLR and RCA audio connection enums

decklinkvideosrc: Add auto value for the connection property

This will use the default/auto connection for video capturing, and can
be set via the Decklink configuration tools.

decklink: Fix indention once again

decklink: Add property for configuring the input connection of the video sources

gl/cocoa: Don't init and clear static GMutex / GCond

We would potentially use it from the main loop later in
gst_gl_window_cocoa_init_nsapp() if it timed out before.

glfilter: remove logically dead code

Soon after setting two variables to 1, the code checks if their values are
different from each other. This would never be true. Removing this.

CID 1226443

Revert "pngparse: improve parsing of the image"

This reverts commit 59e7f0597d1cc524174e847de7b91407922304c8.

This patch should not have been pushed, see
https://bugzilla.gnome.org/show_bug.cgi?id=740058#c3

handdetect: remove unneeded check

Variable hands is already checked to contain a value previously at the beginning
of the current block (in line 504). There is no need to check again. This is
logically dead code.

CID 1197693

pngparse: improve parsing of the image

Everytime a buffer is being provided from baseparse, we are parsing all the data from the beginning.
But since we would have already parsed some of the data in the previous iterations,
it doesnt make much sense to keep parsing the same everytime.
Hence skipping the data which is already read in previous iterations to improve the parsing performance.

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

decklink: Fix latency values for sources

hlsdemux: remove duplicate call to uri_join

hlsdemux: skip checking '#EXT-X-' token for all entries

Put this common check before to avoid repeating it for all possible
entries to save some cycles

hlsdemux: avoid using g_list_append for creating segments list

Use g_list_prepend and reverse it at the end to skip traversing the
same list for every new segment

hlsdemux: Don't use approximate duration for fragment buffer pts

The duration values in playlists are approximate only, and for
playlist versions 2 and older they are only rounded integer values.
They cannot be used to timestamp buffers.  This resulted in playback
gaps and skips because the actual duration of fragments is slightly
different.  The solution is to only set the pts of the very first
buffer processed, not for each fragment.

adaptivedemux: only update segment if we have a valid PTS

Otherwise we set the position to GST_CLOCK_TIME_NONE and it disrupts
bitrate switching

mssdemux: fix crash while setting bitrate in caps structure

q->bitrate is a guint64, but G_TYPE_INT may read fewer bits
off the stack, and if we pass more then the NULL sentinel
may not be found at the right place, which in turn might
lead to crashes.

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

h264parse: Fix periodic SPS/PPS sending work after a seek

Reset the interval variable to have SPS/PPS sending work after
 a (backwards) seek.

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

wrappercamerabinsrc: fix deadlock with pad object lock

The image capture mutex and the pad object lock would cause a race
if the pad query was made right when the image probe was running.

The image probe needs the capture mutex and the querying would need
the pad object lock.

wrappercamerabinsrc: simplify weird if/else clause

It is not an if/else situation but an if error abort otherwise
just continue. Remove else to make it more readable

hlsdemux: Implement live seeking

hlsdemux assumes that seeking is not allowed for live streams,
however seek is possible if there are sufficient fragments in the
manifest. For example the BBC have live streams that contain 2 hours
of fragments.

The seek code for both live and on-demand is common code. The
difference between them is that an offset has to be calculated
for the timecode of the first fragment in the live playlist.

When hlsdemux starts to play a live stream, the possible seek range
is between 0 and A seconds. After some time has passed, the beginning of
the stream will no longer be available in the playlist and the seek
range is between B and C seconds.

Seek range:
start          0 ........... A
later               B ........... C

This commit adds code to keep a note of the B and C values
and the highest sequence number it has seen. Every time it updates the
media playlist, it walks the list of fragments, seeing if there is a
fragment with sequence number > highest_seen_sequence. If so, the values
of B and C are updated. The value of B is used when timestamping
buffers.

It also makes sure the seek range is never closer than three fragments
from the end of the playlist - see 6.3.3. "Playing the Playlist file"
of the HLS draft.

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

adaptivedemux: prepare for supporting seeks in live streams

Add function to allow subclasses to specify seeking range for
live streams

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

aggregator: don't use iterator when setting flush pending on pads

aggregator: check if pads are ready more efficiently

No need to use an iterator for this which creates a temporary
structure every time and also involves taking and releasing the
object lock many times in the course of iterating. Not to mention
all that GList handling in gst_aggregator_iterate_sinkpads().

aggregator: name vfunc arguments consistently

tests: nodist for orc generated code

aggregator: add g-i transfer and scope annotations

aggregator: register names of iterate_sinkpads functions with debug system

aggregator: reduce debug messages for taking/releasing logs to TRACE level

Don't spam debug log with this stuff.

aggregator: move property member into private structure

Our locking (or lack thereof) while accessing this also
looks generally quite dodgy.

aggregator: remove empty dispose function

aggregator: give private functions namespace prefix

Especially the GST_DEBUG_FUNCPTR ones.

aggregator: fix up some docs comments in header

aggregator: remove now-unused system clock member

glmixer, glvideomixer: update for GstAggregatorPadForeachFunc change

glmixer: update for aggregator start/stop vfunc change

aggregator: make GstAggregatorPadForeachFunc take an GstAggregatorPad

audiomixer: update for aggregator start/stop vfunc change

videoaggregator: update for aggregator start/stop vfunc change

aggregator: bring start/stop vfunc semantics in line with other baseclasses

Sub-class should not have to chain up to GstAggregator's start/stop
vfuncs, same as in GstBaseSrc, GstBaseSink, GstBaseTransform etc.

audiomixer: fix output-block-size property description

hlsdemux: only typefind when we have a minimum amount of data

For small amounts some data might be mistyped and it would cause
the pipeline to fail. For example if you have AAC inside mpegts,
for small amounts, the AAC samples would cause the typefinder to
think it is AAC and not mpegts.

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

decklink: Add support for 4K / UHD modes

decklink: Fix indention

dashdemux: do not try to ref null pointer

Playlists might be missing the availability start time, so check for
it before trying to ref the object

dashdemux: run gst-indent on mpdparser

Run gst-indent on mpdparser to have it stop complaining on every
commit

aggregator: remove pointless GST_DEBUG_FUNCPTR

Not useful for GObject vfuncs.

aggregator: remove duplicate pad parent_class variable

G_DEFINE_TYPE already provides one, just need to use it.

aggregator: add _CAST() variants for cast macros

aggregator: make padding larger

Esp. the class structures, can't have enough
spare space for virtual functions.

siren: remove unneeded structure padding

Public headers are private and not installed.

aggregator: Log to the pad instead of the element

More correct way of doing the same thing as before

glvideomixer: Point to compositor for the pad properties documentation

compositor: Document the pad properties

audiomixer: Document the pad properties

aggregator: Make sure that the minimum latencies are never GST_CLOCK_TIME_NONE

aggregator: Wait for the minimum latency, not the maximum

The minimum latency is the latency we have to wait at least
to guarantee that all upstreams have produced data. The maximum
latency has no meaning like that and shouldn't be used for waiting.

aggregator: Clamp the min latency at the max if it's greater

aggregator: Print the sinkpad name while logging latency queries

Very useful while debugging.

aggregator: Take the stream lock when iterating sink pads

When iterating sink pads to collect some data, we should take the stream lock so
we don't get stale data and possibly deadlock because of that. This fixes
a definitive deadlock in _wait_and_check() that manifests with high max
latencies in a live pipeline, and fixes other possible race conditions.

wrappercamerabinsrc: do not set source state with lock

It might be racy with the image probe thread as it uses the capture
mutex just like the start-capture handler from camerabin. The
start-capture would be waiting for the source's streaming thread
to stop to be able to set the source state to ready while the
probe would be blocked waiting to acquire the capture mutex.
It causes a deadlock.

wrappercamerabinsrc: use the drain query instead of flush

The flush might clear any pending captures that are still in
queues being processed while the drain query will wait for those
to be properly done

avfvideosrc: improve timestamp code

Use a combination of the running time and CMClockGetTime from AVF to compute
timestamps.

gdppay: use public API for accessing stream header arrays in caps

Don't rely on core implementation details, which are private and
may change. It's also not needed here, the performance impact is
close to none. Also copy buffer before changing its metadata.

tests: gdpdepay: update for internal gdp refactoring

This test uses some internal/private API, so needs to
be updated for recent changes.

gdppay: refactor payloading code a little

Get rid of some indirections and inefficiencies,
just payload things directly which gives us more
control over what memory is allocated where and
how and makes things much simpler. In particular,
we can now allocate the payload header plus the
GstMemory to represent it in one go.

gdppay: refactor a little

Get rid of now-useless packetizer struct and just
call internal functions directly. Also remove
version property which is now defunct, not least
because we create the packetizer with the
version in the init function before a version
can be set.

tests: gdppay: initialise memory of buffers used in unit test

gdppay: optimise payloading of buffers with multiple memories

Add function to calculate a payload CRC across multiple memories
so we don't have to merge buffers with multiple memories just to
calculate the CRC. Also make CRC calculation function static,
since it's not used outside dataprotocol.h and move special-casing
of length = 0 -> CRC = 0 into CRC function (from caller).

Perhaps more importantly, since payload CRC is off by default:
don't map buffer (and possibly merge memories in the process)
if we are not going to use it to calculate a CRC anyway.

gdppay: dataprotocol: drop bogus const

Doesn't really make sense given that we map
it and possibly merge memories and such.

hlsdemux: typefind might fail if first buffer is too short

If typefind fails, check to see if the buffer is too short for typefind. If this is the case,
prepend the decrypted buffer to the pending buffer and try again the next time around.

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

hlsdemux: port to adaptive base class

Conflicts:
ext/hls/gsthlsdemux.c
ext/hls/gsthlsdemux.h

adaptivedemux: check if element exists before setting it to null

Under certain error conditions it might happen that the element
has been freed already and trying to set it to NULL causes an
assertion

adaptivedemux: make sure task has stopped before freeing it

Otherwise the thread will be running and its memory will get freed.
This leads to memory corruption

adaptivedemux: fix segment start when exposing new streams

Segment start needs only to be updated when starting the streams
or after a seek, doing it during bitrate changes will cause the
running time to go discontinuous (jump back to a previous ts)
and QOS will drop buffers

adaptivedemux: allow subclass to use new pads group when switching bitrate

Fix pad groups switching when bitrate changes to allow HLS to use this feature

audiomixer: If getting a timeout before having caps, just advance our position

This can happen if this is a live pipeline and no source produced any buffer
and sent no caps until the an output buffer should've been produced according
to the latency.