playback: Remove trailing whitespace

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

urisourcebin: Try to link output slot before cleanup

Before cleaning up output slot, check pending pads first, if available.
Then, cleanup it only if linking was failed.

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

urisourcebin: Cleanup unused output slot

Since urisourcebin cannot cleanup unused output slot
in streaming thread, it will be handled in thread pool
with gst_element_call_async ().

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

tagdemux: Fix crash when shutting down element during getrange()

Ensure that nothing is in any of the streaming thread functions
anymore when going from PAUSED to READY. While the parent's state change
function has deactivated all pads, there is nothing preventing
downstream from activating our srcpad again and calling the getrange()
function. Although we're in READY!

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

Automatic update of common submodule

From f980fd9 to 39ac2f5

typefind: add another test to itc typefinder

Report certainty after every test passes.

Additionally:

 - Remove self-explanatory comment.

ssaparse: Free initialization section before storing the next one

If getting multiple caps events.

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

oggdemux: Don't end up ignoring caps just because there are no headers for this stream

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

audioconvert: Error out if mapping input/output buffer failed

audioconvert: Don't map the input buffer in in-place mode

Input and output buffer are the same, let's not do unnecessary work.

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

audio-converter: In passthrough, also don't copy if in and out block are the same

In and out array are usually different, they are stack allocated arrays.
However the blocks inside them still can be the same.

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

audioconvert: Don't call transform_ip() in passthrough mode

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

multihandlesink: Fix buffers-queued being off by one

max_buffer_usage is the index of the oldest buffer in the queue,
starting at zero, not the number of buffers queued.

find_limits returns the index of the oldest buffer that satisfies the
limits in its min_idx parameter, not the number of buffers needed. Fix
this use too in order to keep passing the tests that read
buffers-queued.

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

multifdsink: Add a test involving a slow client

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

multihandlesink: Update bufpos in a separate pass

If a client gets dropped and the iteration gets restarted, bufpos is
incremented again for all clients that preceded the dropped one, causing
havoc.

Adjust the bufpos for all clients first before trying to drop any.

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

playbin: Fix caps memory leak in usage of gst_static_caps_get() API

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

win32: update .def file for new audioconverter API

Fixes distcheck.

meson: Add headers and libm to has_function checks

The functions from math.h may be implemented in libm.

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

discoverer: Handle NULL/ANY/EMPTY caps without crashing

check/videorate: Avoid leaking extra buffers

video-info: Properly initialize/set extra fields

The flags and field order weren't properly initialized in the
gst_video_info_init().

Furthermore in gst_video_info_from_caps() we might set unitiliazed
values previously, this only sets them if valid.

sdp: Fix attribute leak

We need to free the strdup'd string (to_free) in all cases

rtsp: Don't leak authorization string

audio-converter: optimize endian conversion

Optimize LE<->BE conversion by adding a dedicated fast path instead of
using the generic converter. Implement transform_ip function in order to do the
endian swap in place.

This saves buffer allocation for the intermediate format, can be done in place
and also performs the conversion in one step instead of unpack-convert-pack.

For all bit widths the naive algorithm is implemented, which provides the best
performance when compiled with -O3. ORC was considered but eventually removed
as it requires a dedicated function for in-place conversion (due to the
"restrict" parameters).

A more complex algorithm for the 24-bit conversion with unrolled loop and
32-bit processing is implemented in the #if 0 section. It performs better if
compiled with -O2. With -O3 however the naive algorithm performs better.

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

audio-convert: simplify the chain free process

It is not needed to store a pointer to every single chain element to free it.
Instead walk the channel list backwards and free the chain elements one by one.

Rename GstAudioConverter->chain_pack to chain_end.

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

subtitleoverlay: Fix caps memory leak when failing to get sinkpad from subtitle renderer

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

discoverer: Extract video information from caps manually without GstVideoInfo

The caps might not be fixated (which is required by GstVideoInfo) and we
would assert otherwise. However the caps often contain useful
information in the already-fixed parts that we can use here.

discoverer: Also stop waiting for subtitles if we get EOS

We're not going to get a buffer or GAP event anymore after EOS and would
wait forever otherwise.

riff-media: Check if caps are NULL before using them for the first time, not afterwards

Otherwise we'll get a g_critical() before erroring out cleanly on
https://samples.mplayerhq.hu/A-codecs/ATRAC3/SND0.AT3

common: use https protocol for common submodule

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

rtpbuffer: Fix ensure_buffers() if whole packet is in one GstMemory

