examples/snapshot: Does not need GTK but only gdk-pixbuf

audioencoder: also adjust sample count upon discont to avoid ts overflow

Only adjusting the base_ts might lead to a negative ts and as such integer
overflow into a huge timestamp which then propagates into the granulepos
and so on.  Instead, resync to incoming buffer timestamp using both base_ts
and sample count rather than only base_ts.

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

meson: fix a few meson warnings

examples: fix memory leaks in appsrc and appsrc2 examples

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

playbin: Make sure to ref_sink() any sinks before calling activate_sink()

It is forwarding messages to the playbin bus, thus forwarding messages
that contain a floating reference to the application. This generally
makes bindings unhappy, we must not leak floating references to them.

xmptag: Stop parsing GPS coordinate if sscanf() fails

CID 1139610

appsrc: fix doc typos

appsrc: Add out annotations to gst_app_src_get_latency()

audioconvert: Consider channel=1, channel-mask=0x0 as mono

channels=1 is always mono, having it 'unpositioned' does not make
sense.

This fixes pipeline such as:

  gst-validate-1.0 audiotestsrc ! audio/x-raw,channels=2,rate=44100,layout=interleaved ! audioconvert ! audioresample ! audio/x-raw, rate=44100, channels=1 ! avenc_mp2 ! fakesink

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

videotimecode: Add GST_VIDEO_TIME_CODE_INIT macro

videofilter: Unmap input frame if mapping output frame failed

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

videodecoder: Make sure we have an actually writable buffer when modifying metadata

avviddec keeps references to the buffers internally for example, in
which case we need to do a shallow copy of the buffer.

videometa: Don't crash if adding the timecode meta to a buffer failed

ssaparse: Fix buffer leak in error case

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

adder: comment and formatting cleanups

Log a few more details. Update method comments. Remove some extra blank lines.

videorate: flush remaining buffers on SEGMENT_DONE

Just as we do on EOS.

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

fft: don't generate g-i files for bindings

The g-i stuff for this helper lib was never usable from bindings
anyway and there are problems with the latest gobject-introspection,
so we might just as well remove the g-i integration entirely for
this lib.

decodebin3: Remove FIXME and do remove_input_stream() only for the corresponding parsebin

Do not remove other parsebin's input streams. It will cause unexpected
removal of any input streams in multi-parsebin use case.

Basically, the purpose of blocking buffers is similar to checking
no-more-pads of chain/group. That is, it gives hint to know the timing
to remove old (EOSed) streams of the parsebin and to add/reuse slots
for new input streams. But, that doesn't mean that we need to remove
other parsebin's EOSed stream. Each parsebin has most likely its
own streaming thread and therefore EOSed time can be much different.
(i.e., much early EOS of subtitle only parsebin)

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

parsebin: Ensure StreamType and Caps of GstStream object before exposing it

The final StreamType and Caps might not be set yet on GstStream at exposing the pads.

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

playbin3: Delay linking text output until video stream is shown

We are not sure that which stream's collection arrives first
when there are multiple parsebins such as adaptive streaming.

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

tests: appsrc: fix leaks in new unit test

encodebin: fix possible pad ref leak in error code path

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

playbin3: We only care about source pads going away

decodebin3: use lock macro where applicable

decodebin3: Protect fields related to streams handling with the SELECTION_LOCK

Fields related to stream handling (input_streams,
output_streams, slots, guint slot_id) where used totally unprotected
until know.

This lead to several races, especially playing back RTSP streams.

To protect those fields, the OBJECT_LOCK can not be used as we sometimes
need to be able to post message on the bus while holding it.

decodebin3 already has a lock to manage stream selection, and in the end
it makes sense to protect all the stream management fields with the same
lock which is why we reuse the SELECTION_LOCK here.

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

decodebin3: Protect dbin->collection usage

Use the selection lock to protect dbin->collection access

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

tests: adder: set all properties at once

tests: adder: add helper to create buffers

Keeps repeated code out of the test and syncs it with the audiomixer test.

meson: add translations

adder: fix docs typo

adder: document caps negotiation raciness and workaround

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

playback example: Prettify time reporting for big values

When dealing with streams/contents which have large duration, it is
more user-friendly to show more details in the high values (hours or days)
than in the microseconds.

This patch will use the following formatting schemes:
* Below 1hour   : MM:SS.SSS
* Below 24hours : HHhMMmSSs
* Above         : DDdHHhMMm

decodebin3: Push EOS to output stream if they are all drained

decodebin3 checks input streams and pushes EOS if all input streams
are EOSed. If not, fake EOS is pushed to the corresponding slot.

When adaptivedemux is used with multi-track configuration,
adaptivedemux never ever push EOS to non-selected track
because streaming thread for the slot stops with not-linked flow return.
So, decodebin3 should generate EOS itself to finish playback.

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

urisourcebin: Push EOS if slot is still eos state

linked input of slot can be old input, so urisourcebin should check
eos state to figure out whether it's new one or not.
If not, urisourcebin never ever forwards EOS to downstream at the end
of presentation, because the old input is still there without removal

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

decodebin3: Don't send duplicated stream-start event

group-id in stream-start event might be updated in
parse_chain_output_probe (). This cause duplicated stream-start
twice with identical stream-id and seq-num, but only group-id is
different. Although there is no change, stream-start event will
be followed by the first buffer.

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

adder: sync some fixes from the audiomixer test

