basesink: Make sure we never drop the preroll buffer

This is cosmetic as 'late' should never be set during preroll (in pause).
Though code may evolve in the future, so this is good for preventing
potential bugs.

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

basesink: Don't nest prepare/render calls

When the first buffer arrives, we endup calling:

  ->prepare()
    ->prepare()
    ->preroll()
  ->render()

This will likely confuse any element using this method. With this patch,
we ensure the preroll take place before the first render prepare() is
called. This will result in:

  ->prepare()
  ->preroll()
  ->prepare()
  ->render()

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

basesink: fix typo in documentation

Small typo in the documentatin of gst_base_sink_set_drop_out_of_segment().
Fixing it.

basesink: Fix gst_base_sink_set_drop_out_of_segment() documentation

Also silences a GI warning.

clock_gettime.c: Use __APPLE__ instead of __MACH__

Hurd also defines __MACH__, but it does not have mach_absolute_time. Use
the more strict __APPLE__ instead.

Has also been sent upstream: https://github.com/libcheck/check/pull/65

build: Fix AM_CONDITIONAL check for clock_gettime

It was always evaluating to false, so clock_gettime.c was always being
included into libcheck. This breaks building on Hurd and causes us to
always override clock_gettime() even when it is available.

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

gst: Also include the gstdynamictypefactory.c source file in the build

gst: Install gstdynamictypefactory.h header file

Revert "baseparse: fix draining with less data than min frame size available"

This reverts commit 2e278aeb7128e8732f5324ab8c8b22a47950c80a.

Some parsers, specifically audio parsers, assume to get all remaining
data on EOS and just pass them onwards. While the idea here is correct,
we will probably need a property for this on baseparse for parsers to
opt-in.

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

parse-launch: Support linking all pads with new operator

Introduce a new operator ':' - e.g. element1 ':' element2

For example, 'uridecodebin : encodebin' -
if the encodebin has multiple profiles compatible with the
decodebin, multiple links will be created.

With '!' , after one delayed link is successfully done, the
pad-added callback is disconnected.

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

Add new basesink API to exports and Since markers

Add Since markers to the new basesink API to drop
out-of-segment buffers, and add them to the
win32 exports

dynamic types: Implement dynamic types in the registry

Implement GstDynamicTypeFactory as a new registry feature.

GstDynamicTypeFactory provides a way of registering a GType
into the registry, such that it will be registered as a dynamic
type when the registry is loaded, and then automatically loaded
if the type is needed during caps parsing.

This allows using non-core types in pad templates, by loading a
registry feature to create the GType on the fly.

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

fakesink: Add property to not drop out-of-segment buffers

Implement handling in basesink to not unconditionally discard
out-of-segment buffers and expose it as a new property on fakesink
(not unconditionally in all basesink based sinks).

The property defaults to FALSE.

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

value: Update GstValue table size for GValueArray

funnel: Always push all sticky events whenever we forward a serialized event

Otherwise downstream will have an inconsistent set of sticky events at this
point, e.g. when a TAG event is pushed and downstream wants to relate it to
the stream by looking at the current STREAM_START event.

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

tee: Set GST_PAD_FLAG_PROXY_CAPS before forwarding sticky events

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

baseparse: fix draining with less data than min frame size available

baseparse would pass whatever is left in the adapter to the
subclass when draining, even if it's less than the minimum
frame size required. This is bogus, baseparse should just
discard that data then. The original intention of that code
seems to have been that if we have more data available than
the minimum required we should pass all of the data available
and not just the minimum required, which does make sense, so
we'll continue to do that in the case that more data is available.

Fixes assertions in rawvideoparse on EOS after not-negotiated with
fakesrc sizetype=random ! queue ! rawvideoparse format=rgb ! appsink caps=video/x-raw,format=I420

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

info: Replace %p and %r in GST_DEBUG_FILE

It's useful to be able to set a name pattern for GST_DEBUG_FILE so that
the same environment variable can be used for multiple processes and
still write to different files. Especially useful if these processes
run simultaneously.

%p: Replaced with PID
%r: Replaced with random number

%p is obviously useful. %r is useful when for instance running two
processes with same PID but in different containers.

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

registry: set env GST_REGISTRY_DISABLE=yes to disable registry

If GST_REGISTRY_DISABLE=yes the registry is disabled similar to
compile time switch GST_DISABLE_REGISTRY.

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