When gst_rtp_buffer_add_extension_onebyte_header() is used over a
GstRtpBuffer that only contains a memory for the whole packet,
ensure_buffers function crashes at the next point:

mem = gst_memory_copy (rtp->map[i].memory, offset, rtp->size[i]);

when i==2 because the payload is not mapped.

In addition the offset is calculated subtracting in the wrong direction.

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

video-info: Add unit test for overflow checks

And also prevent overflows caused by allowing uint width/height in
gst_video_info_set_format() but storing them as (signed!) ints.

video-info: And change the overflow check to not actually overflow itself

video: Handle errors in gst_video_info_set_format() / gst_video_info_align()

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

video-info: Sanity check the frame size to prevent overflows

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

mikey: Generate the correct SRTP policy

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

rtspdefs: Assert on the excepted MD5 digest string length

CID 1394494.

typefind: bounds check windows ico detection

Fixes out of bounds read

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

tag: fix some warnings in mklicensestables

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

gstfdmemory: log with GST_INFO instead of GST_ERROR on permission denied

For example mmap can fail with EACCES if the the fd has been open
with read only mode. And mapping the memory might be the only way
to check that. So no need to print out an error.

Ex: ioctl(dev, DRM_IOCTL_PRIME_HANDLE_TO_FD, flags & ~DRM_RDWR)

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

typefind: add typefinder for Apple/iTunes itc artwork files

Avoids audio/mpeg false-positive described at:

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

rtsp: Add gst_rtsp_message_parse_auth_credentials() to parse authentication credentials

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

rtsp: Add gst_rtsp_generate_digest_auth_response() to calculate digest auth response

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

win32: remove copies of generated headers

dmabuf-allocator: Add missing padding in the class

This class was made subclassable, though for future growth of the code,
it's better if we have some room for add class members. Using the small
padding since this is unlikely.

textoverlay: Mark pad as needing reconfiguration again if it failed

And return FLUSHING instead of NOT_NEGOTIATED on flushing pads.

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

rtpbasepayload: Ensure to set the RECONFIGURE flag again if reconfiguration failed

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

meson: add_global_arguments -> add_project_arguments

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

videotimecode: Add test for the calculations of distance from the daily jam

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

videotimecode: Fix incorrect nsec_since_daily_jam calculation

For drop-frame timecodes, the nsec_since_daily_jam doesn't necessarily
directly correspond to this many hours/minutes/seconds/frames. We have
to get the frame count as per frames_since_daily_jam and then convert.

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

riff: Extract bpp from the strf for vnmc

Needed for avdec_vnmc to work.

playsink: warn if a custom sink is set that has no 'sink' pad

videodecoder, audiodecoder: parse format before checking in src_query_default

The logic change in these commits misordered the parsing and checking of
format in position queries:

 2b06e54 videodecoder: Don't answer BYTES queries
 1840b02 audio: Don't answer BYTES queries

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

appsink: fix g-i warnings and add since markers

Rename function parameter and make sure the name in the
declaration matches the name in the implementation, to
avoid g-i warnings. Also add Since markers for gtk-doc.

gstappsink.c:1248: Warning: GstApp: gst_app_sink_set_buffer_list_support:
unknown parameter 'buffer_list' in documentation comment, should be 'drop'

discoverer: Do not try to unref the bus if it has not been set yet

It might happen if creation of the discoverer failed

appsink: add support for buffer lists

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

playbin3: remove dead code

It never reach into this code path, custom_combiner always not null
here.

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

videodecoder: Don't answer BYTES queries

Refuse to answer BYTES queries ourselves. The only
time they make sense is on raw elementary streams,
in which case upstream would already have answered.

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

audio: Don't answer BYTES queries

Refuse to answer BYTES queries ourselves. The only
time they make sense is on raw elementary streams,
in which case upstream would already have answered.

They especially don't make sense for encoders to answer
based on upstream values - although perhaps later
we could make it do TIME->BYTES conversion on the source
pad based on bitrate.

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

sdp: cast away const in call to g_free

MSVC warns about the const here. It's safe to cast away.

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

Cast away const from GstMetaInfo in *_get_meta_info() functions

MSVC warns about the const in the implicit argument conversion in the
calls to g_once_init_{enter,leave}. It's OK so explicitly cast it.

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

decodebin3: Clear saw_eos flag of DecodebinInputStream by FLUSH event

Likewise how GstPad is doing, saw_eos flag of DecodebinInputStream
must be cleared by FLUSH event.

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

playbin3: Fix deadlock when adding multiple parsebin

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

meson: vorbis: Add -DTREMOR to flags for gstivorbisdec

