tools: gst-play: remove unnecessary variable

The second assignment of sret is never used. We can remove the first assignment
and use the value directly instead.

id3v2frames: Fix compiler warnings

id3v2frames.c:951:20: error: unused variable 'utf16enc' [-Werror,-Wunused-const-variable]
static const gchar utf16enc[] = "UTF-16";
                   ^
id3v2frames.c:952:20: error: unused variable 'utf16leenc' [-Werror,-Wunused-const-variable]
static const gchar utf16leenc[] = "UTF-16LE";
                   ^
id3v2frames.c:953:20: error: unused variable 'utf16beenc' [-Werror,-Wunused-const-variable]
static const gchar utf16beenc[] = "UTF-16BE";
                   ^

part-stereo-multiview-video: Add a section of open design questions

video-format: Fix minor docs typo

videotestsrc: Document the solid-color pattern

playback: Document GST_PLAY_FLAG_SOFT_COLORBALANCE

video: Make gst_buffer_get_video_meta() a real function, Return lowest id

Instead of returning the first video meta found on a buffer, return the
one with the lowest id (which is usually the same thing, except on
multi-view buffers)

discoverer: don't crash on unknown info types when deserializing

Handle unknown info types when deserializing instead of
dereferencing NULL pointers.

Coverity CID 1302394

sdp: prevent the sdp message parser from reading past the end of the buffer

Otherwise, a malformed SDP message could crash the application,
or even maliciously gather data from the memory located after
this buffer...

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

tests: add test for videorate caps renegotiation after a framerate has been calculated and added to caps

The original 0/1 framerate must still be allowed to be configured
on the upstream side of videorate, otherwise future caps renegotiation
is going to fail.

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

videorate: update the caps framerate only in the GST_PAD_SINK transform_caps direction

When a stream has a variable framerate, videorate calculates it and
forces it on the output caps. However, the code in _transform_caps()
currently also does that if the transform is going in the opposite
direction (GST_PAD_SRC), so during a renegotiation it tries to force
upstream to use the calculated framerate and it fails.

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

playsink: use queue to avoid lock in audiotee audio branches

This part of pipeline is:

tee name=t ! visualizationbin ! streamsynchronizer name=s
t. ! s.

streamsynchronizer might block and it could starve the visualization
branch of the pipeline when it is enabled.

The visualization bin has queues internally but the other branch
that links the audiotee directly to the synchronizer is vulnerable
to block. Adding a queue between "t. ! s." fixes deadlocks.

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

basetextoverlay: make deltax and deltay properties controllable

This will be more useful once we have absolute direct
control bindings.

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

oggdemux: fix chain leak

Don't leak the building_chain when destroying.

Fix leaks with the validate.http.playback.reverse_playback.vorbis_theora_1_ogg
scenario.

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

tag: id3v2: fix parsing of UTF-16 text on systems with crippled iconv

Use g_utf16_to_utf8() instead of the more generic g_convert(), so
that we can extract text in UTF-16 format even on embedded systems
with crippled iconv support.

This code path is exercised by the id3demux test_unsync_v23
check in gst-plugins-good.

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

Add new generated rtp enum files to .gitignore

tools: gst-play: keep configured playback rate and trick mode when seeking

Instead of resetting rate to 1.0

po: update for new translatable strings

tools: gst-play: mark more strings for translation

tools: gst-play: add keyboard shortcut help

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

tests: add back videoscale unit test

Has been removed in 835422b2 as part of porting
things over to the new videoscale API.

tools: gst-play: enable interative mode by default

And change --interactive option to --no-interactive.

rtp: Clean G-I files on make clean too

rtp: Add builddir to the include path for gobject-introspection

And also add missing headers/sources

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

win32: Update exports

rtp: Add GstRTPProfile enum

rtsp: Add FIXME 2.0 comment about GstRTSPTransport being an enum instead of flags

rtsp: Use glib-mkenums to generate GstRTSPProfile and GstRTSPLowerTrans GTypes

Revert "oggdemux: Prevent seeks when _SCHEDULING_FLAG_SEQUENTIAL is set"

This reverts commit 76647f2710d718e27f207b005956b7dba72c2d19.

Avoiding pull mode activation is a feature regression, and
demuxers should always use pull mode where that is possible,
e.g. if there's an upstream queue2 with a ring buffer or
a download buffer.

This patch made reverse playback no longer possible over http.

If the goal is to minimise seeks, then that can still be done
by making the demuxer behave differently in pull mode if
the SEQUENTIAL flag is set. If there are bugs, like the demuxer
needlessly scanning the entire file on start-up in pull mode,
then those should be fixed instead.

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

win32: update .def file for new API

rtsp: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

gio: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