gstvalue: Make GValueArray serializable

For instance very useful for logging GValueArray with GST_PTR_FORMAT

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

element: Allow multiple conversion specifiers for request pads

This allows pad template names like "src_%u_%u", but it does not allow
multiple specifiers of string type %s as that would lead to ambiguities.

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

pad: add no-reconfigure link check

Enable it to prevent sending reconfigure when linking elements.

Useful for autoplugging when we know caps or bufferpools shouldn't change
to save doing caps renegotiation to end up with the same final scenario.

The no-reconfigure is not a proper check, it is a flag. It is implemented
as a GstPadLinkCheck to avoid creating another gst_pad_link variant.

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

meson: update version

docs: remove app dev manual and plugin writer's guide

They have moved to gst-docs and will be maintained there in future.

Back to development

Release 1.10.0

Update .po files

po: Update translations

Update .po files

meson: Add an option to explicitly disable gtk-doc

Similar to how Autotools provides an option, default is 'enabled'.

Revert "collectpads: Assume PTS is equal DTS if PTS is missing"

This reverts commit 9b0d42ceecb3198399d7e05e3d5f080a7ca27ca9.

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

allocator: Mark registered allocator for leak

gstmessage.h: Avoid gcc bit shift overflow compiler warning

Avoids bit shift overflow warning with gcc6.

https://bugzilla.gnome.org/show_bug.cgi?id=767882 (glib)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71803 (gcc)
https://bugzilla.gnome.org/show_bug.cgi?id=767883

parse: fix erroneous use of _("")

Fixes xgettext warnings when doing 'make update-po':
gst/parse/grammar.y:217: warning: Empty msgid.  It is reserved by GNU gettext:
                                  gettext("") returns the header entry with
                                  meta information, not the empty string.

docs: add index for API new in 1.10

docs: add index for API new in 1.8

adapter: Fix mix-up between DTS and PTS

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

Revert "queue: Fix race when calculating cur_level.time"

This reverts commit d03bd547809f849405a3f706920091d9b03147b0.

It breaks the unit test, although it ensures that only correct values
are used for calculations. Needs to be fixed up.

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

Revert "meson: Use the new `pic` argument on static libs"

This reverts commit a5752240a178c2c651ed10167025fad8b9c4e7bd.

pic was added after 0.35 and will be present in 0.36 (meson
documentation was wrong).

meson: Use the new `pic` argument on static libs

We depend on meson 0.35 which makes it simpler to handle

Removes a meson warning

queue: Fix race when calculating cur_level.time

On the first buffer, it's possible that sink_segment is set but
src_segment has not been set yet. If this is the case, we should not
calculate cur_level.time since sink_segment.position may be large and
src_segment.position default is 0, with the resulting diff being larger
than max-size-time, causing the queue to start leaking (if
leaky=downstream).

One potential consequence of this is that the segment event may be
stored on the srcpad before the caps event is pushed downstream, causing
a g_warning ("Sticky event misordering, got 'segment' before 'caps'").

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

element: use g_strcmp0 in set_context

It's NULL-safe while the libc implementation may not be.

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

element: check for invalid gstcontext's being provided to set_context

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

build: Apply XCode 8 workaround for iOS too

clock_gettime was also added for iOS 10.0, so don't use it if we're
targetting an older version. That would've caused the symbol to not be
found at runtime on older devices.

meson: Don't use c_std=c99

Just use the default c_std used by the compiler. With GCC on Linux this
is gnu89.

Tons of errors related to time.h, signal.h, etc when using c99:

FAILED: libs/gst/check/libcheck/check@sta/check_run.c.o
cc  '-Ilibs/gst/check/libcheck/check@sta' '-fdiagnostics-color=always' '-I../libs/gst/check/libcheck' '-Ilibs/gst/check/libcheck' '-I.' '-I../.' '-Ilibs/gst/check/libcheck/..' '-I../libs/gst/check/libcheck/..' '-pipe' '-Wall' '-Winvalid-pch' '-std=c99' '-DHAVE_CONFIG_H' '-fPIC' '-O2' '-g' '-fPIC' '-MMD' '-MQ' 'libs/gst/check/libcheck/check@sta/check_run.c.o' '-MF' 'libs/gst/check/libcheck/check@sta/check_run.c.o.d' -o 'libs/gst/check/libcheck/check@sta/check_run.c.o' -c ../libs/gst/check/libcheck/check_run.c
In file included from ../libs/gst/check/libcheck/check_run.c:21:0:
../libs/gst/check/libcheck/libcompat.h:167:18: warning: ‘struct itimerspec’ declared inside parameter list will not be visible outside of this definition or declaration
     const struct itimerspec *new_value, struct itimerspec *old_value);
                  ^~~~~~~~~~
