Release 1.7.91

Update .po files

po: Update translations

decodebin: Shut down all elements explicitly to NULL state before freeing the decode chain

Due to transient locked state during autoplugging, some elements might be
ignored by the GstBin::change_state() and might still be running. Which could
then cause pad-added and similar accessing decodebin state that does not exist
anymore, and crash.

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

multihandlesink: Remove useless streamheader storage

We don't do anything with it but always get them from the caps anyway, so
stop storing them and having complicated logic around that.

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

multihandlesink: Only don't send HEADER buffers normally if they are actually streamheaders from the caps

And also consider HEADER buffers without DELTA_UNIT flag as sync points. This
fixes sync-mode=2 with mpegtsmux for example, which has no streamheaders but
puts the HEADER flag on its keyframes.

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

decodebin: expose_pad() is always called with lock==TRUE, simplify code

This basically reverts ee44337fc3e3030a5155d28b3561af157e6c6003 .

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

decodebin: Don't check twice if the decode chain is complete in pad_added_cb()

expose_pad() already does the same.

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

decodebin: Don't hold EXPOSE_LOCK in type_found() outside the stream lock

In other places we lock it the other way around, leading to possible
deadlocks. Also this will deadlock if analyze_pad() causes a new element to be
autoplugged that adds new pads on itself when its state is changed.

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

tcp: Remove unused file

It's a copy of multihandlesink, but completely outdated. Let's get rid of it
before it gets even more outdated.

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

basetextoverlay: Add new properties and alignment type for unclamped absolute positions

Introduces [x-absolute, y-absolute] properties
for positioning in +/- MAX_DOUBLE range.

Adds new (h/v)alignment type "absolute" where coordinates
map the text area to be exactly inside of video canvas for [0, 0] - [1, 1]:

[0, 0]: Top-Lefts of video and text are aligned
[0.5, 0.5]: Centers are aligned
[1, 1]: Bottom-Rights are aligned

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

Revert "textoverlay: Do not limit positioning to video area."

This reverts commit a48daf6dd8cb69b4260a03aa7f3cdf227d4f1602.

This changed behaviour in a way that's not always
backwards-compatible.

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

win32: Add a module definitions file for gstfft

theora: fix performance category initialisation

Remove unused _register() functions and look up the performance
debug category in a function that's actually called at some point.

audio-channel-mixer: improve non-interleaved flags

Make separate flags for non-interleaved input and output because the
channel mixer should be able to convert between the two layouts in the
future.

gst-play: remove peculiar setting of invalid -v property

oggdemux: fix chaining causing running time to restart from 0

This fixes:
gst-play-1.0 http://relay-nyc.gameowls.com:8000/chiptune.ogg

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

opusdec: plug caps leak

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

Revert "playbin: use avdeinterlace for deinterlacing until deinterlace is ported"

This reverts commit 0615794300234e3efbcb49a524efdee11171ab4c.

deinterlace was ported at some point in the last 4 years and has better video
format support, and especially better negotiation than avdeinterlace. Having
avdeinterlace but not deinterlace causes various problems in zerocopy
scenarios.

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

encodebin: Make dispose() function safe to be called multiple times

Release 1.7.90

po: Update translations

multisocketsink: handle client close correctly and EWOULDBLOCK

Fixes 100% cpu usage when client disconnects. Commit 6db2ee56
would just make multisocketsink ignore reads of 0 bytes without
removing the client, so we'd get woken up over and over again
for the client.

Fix the original issue differently by handling the non-fatal error code.

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

video: update disted orc backup file

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

video-converter: add direct UYVY to GRAY8 conversion function

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

opus: fix mono<->stereo up/down-mixing

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

pbutils: docs: Remove the empty lines in between <refsect2> and </refsect2>

They are converted into <para></para> by gtk-doc...

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

Automatic update of common submodule

From b64f03f to 6f2d209

opusenc: remove deprecated "cbr", "audio", and "constrained-vbr" properties

They have been replaced by "audio-type" and "bitrate-type".

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

docs: add Opus to docs

opus: move Opus audio decoder and encoder from -bad to -base

Hook into build system after moving history.

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

