convertframe: Fix async video sample conversion with non-default context

The GSource for dealing with timeouts in
gst_video_convert_sample_async() might be attached to a non-default
context, so we should not be using g_source_remove() on the returned ID.

The correct thing to do is to keep a reference to the actual GSource and
then call g_source_destroy() on it.

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

urisourcebin: Fix typefind list leak

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

adder: cleanup the tests

Take a first stab at cleaning up the tests. Extract common code. Make sure
we actually verify things.

decodebin: Close a small race posting 100% buffering

When posting 100% buffering due to removing the last
buffering element, we still need to hold the posting
lock as well, to avoid any race with other elements
that might post a buffering message at that exact
moment

urisourcebin: Fix buffering message aggregation.

Add locking, and handle EOS properly now that urisourcebin
uses custom events in place of real EOS events, so we
need to manually remove buffering messages and potentially
post 100% in that situation

urisourcebin: Store slot info on the queue object too

The buffering message handling tries to look up the
slot info on the queue itself.

videodecoder: Add debug output tracking decode-after-flush

Track how long it takes to generate the first buffer after a flush
as a simple measure of how efficient the decoder is at skipping /
rushing to get to the first decode.

videotestsrc: Publish multiview-mode info in the caps

Don't allow downstream to accidentally pretend that
the output is anything than a mono or single-eye
left/right view.

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

video-format: Shift correctly when packing I420_12BE

video-format: Order all formats in GST_VIDEO_FORMATS_ALL like in the enum

And remove duplicated entries.

encoding-profile: Mark format caps as transfer-none in profile creation

video-format: Add Since markers to all new formats from 1.2

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

video: Add I420/I422/Y444_12LE/BE and GBRA video formats

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

video: Add GBRA_10LE/BE, GBR_12LE/BE, GBRA_12LE/BE pixel formats

With additional bugfixes from Sebastian Dröge <sebastian@centricular.com>

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

multifdsink: Remove GValueArray from the doc

It's been a GstStructure for a long time. Also fix
a typo in the name of one of the fields.

Add old libgstencodebin.so to cruft list

This will help fixing clash in gst-uninstalled setup.

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

docs: Convert gtkdoc comments to markdown

Modernizing the documentation, making it simpler to read an
modify and allowing us to possibly switch to hotdoc in the
future.

videotimecode: Fix invalid timecode right before a new second

When initializing a timecode from a GDateTime, and the remaining time
until the new second is less than half a frame (according to the given
frame rate), it would lead to the creation of an invalid timecode, e.g.
00:00:00:25 (at 25 fps) instead of 00:00:01:00. Fixed.

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

videodecoder: fix build error on i386

Use G_GUINT64_FORMAT for guint64 values.

Introduced by fcb63e77a9de42cfb50da03a4925b06704006be3

Found by Alexander Larsson

gstvideodecoder.c: In function 'gst_video_decoder_have_frame':
gstvideodecoder.c:3312:51: error: format '%u' expects argument of type 'unsigned int', but argument 8 has type 'guint64 {aka long long unsigned int}' [-Werror=format=]

Fix plugin filenames to match pugin names

 - libgstencodebin.so is now libgstencoding.so
 - libgstximage.so is now libgstximagesink.so (meson only)

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

urisourcebin: Shrink queue2 max-size-time buffering

The expanded 4 second buffering was making radio streams that are
being delivered at real-time speeds too slow. We might need
a better plan for matching the queue2 size to incoming bitrate
in the absence of tag information or timestamping.

In uridecodebin, it used tags on the output of decodebin to
adjust the queue2 buffering, but urisourcebin doesn't have that
view - decodebin is downstream from us.

videotimecode: Unref GDateTime in error cases

playbin3/urisourcebin: Enable buffering by default

This only has effect on network streams in these elements, and by
enabling it by default we get the same behaviour as in playbin2.

tools: gst-play: improve verbose output of property notifications

Use new message notify API and print caps and taglists in a nicer
to read way, just like gst-launch-1.0 does nowadays, without
escaping everything three times.

videodecoder: Fix a leak introduced in fcb63e

CID: 1402055

oggdemux: Fix reverse playback

Fix various issues with reverse playback by clearing tracking
vars when working in reverse, and where possible using the
timestamp interpolation code to generate timestamps for
outgoing buffers. Make sure to mark things as discontinuous
only when looping backward to a new position and fix seeking
to the next page when starting.

oggdemux: Timestamp tracking fixes

In gst_ogg_demux_do_seek() when calculating the
keyframe time, account for a non-zero start-time

Handle a discontinuous first packet in
gst_ogg_demux_setup_first_granule() because that's pretty
normal after a seek. Also differentiate between a genuinely
truncated first packet and just bailing out early, by not using
granule = -1 as an error code.

Make the debug output logs clearer about which timestamps
are stream times (PTS) and which are ogg timestamps.

