Revert "Revert "openh264: fix to work with released versions and master of upstream""

This reverts commit 329f0ccba1aa604da1bd80293fd1fe717447df5d.

openh264: remove all trace output to console from encoder

decklink: Use BT2020 colorspace for UltraHD resolutions

decklink: Fix indention once again

audiovisualizer: handle the return of the setup function

Make the class future proof by handling the gboolean return of the setup
function. So if/when a child class uses this the base class is ready.

Revert "audiovisualizer: remove unused value"

This reverts commit 25c97570834d6de1c14424e71221d19eb0d33ee3.

It is preferable to handle the retun of the setup function.

decklinkvideosink: Get our own "start time" instead of the one of the pipeline

decklinkvideosink might be added later to the pipeline, or its state might
be handled separately from the pipeline. In which case the running time when
we (last) went into PLAYING state will be different from the pipeline's.

However we need our own start time to tell the Decklink API, which running
time should be displayed at the moment we go to PLAYING and start scheduled
rendering.

inter: Use a #define for the default value of the channel property

intervideosrc: Make the timeout configurable after which we start showing a black frame

audiovisualizer: remove unused value

klass->setup (scope) will always return TRUE since all children of this class
do so, no need to store the return. Besides, the value is overwritten a few
lines down before it is used.

Change helps keep files in sync after:
-base commit a91d521a3602f33083405467db9454d422b9da1b

compositor: Don't do any conversions if the pad is completely transparent anyway

inter: silence messages for flushed / created samples

These messages are logged periodically and polute the log

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

glcolorconvert: fix memleak

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

glupload: fix some memory leaks

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

glcontext: allow wrapped contexts to be utilised like non-wrapped contexts

Fill in the missing pieces like get_proc_address, the gl function vtable

gitignore: ignore more files

visual: use unused value

ret is assigned but not used and in the next cycle of the loop it is overwritten
with default_prepare_output_buffer (). If there is a flow error the function
should return instead.

CID #1226475

Automatic update of common submodule

From f2c6b95 to bc76a8b

decklink: Initialize refcount of our C++ classes in the constructor

CID 1262288
CID 1262287
CID 1262289

gaudieffects: stored value is overwritten

Value from left_luminance is assigned to out_luminance here, but that stored
value is not used before it is overwritten in the next cycle of the loop.
Removing assignation.

CID #1226473

mpegdemux: fix dead code

found is initialized to FALSE to then only be used in two conditional statements
that will always be false, making the blocks inside them dead code. Looking back
in the file's history the setting of the variable's value before it is checked
was dropped as part of the port to 0.11, bringing that value setting back.

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

mpegdemux: minor optimisation when scanning for SCRs

Bail out early when we're scanning for SCR, no need to
parse stuff we're not going to use nor interested in.

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