Merge branch 'plugin-move-opus'

Move Opus decoder and encoder from -bad to -base.

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

tools: gst-play: add 'n' and 'b' as additional shortcuts for next/previous item

< and > are composed with shift + something else on many keyboards
layouts, so don't work well when injecting them via windowing systems
which will send them as shift key press and separate other key, and
we the don't combine that to < or > properly. n/b are easier.

audiovisualizer: Use the library instead of including the source file

Fixes build now that the shader enum GType has moved to a different file.

audiovisualizer: Let GstAudioVisualizerShader enum GType be autogenerated by glib-mkenums

That happens automatically already anyway.

video: flesh out docs for gst_video_frame_map()

visual: correct type name

Base class type name should not reference libvisual since not all child
elements use this. This was an oversight when merging audiovisualizers into
a common base class.

audio-quantize: fix feedback dither

Make sure we allocated enough extra space in the error buffer to
store the feedback error.

audio-converter: perform dithering on the current format

Use the current (intermediate) format to decide how to set up dithering
instead of the input format.

rtpbasepayload: Handle gst_pad_get_current_caps() returning NULL gracefully

Revert "playsink: Properly mark pending blocked pads"

This reverts commit 62053852de01fb324a915b27c00f5b8dc0f66fb3.

The issue that the patch fixes is only noticeable when using decodebin3,
which isn't yet in master.

tag: id3v2: read conductor tag

ID3v2 features the TPE3 info frame, which contains information
about the conductor.

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

Fix use of undeclared core debug category symbols

libgstreamer currently exports some debug category
symbols GST_CAT_*, but those are not declared in any
public headers.

Some plugins and libgstvideo just use GST_DEBUG_CATEGORY_EXTERN()
to declare and use those, but that's just not right at
all, and it won't work on Windows with MSVC. Instead look
up the categories via the API.

win32: remove outdated build cruft

This hasn't been touched for generations, doesn't work,
and is just causing confusion. We also don't want to
maintain these files manually.

Back to development

Release 1.7.2

po: Update translations

uninstalled.pc: add support for non libtool build systems

Currently the .la path is provided which requires to use libtool as
mentioned in the GStreamer manual section-helloworld-compilerun.html.
It is fine as long as the application is built using libtool.

So currently it is not possible to compile a GStreamer application
within gst-uninstalled with CMake or other build system different
than autotools.

This patch allows to do the following in gst-uninstalled env:
gcc test.c -o test $(pkg-config --cflags --libs gstreamer-1.0 \
    gstreamer-video-1.0)
Previously it required to prepend libtool --mode=link

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

typefind: strengthen check for valid H.263 picture layer

Avoids some false positives leading to miss identification:

* Prevent picture start code emulation for the first 2 bytes read
* Add check for valid "picture coding type" and "PB-frames mode" combination

Additionally, change name on confusingly named TR var to what
it is, the layer's PTYPE.

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

decodebin: return incomplete topology if decode chains' cap could not be obtained

When getting caps of the decode chain, in get_topology, the caps are being
checked if fixed or not. But get_topology will be called when the decode is
chain is being exposed and hence it will always be fixed. Hence removing the
check for fixed caps. Removing gst_pad_get_current_caps for the chain->pad, as
get_pad_caps will again call the same api.

And get_topology can return NULL value if currently shutting down the
pipeline, which on being passed to create message will result in assertion
error. Check if topology is valid before using it

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

rtp: build audio library before rtp

Because audio-enumtypes.h needs to be available for
gstrtpbaseaudiopayload.c

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

decodebin: Fix documentation of the autoplug-query signal

videoencoder: Fix leak when pre_push does not return OK

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

resample: avoid overflows

Avoid overflow in rate calculation. This can cause the resampler to
start on the wrong phase after a rate change.
Avoid overflow in cubic fraction calculation. This can cause noise when
dealing with higher samplerates.

resample: fix double interpolation sse code

We were only reading 2 filter taps and we need to read 4 to do cubic
interpolation.

audio-converter: make a copy if we can't write in unpack