oggdemux: Don't arbitrarily guess a timestamp of 0

When we haven't managed to manufacture a timestamp for
a packet, don't just guess '0', leave it at none and
let downstream decide

theoradec: Make the keyframe condition check clearer

Make the keyframe checking if statement easier to
read, and add some debug when detecting a keyframe

videodecoder: Restrict frame timestamp guessing in reverse mode

Don't guess a timestamp of the start of the segment when running
in reverse mode, as more likely it means we're discontinuous somewhere
in the middle of the segment, and we'll fix up timestamps once
the frames are decoded and reversed.

videodecoder: Store buffer information even without PTS

When a PTS is not set, we still want to store the rest of the
buffer information, or else we lose important things like the
duration or buffer flags when parsing.

vorbisdec: Reset decoder in more situations

This is a followup commit to b95725c37e70ad3c1ec8dadb401388db375df482

* Resetting the decoder should only happen when we get a new initialization
header (0x01) and not on the other headers
* The initialized variable only gets set to TRUE once all headers have
been parsed. Also check if the vorbis_info struct has been properly resetted
also. Failure to do that would cause vorbisdec to error if it got
two initialization header in a row (the first would configure the underlying
library and the second one would error out because it's already initialized)

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

rawbaseparse: Assert that frame size is > 0

We would later divide by zero otherwise, and generally won't do the
right thing.

CID 1401383

rawaudioparse: Fix potential NULL pointer dereference

CID 1364606

decodebin3: Fix potential NULL pointer dereference on slot deactivation

CID 1363331

urisourcebin: Fix inverted check for an existing slot

CID 1363330

decodebin3: Handle stream-start events without group id correctly

CID 1363327

videodecoder: Handle return value of gst_video_decoder_drain()

CID 1362896

vorbisdec: reset decoder on vorbis headers update

if the vorbis encoder receives new headers it must be
reset and re-initialized to continue decoding, e. g.
for live streams

vorbisenc: Resend (new, updated) headers when the caps are changing

The headers are usually changing together with the caps, and they are
nedeed for correct decoding.

rawparse: fix linking for audio/video libs

Broke build in gnome continuous

rawparse: Fix meson build

rawparse: Move to gst-plugins-base

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

Merge branch 'rawparse-moved-plugin'

meson: Update version

Back to development

Release 1.11.2

Update .po files

video-converter: Give a name to the thread-pool threads

This way they can be distinguished from any other threads in the same
process.

video-converter: Only lock the thread pool mutex when running with more than 1 thread

There's no reason to lock anything if only the current thread is ever
going to do any work.

video-converter: Implement multi-threaded scaling/conversion

This adds a property to select the maximum number of threads to use for
conversion and scaling. During processing, each plane is split into
an equal number of consecutive lines that are then processed by each
thread.

During tests, this gave up to 1.8x speedup with 2 threads and up to 3.2x
speedup with 4 threads when converting e.g. 1080p to 4k in v210.

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

videotimecode: Validate for drop-frame correctness

In gst_video_time_code_is_valid, also check for invalid
ranges when using drop-frame TC. Refactor some code which
broke after the check was added.

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

videotimecode: Init from GDateTime

Add a function to init the time code from a GDateTime

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

vorbis: Fix channel reorder map for 5.1, 6.1 and 7.1

rawbaseparse: Drop incomplete frames at EOS

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

This would ideally be solved in baseparse but that requires further
thought at this point, and in the meantime it would be good to have
rawbaseparse not assert on this but handle it gracefully instead.

video-scaler: Fix upscaling if width & height change and we're starting not at y=0

It was taking the initial input y-offset from the output value, which
only works for y=0 (in which case both are the same). If y > 0, we would
always stay behind the requested input offset and never ever read
anything from the input.

Fix distcheck

Buildbot doesn't like wildcards here for some reason.

meson: dist meson build files

Ship meson build files in tarballs, so people who use tarballs
in their builds can start playing with meson already.

win32: dist all .def files

typefindfunctions: prevent unsigned int overflow

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

adder: ensure the discont flag is correct

Previously it happened that reused buffer caused the discont to be on the wrong
buffers.

discoverer: Ignore more parser related fields when comparing streams

The parser might do some conversion on a stream but the stream keeps
being the same, and we need to make sure GstDiscoverer detects it is the
case.

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

audioringbuffer: Also add FLAC to debug strings.

Oops, also add FLAC to the debug strings array.

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

audioringbuffer: Prevent overflow of debug names array

Add new audio types to the list of strings used for debug
so we don't index past the end of that array.

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

multifdsink: Make sure to use a 64 bit integer for the units-max property

decodebin3: Fix crash caused by slot double freeing

Probe for MultiQueue source pad might receive EOS twice,
the first is fake-eos and the other is actual EOS.
And the slot can be freed with fake-eos/EOS if the slot has no input.

Since slot freeing is async, double free can be possible.
So, decodebin3 needs to remove the probe also with slot freeing.

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

examples: Fix leak

decodebin3: Fix list leak on handle_stream_switch()

Free no more used list variables

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

decodebin3: Change requested_selection to have its own memory for stream-id

"requested_selection" list might be generated by select-streams event.
And memory of stream-id(s) in select-streams is independent from that of stream-collection.

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

decodebin3: Change return types of stream_in_{list,collection}

Change return types of functions to get memory address of stream-id.

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

playback: Fix leak on select_streams

Since gst_event_parse_select_streams() returns newly allocated
memory for stream-id(s), it should be freed explicitly.

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

encodebin: fix caps leak in test

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

audio-resampler: Fix integer overflow in clamping code

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

qt: The videooverlay example requires at least C++11

... and clang requires this to be specified on the commandline while gcc
nowadays defaults to C++11 or even newer.

meson: Properly use ':' for defining keywords

videorate: fix LATENCY query

The latency query originally had a fallthrough to the default
label at the end as fallback, but that got messed up when the
DURATION and POSITION queries were added, so it then fell through
to the duration query handler instead. Restore original behaviour.

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

videorate: fix duration and position query handling

Duration query would return TRUE and duration=-1. This
worked in the unit test because the unit test implementation
was a bit broken.

Both queries need to access rate with a lock.

Fix broken duration query test as well. It relied on broken
behaviour by the videorate query handler, and also it was
implemented as a downstream query rather than an upstream
query. And we must return HANDLED from the probe so that the
query we intercept actually returns TRUE.

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

audio: add since markers to docs for new enums

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

audio: add FLAC to GstAudioRingBufferFormatType

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

audioringbuffer: Also support raw AAC

Support raw AAC streams without the ADTS header

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

docs: update libs section

Include documented symbols that were not declared in section file.

riff-media: Don't divide block align by zero channels

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

samiparse: Check that the string has a non-zero length before overwriting the last byte with '\0'

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

riff-media: Don't recurse in for nested WAVEFORMATEX

There was already a check for that, but it failed because
subformat_guid[0] is a guint32 and that is then casted implicitely to a
guint16 when recursing... just that we checked the uncasted value.

This caused an infinite recursion and thus stack overflow.

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

decodebin: Don't leak blocked pad references on errors

When the decodebin state change fails because of an error
message, we might not go through PAUSED->READY. Don't leak
a ref to decodebin pads due to pad blocking in that case.

This is because we return ASYNC going to PAUSED, and if
we fail before reaching PAUSED the only transition we'll
see is READY->NULL.

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

videotestsrc: Add options to make ball pattern based on system time, and invert each second.

This adds some extra options that affect pattern=ball mode, allowing the
animation to be synced to running time or wall-time clock for comparing
sync across different instances / pipelines / machines.

Also added is the ability to invert the rendering colours every second,
and some different ball motion patterns.

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

riff-media: Check for valid channels/rate before using the values

Otherwise we might divide by zero or otherwise create invalid caps.

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

meson: bump version

Back to development

Release 1.11.1

Update .po files

po: Update translations

urisourcebin: Preserve seqnum on EOS events

When converting EOS to/from our custom fake EOS event,
preserve any seqnum on the original event.

urisourcebin: Avoid races when setting up typefind

The state of urisourcebin (and all elements contained within) can
change at any point in time, including when setting up the typefind
element.

In order to avoid ending up with typefind starting without being fully
connected, lock the state and connect to the 'have-type' signal.

video-converter: Fix crashes in fast-paths when converting interlaced formats with different vertical subsampling

E.g. the following pipelines fail because chroma values after the last
line are read (note: 486 % 4 == 2):

gst-launch-1.0 videotestsrc ! "video/x-raw,interlace-mode=interleaved,width=720,height=486,format=UYVY" ! videoconvert ! "video/x-raw,format=I420" ! fakesink
gst-launch-1.0 videotestsrc ! "video/x-raw,interlace-mode=interleaved,width=720,height=486,format=I420" ! videoconvert ! "video/x-raw,format=UYVY" ! fakesink
gst-launch-1.0 videotestsrc ! "video/x-raw,interlace-mode=interleaved,width=720,height=486,format=I420" ! videoconvert ! "video/x-raw,format=AYUV" ! fakesink

urisourcebin: Drop fake EOS if the pad got relinked

If our pad got relinked in since the fake-EOS was sent to the
pad, then drop the fake-EOS event.

CID: 1398546

common: Revert unintentional change in previous commit

urisourcebin: Drop some dead code

The ret == GST_STATE_CHANGE_FAILURE is handled in the previous for loop
already.

CID: 1398544

urisourcebin: Clear pad from pending list if it was linked

If not, the other slots might try to link the pad again.
This can happen when the demuxer has multiple src pads
and their caps are identical

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

win32: update def file