Matching the flags set by Makefile.am

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

meson: Fix build when orc is disabled

Making sure not to use the orc_dep variable in case
orc has been explicitely disabled.

Use intermediate guint when handling GstVideoMultiviewFlags

The underlying integer type of the enum GstVideoMultiviewFlags is
implementation defined and may not have the same size as guint.

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

Remove 'return' from `void` functions

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

meson: don't add_global_arguments when being built as a subproject

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

rtsp: Include GstSdp-1.0.gir when generating the gir

It is actually needed as we need some symbols. We do not link
to libgstsdp as the user of the lib should do it (same with
autotools build).

This reverts previous commit

libs:rtsp: Remove wrong dependency on Sdp for the gir file

libs:rtsp: Remove wrong dependency on Sdp for the gir file

meson: Generate girs

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

audio-channels: map buffer read-write only if channels differ

gst_audio_buffer_reorder_channels() was always mapping the buffer read-write
regardless whether any reordering was needed.  If the from and to channel order
is identical return immediately without remapping the buffer.

Add a small helper function gst_audio_channel_positions_equal() which is used
in both gst_audio_reorder_channels() and gst_audio_buffer_reorder_channels().

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

videorate: Add fixed rate property

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

opusdec: fix 120 ms buffers being wrongly emitted

Using the max 120 ms buffer size to ensure we have enough space
for decoded data meant that Opus could actually return 120 ms'
worth of data.

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

win32: Update exports for new API

video: add gst_video_decoder_allocate_output_frame_with_params

It adds a third argument to pass GstBufferPoolAcquireParams
to gst_buffer_pool_acquire_buffer.

If a user subclasses GstBufferPoolAcquireParams, this allows to
pass an updated param to the underlying buffer pool at each
gst_video_decoder_allocate_output_frame_with_params call.

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

video-info: Fix the docs to say interlace-mode, not interlaced-mode

win32: add new API to .def file

Fixes make check and make distcheck

allocators: define GST_CAPS_FEATURE_MEMORY_DMABUF

Adds "memory:DMABuf" caps feature. Since 1.11 tag.
Useful when the the dma-buf buffer cannot be mapped to CPU for r/w requests.
Example: protected content or platform constraints.

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

dmabuf: Make the allocator sub-classable

This should allos for cleaner code when implement such allocator.

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

audiosrc: Leave read loop if no longer running

In the case a src stops providing data (read calls returns 0). The audio
src thread will never leave. Instead, check the condition and leave the
loop.

check: Fix corrupted xml check files

By making sure each different videoscale check instance gets logged
into different output file

video: Update orc generated files

meson: Add pbtypes plugin

pbtypes: Add a stub plugin that owns the plugins-base dynamic types

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

urisourcebin: Fix adaptive demuxer's property checking and buffering setting

- Add adaptive demuxer's 'connection-speed' property checking
- Set adaptive demuxer q2 buffering property via urisrc use_buffering value

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

Revert "video-color: Allow converting incomplete colorimetry to a string"

This reverts commit 158eae7e7e3da3545712dd7d6121492c53085fd9.

It already *always* allowed to convert incomplete colorimetry to a
string.

rtpbasedepayload: Reject non-TIME segments

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

Revert "basertpdepayload: create valid segment when given non-time segment"

This reverts commit 0f609bc6c67fea294f4556627228fed72a74d0fb.

video-color: Allow converting incomplete colorimetry to a string

This is only a good idea for non-raw caps.

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

videoencoder: Proxy colorimetry and chroma-site from input to output caps

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

videodecoder: Proxy field order to the output caps

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

videoencoder: Proxy interlace-mode and field-order fields from the input to the output caps

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

video-info: Add optional field-order caps field for interlaced-mode=interleaved

Usually this information is static for the whole stream, and various
container formats store this information inside the headers for the
whole stream.

Having it inside the caps for these cases simplifies code and makes it
possible to express these requirements more explicitly with the caps.

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

meson: update version

urisourcebin: Remove trailing whitespace

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

videotestsrc: Make snow deterministic

Deterministic generation of snow and smpte is important for tests so
that it's not affected by other videotestsrc elements in current or
possibly previous tests.

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

audioconvert: optimize mask calculation

find_suitable_mask() had complexity O(n^2) on the number of bits.
For common case like 2-channel audio the mask was calculated in about 4k loop
cycles.

Optimize both n_bits_set() and find_suitable_mask() to O(n) where n is the
number of bits set in the mask.

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

parsebin: Rename variables include 'decode' to 'parse'

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