tags: Add new _USER_RATING tag

Adds a new tag for user favorite media rating.
User rating informs how much (from 0 to 100) a user
'likes' a media.
Having an percent uint range for this is easy to map into other scales,
like some players that allow users to attribute 'stars' to its
media.

API: GST_TAG_USER_RATING

Fixes #520697

queue2: add more info in the buffering query

Add the estimated download time and estimated time left to the buffering query
results along with the estimated download and playback speed.

queue2: implement flushing in download buffering

Maintain a separate variable to control src and sink flowreturn values so that
we can unlock the src part without shutting down the sink part.
Add flushing for upstream pull based elements that unblocks our getrange
function. This implements seeking when blocking for more data.
Add some arbitrary threshold before attempting a seek. Add a FIXME for this
because we need to find a sensible threshold based on the input rate.

Automatic update of common submodule

From 55cd514 to c1d07dd

Remove unused code

OPT_CFLAGS was never AC_SUBST()'ed so it wasn't used. And the last time
it was touched was in 2005.

build: fix make distcheck

The change from GST_ALL_CFLAGS to GST_OPTION_CFLAGS dropped the includes,
putting them back fixes make distcheck.

Fix tests after set_element_details() deprecation

git-update: Fix and restructure logic

Deprecated gst_element_class_set_details()

Use gst_element_class_set_details_simple() instead. If you want to
convert automatically, here's a script:

for file in `git grep -l GstElementDetails`; do
  sed -i -n -r '
    1h
    1!H
    $ {
      g
      s/((\/\*[^\n]*\*\/)?\n)*[^\n]*GstElementDetails .* =\s*GST_ELEMENT_DETAILS\s*\((\"[^\"]*\",\s*\"[^\"]*\",\s*\"[^\"]*\",\s*(\"[^\"]*\"\s*)*)\);\n*(.*)gst_element_class_set_details \(([^,]*),\s*[^)]*\)/\n\n\5gst_element_class_set_details_simple (\6, \3)/
      s/((\/\*[^\n]*\*\/)?\n)*[^\n]*GstElementDetails .* =\s*\{\s*(\"[^\"]*\",\s*\"[^\"]*\",\s*\"[^\"]*\",\s*(\"[^\"]*\"\s*)*)\};\n*(.*)gst_element_class_set_details \(([^,]*),\s*[^)]*\)/\n\n\5gst_element_class_set_details_simple (\6, \3)/
      p
    }' $file
  ~/gst/gstreamer/tools/gst-indent $file
done

queue2: Fix uninitialized variable compiler warning

tests: remove unused CHECK_CFLAGS and CHECK_LIBS from Makefile.am

structure: add mapping for (uint) to allow deserialisation of unsigned integers

Unsigned ints are used in taglists, would be nice to be able to
deserialise them, esp. in connection with the taginject API.

queue2: implement seeking in download mode

When in download mode and the requested offset is too far away, attempt to do a
seek request to fetch the data.
Keep track of all downloaded parts and merge ranges when needed.

Fixes #600877

git-update.sh: use autogen.sh instead of autoregen.sh on fresh repositories.

Fixes #613593.

gettext: build fixes: #if -> #ifdef

parse-launch: make delayed set recursive

Right now deleyed set would only try for first set of children. We need to keep
trying to support arbitrary deep hierarchies (like in playbin2 with auto*sinks).
Also GstBin would need to actualy emit the child-added/removed signal as it
implements the iface. Fixes #613215.

pkgconfig: Use @LIBM@ instead of -lm

pkgconfig: add back support for builddir != srcdir case in uninstalled setup

Attempt to add back support for builddir != srcdir. Use absolute paths
instead of relative paths based on pcfiledir this time to make things
clearer - there's not really any need for uninstalled trees to be
relocatable without re-running configure.

basetransform: Implement QoS message posting

And some more for bug #322947

basesrc: catch, parse and store QoS event values

Catch, parse and store the QoS values from QoS events for later use.

basesink: Implement QoS message posting in basesink

Post QoS messages when frames are dropped.
This goes a little further towards resolving bug #322947

message: improve docs a little

message: add QoS message to inform apps of lost data

This has been implemented as per part-qos.txt and partially addresses
bug #322947

test: Remove needless cast

GstValueArray.name is const now

Revert "Add srcdir to includes for out-of-source builds"

I don't know how this ever worked, as it seems to put -I./..
and -I./../libs verbatim into the includes, at least with
current autotools versions.

This reverts commit 279a80ff27ba4c90f52981e89e710eb1181bc201.

docs: avoid confusion between events and messages

controller: Mark property_name in GstValueArray as const

This won't and should not be changed from any API

docs: fix typo

docs: add missing "Since: 0.10.26" marker for gst_data_queue_new_full()

docs: fix since tag for gst_structure_id_has_field_typed()

gst_structure_id_has_field_typed was added in 0.10.26, not 0.10.16.
Apparent typo in commit f9e3b72f when the API was added.

docs: fix Since markers for gst_byte_writer_put_float*()

As the headers were broken in 0.10.26 the functions weren't really
usable back then, so we should advertise them as being there only
since 0.10.27.

Spotted by Mart Raudsepp.

gst: use #if GLIB_CHECK_VERSION instead of #ifndef GLIBMACRO

for better greppability at the time we bump GLib version requirements.

typefind: deactivate pad if we can't get length or it's a length of zero.

Fixes issues when re-using typefind after a file of length zero.

basetransform: Accept non-fixed caps suggestions

When doing pad_allocs, use non-fixed caps suggestions and
try to fixate them before using. This makes possible to
have suggested buffer size with 0 in basetransform just
to signal upstream a renegotiation is needed

Fixes #576234
Fixes #609046

docs: merge QoS message fields

There was already a section about QoS messages that is now merged with the new
information.

pwg: mention how to build after using the project stamp

task: snprintf needs to include "stdio.h"

docs: update QOS docs to include QOS messages

Add some docs about the values needed for a QoS message and some use
cases.

See #322947

task: use bionic/libc friendly arguments to prctl

prctl is supposed to take 5 arguments. It used to work with 2 arguments on some
versions of libc because it is defined as a varags function there.

See #611911

task: update docs.

taglist: Work around gtk-doc problem

bytewriter: Use correct gtk-doc workaround

utils: Handle iterator resyncs in gst_pad_proxy_setcaps()

Fixes bug #612881.

task: configure the object name as thread name

When we have prctl available, use it to set the configured object name as the
thread name for better debugging.

Based on patch by Robert Swain.

pad: set a good name on the task of the pad

Use the element:pad names to configure a good name for the pad task.

task: retab

logging: remove extra newline

bus: turn g_return_if_fail into g_assert.

This either must never happen (which makes sense in this case) and thus should
use assert() or we should use a traditional if (poll_data->message) return;
to avoid differnet behaviour of intenal api when compiling with
G_DISABLE_CHECKS.

caps: move the check to the public api.

This avoids creating empty caps and destroying them in the case of an error. We
also avoid double checking in other code path where we call the internal api.

caps: this is internal API where we need to ensure !NULL higher up

gst: this is an internal function where we already ensure !NULL when calling

debugutils: fix case of pad flag

Due to a typo the code was always showing the flag as 's' (lower case).
Fixes #611075

debug: add pretty printer for events

Adder is using GST_PTR_FORMAT for events already, so we might actualy
implement this and print out some useful info.

Add some 0.11 FIXMEs for GstPluginInitFunc

See 8fe63000de31bb2bcf346d59230dea06117997cd for why having a TRUE/FALSE
return value is a bad idea.
I've scanned a few plugins and they generally get it wrong and aren't
unloadable when they return FALSE.

po: update for new strings

gstinfo: fix compilation error if HAVE_REGISTER_PRINTF_SPECIFIER is undefined

Use #if HAVE_FOO instead of #ifdef HAVE_FOO.

Fixes #612733.

plugins: Do not ever unload a plugin after calling into it

This is what can happen in a plugin_init function:
- An element based on GstBaseSink is registered
- Other elements fail to register
- The plugin_init function returns FALSE

Now if this the plugin is the first plugin to link against
libgstbase.so, it will have caused libgstbase.so to be loaded and static
strings from that library will have been added to gobject while
registering GstBaseSink.

So unloading the plugin will cause those strings to go stale and the
next plugin using GstBaseSink will crash. So we must not unload modules
after calling into them ever.

https://bugzilla.redhat.com/show_bug.cgi?id=572800

interpolationcontrolsource: Don't pass NULL to the GSequence API

git-update: Fix error return value and make the script exit on errors

Newer versions of BASH (4.x?) seem to dislike using -1 for a return. Even
though it's documented as being signed, BASH complains about it, so use
255 instead.

Automatic update of common submodule

From e272f71 to 55cd514

gst: Use G_VALUE_COLLECT_INIT if available

This brings total call speedups between 5% and 25%.
gst_caps_set_simple_valist: +5%
gst_structure_set_valist: + 10%
gst_structure_id_set_valist: +25%
gst_tag_list_add_valist: +5%

Measured using valgrind when run over the discovery of 200 media files.

Fixes #610256

win32: Fix build failures of tests

tags: Adds new geo location tags

Adds new tags GST_TAG_GEO_LOCATION_COUNTRY,
GST_TAG_GEO_LOCATION_CITY and GST_TAG_GEO_LOCATION_SUBLOCATION.

API: GST_TAG_GEO_LOCATION_COUNTRY
API: GST_TAG_GEO_LOCATION_CITY
API: GST_TAG_GEO_LOCATION_SUBLOCATION

Fixes #612410

win32: Add prototype for DllMain()

Update .po files

gstreamer: remove unneeded casts

G_PARAM_SPEC_VALUE_TYPE does an expensive type check, whereas the
value_type field is a public field, so we can just use it directly.

Remove -Winline flag again

It triggers for a lot of GStreamer API (even though those triggers are
wrong most of the time).
I missed it because it only triggers with -O2, and I was using -O0.

Automatic update of common submodule

From df8a7c8 to e272f71

Add a bunch more warning flags to configure

None of these flags cause warnings anymore, so no fixes necessary.
The flags are:
 -Wformat-nonliteral
 -Wformat-security
 -Wold-style-definition
 -Wcast-align
 -Winline
 -Winit-self
 -Wmissing-include-dirs
 -Waddress
 -Waggregate-return
 -Wno-multichar
 -Wnested-externs

Fixes for -Wmissing-declarations -Wmissing-prototypes

Also adds those flags to the configure warning flags

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

buffer: fix printf format

Use %u to print unsigned integers.

tests: cast the arg. to fix the build with new compiler opts.

This is ugly. I am not sure if we really want to have such casts all over the
place.

i18n: define dummy ngettext if i18n is disabled.

We cannot blindly use gettext function and not define them when not using gettext.

i18n: fix the build with i18n disabled.

Don't include gettext.h if !ENABLE_NLS.

buffer: allow configurable memory alignment. Fixes #596832

The alignment guaranteed by malloc is not always sufficient. E.g. vector
instructions or hardware subsystems want specifically aligned buffers. The
attached patch will use posix_memalign if available to allocate buffers.
The desired alignment can be set when running configure using the new
--with-buffer-alignment option.

Automatic update of common submodule

From 9720a7d to df8a7c8

Merge branch 'work'

Fixes for -Wwrite-strings

This changes some APIs in compatible ways:
- Some functions now take "const char *" arguments, not "char *"
- Some structs now have "conts char *" members, not "char *"
The changes may cause warnings when compiling with the right warning
flags. You've been warned.

Also adds -Wwrite-strings as a warning flag in configure.ac.

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

Fixes -Wundef warnings

... and adds that flag to configure.ac

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

benchmarks: Remove unneeded g_thread_exit()

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

Fixes for -Wold-style-definition

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

Make code safe for -Wredundant-decls

Adds that warning to configure.ac

Includes a tiny change of the GST_BOILERPLATE_FULL() macro:
The get_type() function is no longer declared before being defined.

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

Automatic update of common submodule

From 0b6e072 to 9720a7d

Make sure generated code doesn't run with -Werror

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

Update to common/ changes to ERROR_CFLAGS

Revert "registry: remove unused function"

Turns out  the function is not unused, but was in an #ifdef WIN32
section.
Whoops.

This reverts commit 57d5db424c68ab5a61f33ce36ce0179eb30251ac.

Automatic update of common submodule

From 7cc5eb4 to 0b6e072

registry: remove unused function

Actually, there was two functions with the same name, but only one was
used.

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

remove unused gst_element_default_error()

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

utils: Use mulq instead of mul as some assemblers can't guess the size of the operands

Fixes bug #612370.

Automatic update of common submodule

From 7aa65b5 to 7cc5eb4

Automatic update of common submodule

From 44ecce7 to 7aa65b5

build: Make some more rules silent if requested

configure: Use automake 1.11 silent rules instead of shave if available

This makes sure that we use something that is still maintained and
also brings back libtool 1.5 support.

lfocontrolsource: Optimize get_value_array()

Don't convert from GValue to the actual type for every single
value.

interpolationcontrolsource: Optimize get_value_array()

This makes it >10x faster if more than a single value is requested
by not searching in the GSequence for every value and converting
the value from GValue to the real value type.

controller: Add benchmark for getting a value array of the control points

Fix typos in documentation

caps: Fail when fractions are followed by random text

Previous code treated "1/1yourmom" the same as "1/1" and "1wimsmom" the
same as "1". Now the code is stricter and will fail to convert a
fraction when followed by garbage text.

Back to development