meson: only check for c++ compiler once

meson: make C++ compiler optional

It's only used to check our headers are C++ clean and
for the Qt example.

videodecoder: Don't queue frames in TRICKMODE_KEY_UNITS

When the input is TRICKMODE_KEY_UNITS, we expect to only receive keyframes
which we want to decode/push immediately. Therefore don't queue them.

If upstream didn't send just keyframes (which is the ideal situation), two
different things can happen:
1) Either the subclass checks the segment flags and properly configures
 the decoder implementation to only decode/output keyframes,
2) Or the subclass really decodes and outputs everything, in which case
 the reverse frames will end up arriving "late" downstream (and will
 be dropped). If upstream did properly send GOP in reverse order, we
 still end up just showing keyframes (but at the overhead of decoding
 everything).

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

videorate: stop copying buffers in drop-only mode

gst_video_rate_flush_prev() ensures that the pushed buffer is writable
by calling gst_buffer_make_writable() on videorate->prevbuf.

In drop-only mode we always push buffers directly when they are received
from GstBaseTransform (gst_video_rate_transform_ip()) and do not keep them
around. GstBaseTransform already ensures that those buffers are
writable so there is no need to do it twice.

This change saves us from copying buffers in drop-only mode as we no longer
calls gst_buffer_make_writable() with a buffer having a refcount of 2
(one ref owned by GstBaseTransform and one in videorate->prevbuf).

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

videorate: factor out gst_video_rate_push_buffer()

No semantic change, just factor out this function from
gst_video_rate_flush_prev().

I'm about to use it to change the 'drop-only' code path.

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

meson: add options to set package name and origin

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

Fix unit test for videodecoder now outputting mono multiview by default

videodecoder: Output mono multiview caps if none specified

Always put multiview-caps onto the output caps, assuming
mono if we've got no other information. It's still easy for
downstream elements to override using a capssetter or event
probe if desired.

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

discoverer: Clean up more fields to decide if parent/child streams are equivalent

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

Revert "discoverer: Consider parent/child streams the same if they have caps with the same name"

This reverts commit 478b7a8eb49d285c3ff0b73e1fe2929b9418be91.

video/mpeg,systemstream=true / false distinguishes between container and
elementary stream.

discoverer: Consider parent/child streams the same if they have caps with the same name

Child streams could have more accurate width/height or various other
information added. If they have the same name, they are likely to be the
same streams.

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

urisourcebin: Unref query with gst_query_unref()

Not gst_object_unref().

alsa: Add mapping for PCM F32/F64 formats

Fix enables float pcm formats for both the alsasrc and alsasink

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

gst: Clear floating flag in constructor of all GstObject subclasses that are not owned by any parent

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

audioclock: Sink the reference in the constructor

This is now needed as GstClock does not do that internally anymore,
because that broke bindings.

And mark the function correctly as (transfer full), which it already was
before.

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

allocators: Annotate constructors with (transfer floating)

GstAllocator is a GstObject and as such uses floating references.

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

alsasink: Accept MPEG 1 layer 3 version 2.5

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

audioringbuffer: Accept MPEG 1 layer 3 version 2.5

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

Remove plugin specific static build option

Static and dynamic plugins now have the same interface. The standard
--enable-static/--enable-shared toggle are sufficient.

video: mark symbols explicitly for export with GST_EXPORT

tag: mark symbols explicitly for export with GST_EXPORT

sdp: mark symbols explicitly for export with GST_EXPORT

rtsp: mark symbols explicitly for export with GST_EXPORT

rtp: mark symbols explicitly for export with GST_EXPORT

riff: mark symbols explicitly for export with GST_EXPORT

pbutils: mark symbols explicitly for export with GST_EXPORT

app: mark symbols explicitly for export with GST_EXPORT

fft: mark symbols explicitly for export with GST_EXPORT

audio: mark symbols explicitly for export with GST_EXPORT

allocators: mark symbols explicitly for export with GST_EXPORT

gst-libs: Remove empty ext subdir

rtpbasedepayload: fix class description in docs

examples: fix element leak in volume example

playback-test: guard against crash on failure to create pipeline

It can happen when giving incorrect parameters (ie, a URI when
expecting a pipeline, etc)

g-i: no need to load registry in g-i scanner

Back to development

Release 1.12.0

Update .po files

decodebin2: fix use after free from demuxer flush pad probe

In some cases, we could get a flush-stop event after the chain structure
containing the demuxer was freed.

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

Revert "videorate: In reverse playback mode, don't output the first buffer with ts=segment.stop"

This reverts commit e6736e992576d174707af339aeb908722492ebca.

This one is for after 1.12.0 and shouldn't have been merged yet.

libs: Check if meta transform_func is NULL before using it

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

videorate: In reverse playback mode, don't output the first buffer with ts=segment.stop

Instead go backwards before segment.stop based on the framerate or the
next buffers end timestamp. Otherwise the first buffer will usually be
dropped because outside the segment.

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