If we don't have writable memory, make sure to make a copy of the input
samples into a temporary (writable) buffer, even if we are dealing with
a native intermediate format that we don't need to call the unpack
function for.

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

tests: extend the AM_TESTS_ENVIRONMENT from check.mak

To get the CK_DEFAULT_TIMEOUT defined for all tests.

Also replaces a 120 timeout that was set.

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

Automatic update of common submodule

From 86e4663 to b64f03f

textoverlay: Expose rendering dimensions as properties.

In order to detect graphical user input on the
textoverlay, the resulting rendering properties
need to be exposed to applications.

Fixes delayx property declaration.

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

textoverlay: Do not limit positioning to video area.

The current position property is limited to X,Y positions
in the range of [0, 1]. This patch allows full control
over the overlay position, including partially outside
of the video area.

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

opus: fix FEC

FEC may only be used when PLC is enabled on the audio decoder,
as it relies on empty buffers to generate audio from the next
buffer. Hooking to the gap events doesn't work as the audio
decoder does not like more buffers output than it sends.

The length of data to generate using FEC from the next packet
is determined by rounding the gap duration to nearest. This
ensures that duration imprecision does not cause quantization
to 2.5 milliseconds less than available. Doing so causes the
Opus API to fail decoding. Such duration imprecision is common
in live cases.

The buffer to consider when determining the length of audio
to be decoded is the previous buffer when using FEC, and the
new buffer otherwise. In the FEC case, this means we determine
the amount of audio from the previous buffer, whether it was
missing or not (and get the data either from this buffer, or
the current one if the previous one was missing).

opusdec: fix wrong buffer being checked for missing data

This caused a decoding error if the resulting (wrong) buffer size
was passed to the Opus decoding API.

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

audiorate: Use gst_audio_format_fill_silence() instead of memset with 0 for generating silence

For unsigned formats, silence is not all bits 0.

audio/videodecoder: Minor cleanup of last commit

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

audio/videodecoder: use gst_pad_peer_query_caps to make output caps

gst_pad_get_allowed_caps() will return NULL if the srcpad has no peer.
In that case, use gst_pad_peer_query_caps() with template caps as filter
to have negotiated output caps properly before forwarding GAP event.

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

encodebin: Allow streamheader update when profile.allow_dynamic_output == FALSE

Some encoders can update the stream header through time (for example
vp8 might do that) but it does not strictly changes the output format.

video-format: fix GstVideoFormatInfo documentation warnings

Add missing ':' to tile_ws and tile_hs fields documentation to avoid
bad render of these two fields, mark reserved bytes as private to hide
field and avoid gtkdoc warning and add parameters description to
documented macro to avoid gtkdoc warnings.

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

audio-converter: add reset function

audio-converter: handle NULL input

Allow NULL as input to mean silence samples.

audio-converter: improve _update_config

Allow NULL config to keep the existing parameters.
Fix the docs.

audio-converter: audio-converter: make some optimized functions

Make optimized functions for generic and passthrough conversion.

audio-quantize: add _reset function

Add a reset function that clears any history.

build: remove nonsensical check for freetype

The examples need Gtk+, nothing uses freetype directly.

tests: libvisual: make run faster

Reduce resolution, which shouldn't make any difference
to what's tested here. Makes test finish in less than
half the time it took before (8s vs. 21s).

alsa: Trivial doc update

alsasink now does more than just raw audio.

decodebin: Correctly expose pads from elements that have directly exposable pads

analyze_new_pad() can return a new decode chain, which might have a new
GstDecodePad in the end. We should use those two for expose_pad() and not the
original ones that were passed to analyze_new_pad().

This fails when having a demuxer element that has raw pads immediately or
if a decoder with raw caps is after an adaptive demuxer.

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

audio-converter: ensure correct alignment of samples

Make sure that the data we allocate for our temporary buffers is
properly aligned.

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

video-color: add Adobe RGB primaries and transfer function

video-info: enfore RGB matrix for RGB formats

In gst_video_info_to_caps(), make sure we end up with an RGB matrix for
RGB formats and warn when the GstVideoInfo colorimetry is wrong.
In gst_video_info_from_caps(), fix the GstVideoInfo with an RGB matrix
for RGB formats and warn about inconsistent caps.

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