tcp: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

gstdiscoverer: Add since annotation.

Forgot to add the since annotation to the
GstDiscovererSerializeFlags in the previous commit.

discoverer: Add serialization methods.

[API] gst_discoverer_info_to_variant
[API] gst_discoverer_info_from_variant
[API] GstDiscovererSerializeFlags

+ Serializes as a GVariant
+ Adds a test
+ Does not serialize potential GstToc (s)

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

rtpbasepayload: Try harder to reuse previously configured caps values and give more preference to anything set as properties

This affects the pt, ssrc, seqnum-offset and timestamp-offset properties. If
they were set from a property, or we configured caps before, we try to use
that value for them. Even if the first structure of the downstream caps
specifies a different value, we check if the value is supported by other
structures.
Only if all this fails, we use the values given by downstream in the first
structure, i.e. if no properties were set and these are the first caps we
negotiate or downstream does not support our values.

By doing this we ensure that we don't spuriously change ssrcs or other fields
in the middle of the stream (and also consider property values more). Ssrc
changes would currently happen after sending an RTX packet (thus creating a
new internal source inside the rtpsession), and then renegotiating the
payloader (which then gets the RTX ssrc from rtpsession).

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

docs: a random set of trivial fixes for the library docs

Warnings down to 35, unused symbols doen to 112.

docs: add fdmemory to docs

docs: a random set of trivial fixes for the library docs

All those where super straight forward from the warnings gtkdoc prints. It kind
of makes sense to apply them before the list of warnings is >100 and people
complain that gtkdoc is noisy.

mikey: fix a bunch of doc warnings

Rename header/source mismatch of parameters. Update the exposed API in
sections.txt.

Revert "doc: Workaround gtkdoc issue"

This reverts commit df7ef3c35d34352257a28307c07d4673f239452e.

This is fixed by the gtk-doc 1.23 release.

appsrc: optimise caps changing when previously-set caps have not taken effect yet

Only negotiate/change caps once when setting caps twice and
the first-set caps have not been used yet.

Based on patch by Eunhae Choi.

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

xvimagesink: fix pool leak

During set caps when config fails, the referenced newpool
is not unref ed.

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

playbin: check the flags before set again

check the previous flags of playsink to avoid the reconfigure of playsink repeatedly

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

doc: Workaround gtkdoc issue

With gtkdoc 1.22, the XML generator fails when a itemizedlist is
followed by a refsect2. Workaround the issue by wrapping the refsect2
into para.

playback: use the new gst_object api

Use gst_object_has_as_anchestor instead of the now deprecated _has_ancestor.

docs: fix up example pipeline

docs: update element example pipelines

- gst-launch -> gst-launch-1.0
- use autoaudiosink and audiovideosink more often
- review pipeline examples and descriptions

video: Update win32 exports for new libgstvideo API

videoconvert: Expose some properties from the videoconverter API

Expose chroma resampler, alpha mode, alpha value, chroma mode, matrix mode,
gamma mode and primaries mode from the videoconverter API.

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

video-converter: Change some implicit string enums to real enums

GST_VIDEO_CONVERTER_OPT_ALPHA_MODE, GST_VIDEO_CONVERTER_OPT_CHROMA_MODE,
GST_VIDEO_CONVERTER_OPT_MATRIX_MODE, GST_VIDEO_CONVERTER_OPT_GAMMA_MODE and
GST_VIDEO_CONVERTER_OPT_PRIMARIES_MODE were G_TYPE_STRING with only a few valid
options. Changed those to real enums.

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

audiodecoder: Also negotiate with downstream if needed before handling a GAP event

videodecoder: Also negotiate with downstream if needed before handling a GAP event

videodecoder: Try to be smarter when clipping buffers without duration/framerate to the segment

2 second frame duration is rather unlikely... but if we don't clip
away buffers that far before the segment we can cause the pipeline to
lockup. This can happen if audio is properly clipped, and thus the
audio sink does not preroll yet but the video sink prerolls because
we already outputted a buffer here... and then queues run full.

In the worst case we will clip one buffer too many here now if no
framerate is given, no buffer duration is given and the actual
framerate is less than 0.5fps.

Fixes seeking on HLS/DASH streams, when seeking into the middle of
fragments and having no framerate/buffer duration.

xvimagesink: fix navigation event leak when early returning

Create the event *after* the early return check so it's not leaked.

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

xvimagesink: fix navigation event leak when not handled

gst_navigation_message_new_event() is *not* consuming the event so we should
always drop our extra reference.

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

navigation: fix structure leak if subclass doesn't implement send_event()

The send_event() implementation is supposed to consume @structure.

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

streamsynchronizer: Don't override segment.base from upstream with 0