textoverlay: Removing duplicated command.

gst-launch-1.0 was duplicated on textoverlay example.

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

playbin: minor docs fix

The "frame" property is no more, it's "sample" these days.

Release 1.11.91

Update .po files

po: Update translations

Automatic update of common submodule

From 60aeef6 to 48a5d85

video: fix typo in GstVideoAlignment doc

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

audio: resampler: fix typos in docs

meson: Pass --c-include accordingly to GIR builds

No need for newlines in debug log statements

tools: gst-device-monitor: Print gst-launch example

Print a gst-launch-1.0 line that could get to this device,
useful as we don't have other ways to see what it does exactly.
This may not work if the create element has configurations other than
properties.

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

.gitignore: ignore all generated enumtypes files

meson: dist more meson build files

meson: install gstappsink.h

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

volume: Store volume as a double for consistency with the property type

And the potential increased precision, which shouldn't matter much here.

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

Automatic update of common submodule

From 39ac2f5 to 60aeef6

Revert "tag: Fix enum nicks for backwards compatibility"

This reverts commit 595b29519ab6850811ed2e68b75e29a2240a5432.

Bindings are not using the nicks but the actual enum names.

app/tag: Fix build with srcdir!=builddir

tag: Fix enum nicks for backwards compatibility

The enum values got the wrong name at some point, something to fix for 2.0.

app/tag: Add missing sources/headers to the GIR build

meson: Define missing variable

tag: Generate GLib enums/flags with glib-mkenums

app: Generate GLib enums with glib-mkenums

encoding-profile: It's (transfer none), not (transfer-none)

audio: Generate audiobasesink/src and audiocdsrc GLib enums automatically

And ensure that GstAudioBaseSrcSlaveMethod's re-timestamp stays
re-timestamp and doesn't become retimestamp.

timecode: Fix invalid drop-frame timecode right before a new second

The previous fix was only working for non-drop-frame timecodes.

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

Release 1.11.90

Update .po files

po: Update translations

sdp/media: caps_from_media() don't modify the input media

Performing a gst_sdp_media_get_caps_from_media() would result in
changing fields in the GstSDPMedia violating the const tag in the
function declaration.

Before there would be a line with a=rtpmap:96 VP8/90000
after, that attribute would only contain a=rtpmap:96

Fix by performing modifications on duplicated strings instead of on
the internal values.

Also add a simple test for checking that the representation doesn't
change by a gst_sdp_media_get_caps_from_media()

sdp: add g_return*_if_fail assertions on invalid inputs

Prevents some programming errors and invalid modifications.

sdp/message: fix segfault copying NULL in the boxed copy impl

Allows passing NULL as a value to g_object_set and as signal
parameters without crashing.

video-converter: fix scaler leak

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

encodebin: fix list leak when requesting a new pad

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

meson: Use get_pkgconfig_variable instead of calling pkg-config ourself

It is avalaible in meson 0.36 which is now are requirement

urisourcebin: Set removed flag when removing buffering msg

Forgot to set the flag, resulting in a missed check for
sending 100% in some cases. Spotted by Coverity.

CID: 1403263

docs: Fix broken URL in reference manual

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

rawvideoparse: fix assert on large width/height in caps

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

typefind: Expand the search range for HLS detection

HLS files can have arbitrary extra tags in them, and
those can be quite long lines. We need to search
further than 256 bytes sometimes just to get past the
first few lines of the file. Make the limit 4KB,
which matches a typical input block size and should
hopefully cover every crazy input.

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

meson: Fix build failure from previous commit

Mistake while rebasing

meson: Specify pluginsdir setting the environment

And do not specify the separator as this is OS
dependent and will be handled properly by default

meson: Reuse plugins_install_dir for building env

meson: Fix plugin path when running test

The path was only adding the build root. We need to also add the
prefix for the case we work with installed setup. As the search is
recursive, I had to remove any subdirectory to the already present build
root.

rawvideoparse: Rename frame-stride in to frame-size

The term stride is confusing here, since the stride is always use
to signal the pixel row size of an image (including padding). Also
a frame may have a single stride, which adds to the confusion. This
patch uses frame-size, which simply indicate the frame size in the
case the images have some padding in between.

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

rawvideoparse: Use GstValueArray for strides and offsets

This allow using those property through gst-launch-1.0. This type
gained a deserilizer recently. The syntax is: <val1, val2, ...>.
Note that we also use the type int instead of uint to avoid having
to cast when specifying the values. The deserilizers assume
int by default.

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

streamsynchronizer: every stream need keep their own send_gap_event flag

When a clip has video audio and subtitle, if need send gap event
to audio and subtitle, we should make sure all has been sent, so
need every stream keep one send_gap_event.

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

urisourcebin: Mention which queue is being linked in debug output

examples: jsseek: fix typo

Spotted by Yaakov Selkowitz

examples: jsseek: update for removal of mad plugin

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

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