video-converter: ignore matrix for RGB formats

For RGB formats, the matrix in the colorimetry (conversion from YUV to
RGB) is irrelevant and we should ignore it and assume the identity
transform for everything we do.

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

videoencoder: Deprecate GST_VIDEO_ENCODER_FLOW_DROPPED

It was never actually supported or used

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

Revert "videoencoder: Release video frame when ->handle return ERROR or DROPPED"

This reverts commit 63517d0ed348784cce4ab4b295c2c0f1b78baa81.

It was wrong ref counting wise and we decided to deprecated DROPPED
return value

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

tests:audioconvert: Fix integer overflow build error

value of 32768L << 16 and 1L << 31 is 2147483648
but it exceeds the positive range of int which is 2147483647
resulting in integer overflow error. Use G_GINT64_CONSTANT instead of L.

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

appsrc: Minor documentation cleanup

tools: gst-play: allow setting of flags in serialized foo+bar format

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

tools: gst-play: add command line options for verbose output and playbin flags

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

win32: Update exports

Add WAIT_ON_EOS flag to gstappsink.

If set, an appsink that receives an EOS will wait until all of its buffers have been processed before continuing.

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

audioencoder: Add note to the documentation about various settings being reset before set_format()

It's quite unexpected behaviour that various subclass settings are just
reset before set_format(). Unfortunately changing this now has the risk
of breaking existing code but we should reconsider this for 2.0.

streamsynchronizer: Ignore flushing streams [..]

[..] when resetting group start time. In GES, we are usually connected
to the streamsynchronizer on one audio and one video pad.

When seeking the timeline, both nlecompositions often output their flush_start
before any of them has output its flush_stop.

The current code, when receiving the first flush stop was using the
running time of the start of the second composition, which could
be pretty much anything, and means nothing at that point.

This patch is thread-safe, as STREAM_SYNCHRONIZER_LOCK is taken
both when setting flushing and when checking it.

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

playbin: Only append non-raw and sysmem pad template caps to the autoplug-query result

Otherwise a decoder supporting GL memory will think that all downstream can
support GL memory because of seeing its own template caps.

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

Revert "playbin: only add the template caps when the result is empty"

This reverts commit 023af2d3b192f8ebf1bd4fe75a22a4adaedc1e05.

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

videoencoder: Release video frame when ->handle return ERROR or DROPPED

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

playsink: Properly mark pending blocked pads

When blocking input pads, we also need to properly set the appropriate
pending flag.

Without this, when switching stream types after initial configuration
(like going from Audio+Video to Audio+Video+Sub) playsink would never
wait for *all* input streams to be blocked (it would just wait for the
new input pad (text in this case) to be blocked).

Since the reconfiguration might introduce unlinking/relinking of elements,
we need to ensure that *ALL* input streams are blocked.

Failure to do so would result in having some input streams pushing data
to inactive elements (returning GST_FLOW_FLUSHING) or unlinked pads
(returning GST_FLOW_NOT_LINKED).

A later optimization could involve only blocking the input pads that
might be involved in reconfiguration. But better be safe than sorry for
now :)

gst-device-monitor: Use g_printerr instead of g_error

g_error is meant to be used for programmer errors (causes an abort),
not for expected runtime errors.

subtitleoverlay: replace gst_caps_can_intersect() with is_subset()

Subset check verifies also that all required fields are present
and is mostly commonly used when checking if an element accepts
a certain caps

playbin: use subset check instead of intersect

Elements usually require that all fields on their caps are present
on the fixed caps they receive. Using intersection won't verify it,
resort to using is_subset() checks.

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

audio-channel-mixer: round before truncating

Round the result before truncating for int channel mixing.

audio-converter: Avoid conversion when possible

When the input and output formats are the same and in a possible
intermediate format, avoid unpack and pack.
Never do passthrough channel mixing.
Only do dithering and noise shaping in S32 format

audio-channel-mixer: add more formats

Add support for float and int16 mixing
Remove in-place processing, this simplifies things as we won't be using it.
Don't do clipping for float audio formats