../libs/gst/check/libcheck/check_run.c:98:25: error: array type has incomplete element type ‘struct sigaction’
 static struct sigaction old_action[3];
                         ^~~~~~~~~~
../libs/gst/check/libcheck/check_run.c:99:25: error: array type has incomplete element type ‘struct sigaction’
 static struct sigaction new_action[3];
                         ^~~~~~~~~~

[...]

ninja: build stopped: subcommand failed.

The change was originally made because gnu99 was causing issues on OS X.

stats: Fix warning

meson: Make use of new environment object and set plugin path to builddir

- Properly set where to find gst-plugin-scanner
- Use GST_LOADING_WHITELIST so that only core plugins are used

Bump meson requirement to 0.35

controlbinding: Store object in a thread-safe GWeakRef

g_object_weak_ref() is not thread-safe.

argbcontrolbinding: gst_object_replace() is transfer none

directcontrolbinding: Clarify in the documentation what the difference between absolute and not is

controlsource: Remove misleading sentence from the documentation

control sources are not required to return values between 0.0 and 1.0.
This is completely up to the control binding that is used.

object: Fix broken sentence structure in docs

tracers: fix structure leak

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

streams: Extend GstStreamType documentation

Users shouldn't assume it will be a single value since it's a flag.

build: Fix clock_gettime check with XCode 8

With XCode 8, clock_gettime will be incorrectly detected as being
available regardless of what OS X version we're targetting because the
symbol is available in the .tbd library as a weak symbol.
See: https://github.com/Homebrew/homebrew-core/issues/3727#issue-170086273

It's only starting from macOS 10.12 that clock_gettime is actually
available, so we can unconditionally disable it when targetting older
versions. We cannot simply do AC_CHECK_FUNCS with -Wl,-no_weak_imports
because the autoconf check does its own prototype declaration that
doesn't trigger that compiler flag.

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

bin: Add a new GST_BIN_FLAG_STREAMS_AWARE flag

This flag is to indicate to child elements that they can add and
remove pads at any point in time without re-adding existing ones.

Elements should post before-hand a GST_MESSAGE_STREAM_COLLECTION

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

docs: paths in env vars are separated by semicolons on windows

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

docs: fix GST_LEAKS_TRACER_SIG doc

The documentation wasn't mentioning the SIGUSR2 signal.

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

bin: Fix iterator resync'ing

When we get GST_ITERATOR_RESYNC, we need to call gst_iterator_resync()
otherwise we will always get GST_ITERATOR_RESYNC (and that loop would
run forever).

queue2: Implement gst_event_full_func handling

Same as we do for queue

tracer/rusage: fix format string args

The format string contains a process id, but we did not provice one. This
caused us to log garbage since all args got shifted.

message: Fix typo in gst_message_new_progress() docs

test -> text

meson: Setup pre-commit hooks when configuring

meson: update version

Release 1.9.90

Update .po files

po: Update translations

meson: tracers: signal availability of libunwind and backtrace() to code

Not setting cdata here on purpose because of .. complications.

meson: Build tracers

meson: Fix gtkdoc using new meson features

tests: remove unused valgrind stuff

Code was also checking the wrong define anyway.

tests: parse-launch: looks clean nowadays, so re-enable for valgrind

Also, the valgrind bits weren't hooked up properly anyway,
checking the wrong define.

meson: remove incorrect and unneeded check for ptrdiff_t

Need to include stddef.h for it, so this would've worked:
if cc.has_type('ptrdiff_t', prefix: '#include <stddef.h>')

meson: fix internal printf for %ll format modifier on 32-bit systems

gst/gstprintf unit test would fail on 32-bit x86 with:
gstprintf.c:83:printf_I32_I64:0: 'str' (64-bit x value = b5a6978f) is not equal to '"64-bit x value = f1e2d3c4b5a6978f"'