typefind: Detect JPEG2000 codestreams

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

meson: find python3 via python3 module

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

convertframe: Fix leak in case of vcrop is disabled

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

test-appsrc: Test state when blocked in caps Event

In GStreamer 1.12 and older, the GstBaseSrc live lock used to be held while
create() virtual function was called. As appsrc pushes serialized event in
that virtual function, we ended up with some deadlock while setting the
state to NULL. This test simulates this situation.

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

uridecodebin: aggregate topology messages

This makes it possible for GstDiscoverer to work with sources that
have multiple source pads and hence will trigger the creation of multiple
decodebin instances such as rtspsrc.

Based on the work of Vineeth TM <vineeth.tm@samsung.com>

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

textoverlay: ensure text buffer has writable metadata when modifying

textoverlay: adjust a valid text buffer duration to fall within segment

... as expected later on when end time is used to determine end running time.
Otherwise the latter is determined as NONE and the resulting text buffer is
then used indefinitely.

textoverlay: make debug statement more informative

encodebin: Simplify the are_raw_caps function

Remove unnecessary if statement in are_raw_caps function.
we can use result returned by gst_caps_can_intersect quite simple.

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

meson: Allow using glib as a subproject

meson: fix with-package-name option

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

subparse: ensure serialized sending of segment event at proper time

alsasrc: Handle newly added GstStateChange values

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

libs: Export boxed type copy/free functions for the remaining types

urisourcebin: Call do_async_done when source state change returns NO_PREROLL

Otherwise for RTSP streams for example, the pipeline will never go to
PLAYING as it will be missing an ASYNC_DONE message.

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

rawvideoparse: Fix missing VideoMeta

The base class is trying to align the processed data, but it endup
removing the GstVideoMeta. That caused wrong result. Instead, just copy
from the process function with the appropriate alignment.

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

urisourcebin: Use downloadbuffer element

And only set low-percent/high-percent if not using downloadbuffer, just
like in old uridecodebin. using the watermark based buffering causes
playback to hang never finish buffering with downloadbuffer.

encodebin: Don't try rate adjustment before the first buffer

With both audiorate and videorate, it seems more sensible to apply rate
adjustments after the first buffer appears. For example, with v4l2src,
there is often a small delay before the first video buffer turns up, and
this can cause a stuttery start because of videorate trying to ensure a
perfect stream.

encodebin: Don't set audiorate property before NULL check

meson: Do not use path separator in test names

Avoiding warnings like:

  "WARNING: Target "elements/audioamplify" has a path separator in its name."

navigation: Add some validation while sending key/mouse events

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

navigation: Add introspection annotations for some output parameters

The missing annotations prevented proper usage from introspected
bindings like python.

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

meson: use dep.get_pkgconfig_variable()

instead of calling pkg-config ourselves.

audioconvert: resize output buffer to correct size

If we are using a downstream bufferpool we need to set the size of the
buffer to our output size.

video: update orc generated files

Includes updates from:

  103d265 Fix RGBA and ABGR pack/unpack on big endian cpu

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

Fix RGBA and ABGR pack/unpack on big endian cpu

The pack and unpack functions for RGBA and ABGR only work for little
endian cpus. Add variants for big endian as well.

Revert "update def files"

This reverts commit 2e6dba811437e31af7e6071efe03b6e3751b2289.

update def files

Revert "decodebin2: Set a time limit on "upstream" multiqueues"

This reverts commit 07dc9ba0712c26be86f031fb6f77bee177cbb828. It causes
timeouts in validate because queues run full before prerolling.

decodebin2: Set a time limit on "upstream" multiqueues

Those multiqueue are the ones dealing with adaptive demuxers. They should
have a time limit set so that they don't end up buffering too much data.

They would previously be set with no limits at all, which would cause them
to grow indefinitely until downstream blocks.

gst-discoverer: use state changes instead of ASYNC_DONE.

And monitor no_more_pads.

With live sources such as rtsp, uridecodebin only creates its
child decodebins between PAUSED and PLAYING.

This means that the ASYNC_DONE it posts when getting NO_PREROLL
in its change_state method gets immediately propagated by the
GstBin parent class, as opposed to a situation where a
decodebin has been added to it already, and has posted ASYNC_START.

The proposed solution, instead of simply waiting for ASYNC_DONE,
and finishing prematurely in that case, waits for three conditions
to be true:

* the uridecodebin needs to have emitted no_more_pads
* its current state must be PAUSED if not live, PLAYING otherwise
* There must be no "pending subtitle pads", ie pads where we haven't
  received tags yet.

All these conditions are checked in the message handler, as we
post custom messages on it when we get subtitle tags or no_more_pads.

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

oggdemux: fix artifacts at chain boundaries

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

pango: Handle failure to multiply fractions

And set PAR back to the default value of 1:1

CID #1409851

oggdemux: fix clipping more samples than exist in the first packet

This can happen in Opus (and maybe other codecs ?), and would cause
failure to play.

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

videotestsrc: Protect videoinfo with object lock

The videoinfo is set in the streaming thread, but can be read by any
thread from the various queries, so protect it with the object lock.

pbutils: fix stand-alone version header include

Include gst headers for GST_EXPORT.

meson: don't need config.h.meson any longer

textoverlay: optionally scale text to ensure proper display text aspect ratio

... by prescaling with an inverse aspect scaling as applied by video scaling

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