Upstream might want to use it to properly map timestamps to running/stream
times, if we just override it with 0 synchronization will be just wrong.

For this we remove some old 0.10 code related to segment accumulation, and
remove some more code that is useless now, and accumulate the group start time
(aka segment.base offset) manually now.

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

rtpbasedepayload: Add some debug output

video: add NV61 format support

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

docs: add new video API to docs

video: check colorimetry and chroma_site equality in gst_video_info_is_equal()

Add VideoInfo accessors for colorimetry and chroma_site and use them
when checking the equality of two GstVideoInfo

video-color: Add gst_video_colorimetry_is_equal()

Add a function for comparing the equality of 2 colorimetry
structures.

oggdemux: remove unused code

These lines have done nothing for about 10 years.

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

pbutils: Use more strict profile checking for hevc

Use the profile_idc value to set the profile string in caps.
Don't use compatibility flags for this purpose.

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

video-converter: Remove unused macro

Remove unused macro GET_TMP_LINE

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

gst-play: add some more key navigation mappings

And don't feed multi-character key descriptors to the
event handler, it won't be what it expects.

navigation: sprinkle some since markers and add new API to .def file

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

tools: Add mouse/keyboard handling from messages

Allows the user to control playback with the window in focus

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

xvimagesink: Post unhandled navigation events on the bus

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

video: Add a new "event" navigation message type

This will be useful for elements that wish to post unhandled navigation
events on the bus to give the application a chance to do something with
it

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

video-info: expose InterlaceMode conversion to/from string

Expose the methods used to convert a GstVideoInterlaceMode to and
from a string.

Rename property enums from ARG_ to PROP_

Property enum items should be named PROP_ for consistency and readability.

videoconvert: Keep colorimetry and chroma-site fields if passthrough

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

audio: Change the remaining "samples" in the ::delay() vfunc docs to "frames"

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

tests: tcp: remove SOCK_CLOEXEC which causes build problems on OS/X

It's not needed here.

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

audio: The delay vfunc returns the number of frames, not samples

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

Remove obsolete Android build cruft

This is not needed any longer.

tests: typefindfunctions: add test for UTF-16 MSS manifest typefinding

typefinding: don't read more data than needed in MSS typefinder

typefinding: detect MSS manifests without using g_convert()

Embedded systems often have limited charset conversion
functionality, so don't rely on g_convert() (i.e. iconv)
for UTF-16 to UTF-8 conversions, we can easily enough do
that ourselves by converting to native endianness and
then using GLib's helper functions.

audiovisualizer: fix the license from GPL to LGPL

This was a copy'n'paste buf in the initial commit done by myself.

xmptag: fix invalid reads in GST_DEBUG statement

Don't try to print a string that is not NUL-terminated. This
log line does not really seem useful so let's just drop it.

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

remove unused enum items PROP_LAST

This were probably added to the enums due to cargo cult programming and are
unused. Removing them.

audiodecoder: Add sink and src query virtual method

API: GstAudioDecoderClass::src_query()
API: GstAudioDecoderClass::sink_query()

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

tests: define GST_CHECK_TEST_ENVIRONMENT_BEACON

Make sure the test environment is set up.

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

configure: bump automake requirement to 1.14 and autoconf to 2.69

This is only required for builds from git, people can still
build tarballs if they only have older autotools.

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

Update .gitignore

video-converter: n_lines member should be a guint not a boolean

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

oggdemux: fix event leaks

gst_event_replace() takes its own reference on the event so we should drop
ours after creating and storing an event using it.

This fix leaks which can be reproduced using the
validate.http.media_check.vorbis_theora_1_ogg scenario.

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

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

Remove LICENSE_readme

It's completely outdated and just confusing, better if people are
forced to look at the actual code in question than trusting this file.

video-scaler: fix YUY2 scaling some more

Take into account the different steps between Y and UV when calculating
the line size for vertical resampling or else we might not resample
enough pixels and leave bad lines.

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

video-scaler: scale enough pixels in YUY2 (and friends) mode

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

tests: rtpbasedepayload: fix crash in test when passing varargs

Need to pass 64 bits where 64 bits are expected.

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

video-converter: Remove unused variables

Remove unused variables n_taps, max_taps in setup_scale()

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

video: add missing part of documentation text

discoverer: fix GstToc leak when parsing toc messages

gst_message_parse_toc() returns a reffed GstToc which is owned by the
GstDiscovererInfo. But we have to make sure we unref its previous value before
setting the new one.

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

win32: Update defs for new API

allocators: make GstFdAllocator non-abstract

Make the GstFdAllocator non-abstract because it is perfectly possible
to make memory from a generic fd. Mark the memory as simply "fd".