Revert "meson: Force gstenum_h to be built when using gst_dep"

This reverts commit cfc565e2d88a8e7d656b68c5c2a1b7acb08cdb7f.

The commit was redundant since gst_gen_sources already contains
gstenum_h. We're still investigating why some people are still seeing
a racy build failure.

meson: Force gstenum_h to be built when using gst_dep

This forces gstenumtypes.h to be built whenever something uses gst_dep
as a subproject dependency. This is needed since gst/gst.h includes
gstenumtypes.h

Closes https://github.com/mesonbuild/meson/issues/714 which is not
actually a Meson bug.

bin: Add forgotten "git commit --amend" for last commit

Need to cast away the const as g_queue_foreach() takes a non-const GQueue*

bin: When copying the sort iterator, also copy its internal queue

Otherwise both iterators share the same references, the second one
usually resulting in a crash when being freed.

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

queue2: Fix watermark test

This carries over code for a similar test from multiqueue to ensure full
control over the dataflow while testing. (The previous attempt was racy
since the fill level changed without any thread sync with the test code.)

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

queue2: Update buffering if its enabled and low/high watermarks are changed

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

message: Fix documentation for gst_message_new_duration()

Seems like there was some documentation left over from when this was
gst_message_new_duration().

basesink: Remove unused fields and always use the buffer timestamp difference for calculating the QoS proportion

The buffer timestamps are only hints and more often than not have
nothing to do with reality.

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

gstconfig: Use __declspec when built with MinGW and linking with MSVC

Earlier we were only using __declspec(dllexport/import) when we were
built with MSVC because when built with MinGW and linking with MinGW we
don't need it (and we get linker errors because of it).

However, when we're built with MinGW and someone wants to link to us
with MSVC, we still need the prototypes to have __declspec(dllimport)
since MSVC cannot do auto-import like GCC can.

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

bin: Also don't *unset* element flags if they're in the suppressed flags

Otherwise our bin might lose various flags that were explicitly set on
it at arbitrary times.

tests: gstbin: add tests for suppressed flags

Some simple tests to make sure it keeps working

bin: Add setter and getter to suppress element flags

Suppress-flags is for preventing propagation of child element's specific
flag when it is added to the bin.

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

Automatic update of common submodule

From b18d820 to f980fd9

gst: Ensure gst_value is initialised before gst_tag

Otherwise GST_TYPE_FRACTION will work correctly in tags.

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

element: Reset the stop position to NONE in seek_simple()

When using seek_simple() in combination with other kinds of seeks, this
becomes problematic. seek_simple() does not reset the stop position to
GST_CLOCK_TIME_NONE but keeps whatever a previous seek did. So for example
when doing a seek_simple() after a rate=-1 seek, we would usually get
assertions that start>stop (and stop being the old stop from the rate=1 seek).

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

Automatic update of common submodule

From f49c55e to b18d820

meson:tests: Bump timeout to 3 minutes

Basically we already have each test with a 20sec timeout,
and testsuite can last more than the default 30secs from
meson. 3 minutes is another arbitrary timeout but should
be good enough.

basesink: Use the average durations based on timestamps for the QoS proportion when doing trickmodes

The durations of the buffers are (usually) assuming that no frames are being
dropped and are just the durations coming from the stream. However if we do
trickmodes, frames are being dropped regularly especially if only key units
are supposed to be played.

Fixes completely bogus QoS proportion values in the above case.

meson: Fix building with meson 0.34

meson: Allow others to build GIR files when using GStreamer as subproject

And add a way to disable the introspection and bump version to 1.9.2

gstconfig.h.in: Add another version of the SH4 #define and S390x

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

config: newer gcc doesn't know __ppc__ and __ppc64__ anymore

__ppc__ and __ppc64__ are non-standard defines, we should use
__powerpc__ and __powerpc64__ instead because newer gcc doesn't know
them anymore.

tools: gst-inspect: add * for pointer signal arguments where needed

Print GObject argument properly with pointer marker:

  "client-added" :  void user_function (GstElement* object,
                                        GObject* arg0,
                                        gpointer user_data);

instead of

  "client-added" :  void user_function (GstElement* object,
                                        GObject arg0,
                                        gpointer user_data);

for gst-inspect-1.0 tcpserversink.

tests: don't build misc subdir if both examples and benchmarks have been disabled

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

config: support System z

Back to development