platform/upstream/gstreamer.git
8 years agoRelease 1.7.91
Sebastian Dröge [Tue, 15 Mar 2016 09:56:10 +0000 (11:56 +0200)]
Release 1.7.91

8 years agoUpdate .po files
Sebastian Dröge [Tue, 15 Mar 2016 09:44:03 +0000 (11:44 +0200)]
Update .po files

8 years agopo: Update translations
Sebastian Dröge [Tue, 15 Mar 2016 09:39:42 +0000 (11:39 +0200)]
po: Update translations

8 years agotypefind: Store caps on the pad before emitting have-type but send it downstream...
Sebastian Dröge [Fri, 11 Mar 2016 12:17:13 +0000 (14:17 +0200)]
typefind: Store caps on the pad before emitting have-type but send it downstream only in the default signal handler

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

8 years agobaseparse: Recheck after pre_push_frame() if there are tags pending
Sebastian Dröge [Sun, 13 Mar 2016 08:33:53 +0000 (10:33 +0200)]
baseparse: Recheck after pre_push_frame() if there are tags pending

Many parsers are storing tags only in pre_push_frame(), if we wouldn't check
afterwards we would push buffers before those tags and a lot of code assumes that
tags are available before preroll.

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

8 years agoconcat: Fix comment typo
Carlos Rafael Giani [Mon, 14 Mar 2016 10:15:07 +0000 (11:15 +0100)]
concat: Fix comment typo

8 years agoRevert "typefind: Store caps on the pad before emitting have-type but send it downstr...
Sebastian Dröge [Sat, 12 Mar 2016 10:56:28 +0000 (12:56 +0200)]
Revert "typefind: Store caps on the pad before emitting have-type but send it downstream only in the default signal handler"

This reverts commit 0835c3d6569dde0ec9e5524436367c7678cc4a4a.

It causes deadlocks in decodebin, which currently would deadlock if the caps
are already on the pad in have-type and are forwarded while copying the sticky
events (while holding the decodebin lock)... as that might cause the next
element to expose pads, which then calls back into decodebin and takes the
decodebin lock.

This needs some more thoughts.

8 years agotypefind: Store caps on the pad before emitting have-type but send it downstream...
Sebastian Dröge [Fri, 11 Mar 2016 12:17:13 +0000 (14:17 +0200)]
typefind: Store caps on the pad before emitting have-type but send it downstream only in the default signal handler

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

8 years agodocs: Flesh out element and object macro accessor docs a bit
Carlos Rafael Giani [Thu, 10 Mar 2016 09:35:40 +0000 (10:35 +0100)]
docs: Flesh out element and object macro accessor docs a bit

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

8 years agonetclientclock: Remove some obsolete code that can cause warnings
Sebastian Dröge [Wed, 9 Mar 2016 14:06:58 +0000 (16:06 +0200)]
netclientclock: Remove some obsolete code that can cause warnings

8 years agonetclientclock: Don't reset calibration of internal clock whenever a new netclient...
Sebastian Dröge [Wed, 9 Mar 2016 11:44:24 +0000 (13:44 +0200)]
netclientclock: Don't reset calibration of internal clock whenever a new netclient clock is created

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

8 years agogstbuffer: fix GstParentBufferMeta GType name
Víctor Manuel Jáquez Leal [Fri, 4 Mar 2016 17:23:18 +0000 (18:23 +0100)]
gstbuffer: fix GstParentBufferMeta GType name

The alias define GST_TYPE_PARENT_BUFFER_META_API_TYPE is wrong and
breaks the usage of gst_buffer_get_parent_buffer_meta().

This patch fixes the GType alias and make another alias to keep the API
compatibility guarded by GST_DISABLE_DEPRECATED.

Also added a unit test.

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

8 years agotracerrecord: Remove useless NULL check and add assertion for making assumptions...
Sebastian Dröge [Wed, 2 Mar 2016 08:37:09 +0000 (10:37 +0200)]
tracerrecord: Remove useless NULL check and add assertion for making assumptions explicit

gst_structure_new_empty() is not returning NULL in any valid scenarios,
checking for NULL here is useless. Especially because we would dereference any
NULL right after the NULL check again.
CID 1352037.

We previously check if the string ends on .class, as such strrchr() should
return something non-NULL. Add an assertion for that.
CID 1349642.

8 years agoelement: minor docs fix
Tim-Philipp Müller [Tue, 1 Mar 2016 19:50:26 +0000 (19:50 +0000)]
element: minor docs fix

Make gtk-doc happy.

8 years agoRelease 1.7.90
Sebastian Dröge [Tue, 1 Mar 2016 16:14:03 +0000 (18:14 +0200)]
Release 1.7.90

8 years agopo: Update translations
Sebastian Dröge [Tue, 1 Mar 2016 14:52:41 +0000 (16:52 +0200)]
po: Update translations

8 years agoRevert "bus: change GstBusSource to hold a weak ref to GstBus"
Sebastian Dröge [Mon, 29 Feb 2016 21:33:03 +0000 (23:33 +0200)]
Revert "bus: change GstBusSource to hold a weak ref to GstBus"

This reverts commit 894c67e642c0f858b5b18097fa7c995bf3cc50c1.

8 years agoRevert "bus: Make sure to remove the GPollFD from the GSources when destroying the...
Sebastian Dröge [Mon, 29 Feb 2016 21:32:58 +0000 (23:32 +0200)]
Revert "bus: Make sure to remove the GPollFD from the GSources when destroying the bus"

This reverts commit 05700a7082c145057ccc0be763067bcc263239eb.

8 years agoelement: Remove GST_STATE_LOCK_FULL() / UNLOCK_FULL()
Sebastian Dröge [Mon, 29 Feb 2016 15:06:36 +0000 (17:06 +0200)]
element: Remove GST_STATE_LOCK_FULL() / UNLOCK_FULL()

There is no corresponding API for that in GLib and nobody could've ever used
these macros without compiler errors anyway.

8 years agobus: Make sure to remove the GPollFD from the GSources when destroying the bus
Sebastian Dröge [Mon, 29 Feb 2016 08:01:50 +0000 (10:01 +0200)]
bus: Make sure to remove the GPollFD from the GSources when destroying the bus

Otherwise the GSource can look into our already destroyed bus where the
GPollFD is stored.

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

8 years agotests: ghostpad: Fix memory leaks
Vineeth TM [Mon, 29 Feb 2016 02:06:50 +0000 (11:06 +0900)]
tests: ghostpad: Fix memory leaks

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

8 years agotaglist: add guard to check writability when removing tags from a taglist
Tim-Philipp Müller [Sun, 28 Feb 2016 13:59:48 +0000 (13:59 +0000)]
taglist: add guard to check writability when removing tags from a taglist

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

8 years agoelements: use new gst_element_class_add_static_pad_template()
Tim-Philipp Müller [Sat, 27 Feb 2016 15:36:28 +0000 (15:36 +0000)]
elements: use new gst_element_class_add_static_pad_template()

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

8 years agoelement: add gst_element_class_add_static_pad_template()
Tim-Philipp Müller [Sat, 27 Feb 2016 15:28:49 +0000 (15:28 +0000)]
element: add gst_element_class_add_static_pad_template()

Pretty much every single element does

  gst_element_class_add_pad_template (element_class,
      gst_static_pad_template_get (&some_templ));

which is both confusing and unnecessary. We might just
as well add a function to do that in one step.

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

8 years agotypefind: fix indentation
Tim-Philipp Müller [Sat, 27 Feb 2016 15:32:19 +0000 (15:32 +0000)]
typefind: fix indentation

8 years agoAutomatic update of common submodule
Sebastian Dröge [Fri, 26 Feb 2016 10:40:55 +0000 (12:40 +0200)]
Automatic update of common submodule

From b64f03f to 6f2d209

8 years agobus: Prevent gst_bus_add_watch_full_unlocked from a segfault if priv->poll == NULL
James Stevenson [Thu, 25 Feb 2016 22:36:14 +0000 (22:36 +0000)]
bus: Prevent gst_bus_add_watch_full_unlocked from a segfault if priv->poll == NULL

This happens if the process runs out of file descriptors. Better print
a critical warning instead of just crashing.

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

8 years agobus: change GstBusSource to hold a weak ref to GstBus
Thiago Santos [Wed, 24 Feb 2016 13:56:24 +0000 (10:56 -0300)]
bus: change GstBusSource to hold a weak ref to GstBus

When holding a regular ref it will cause the GstBus to never
reach 0 references and it won't be destroyed unless the application
explicitly calls gst_bus_remove_signal_watch().

Switching to weakref will allow the GstBus to be destroyed.
The application is still responsible for destroying the
GSource.

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

8 years agoidentity: Add a " " after pts: in the silent=false output
Sebastian Dröge [Thu, 25 Feb 2016 12:11:34 +0000 (14:11 +0200)]
identity: Add a " " after pts: in the silent=false output

8 years agomanual: Fix examples to check for gst_buffer_map return values
Edward Hervey [Wed, 16 Apr 2014 09:42:18 +0000 (11:42 +0200)]
manual: Fix examples to check for gst_buffer_map return values

Otherwise people reading the manual will expect it to always
succeed :)

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

8 years agogstcheck: Check return value of gst_buffer_map
Edward Hervey [Wed, 16 Apr 2014 09:40:46 +0000 (11:40 +0200)]
gstcheck: Check return value of gst_buffer_map

We can't check contents if we don't have access to it

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

8 years agoplugins: Check return values of gst_buffer_map()
Edward Hervey [Wed, 16 Apr 2014 09:39:15 +0000 (11:39 +0200)]
plugins: Check return values of gst_buffer_map()

They can fail for various reasons.

For non-fatal cases (such as the dump feature of identiy and fakesink),
we just silently skip it.

For other cases post an error message.

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

8 years agobuffer: Check return value of gst_memory_map()
Edward Hervey [Tue, 23 Feb 2016 16:23:43 +0000 (17:23 +0100)]
buffer: Check return value of gst_memory_map()

Only do memory operations if the memory was succesfully map'ed

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

8 years agotaglist: add GST_TAG_CONDUCTOR
Adam Miartus [Thu, 10 Dec 2015 14:32:27 +0000 (15:32 +0100)]
taglist: add GST_TAG_CONDUCTOR

This is useful for metadata which explicitely distinguishes
between artist/composer and conductor.

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

8 years agoevent: add some more documentation on stream-id
Mark Nauwelaerts [Mon, 22 Feb 2016 13:09:56 +0000 (14:09 +0100)]
event: add some more documentation on stream-id

... where it might end up being used for.

8 years agotracer: Initialize GstTracer _priv_tracers and quarks unconditionnally
Thibault Saunier [Fri, 22 Jan 2016 10:25:30 +0000 (11:25 +0100)]
tracer: Initialize GstTracer _priv_tracers and quarks unconditionnally

Some people might use tracer hooks even if GST_TRACER_PLUGINS is not
set.

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

8 years agodocs: manual: remove dead link from integration page
Tim-Philipp Müller [Sat, 20 Feb 2016 10:18:06 +0000 (10:18 +0000)]
docs: manual: remove dead link from integration page

8 years agodocs: manual: fix formatting
Tim-Philipp Müller [Sat, 20 Feb 2016 10:13:38 +0000 (10:13 +0000)]
docs: manual: fix formatting

advanced-dataaccess.xml:1210: element listitem: validity error : Element
listitem content does not follow the DTD, expecting (...),
got (para CDATA para )
      </listitem>

8 years agoscripts: check for git in create-uninstalled-setup.sh as well
Tim-Philipp Müller [Sat, 20 Feb 2016 00:55:30 +0000 (00:55 +0000)]
scripts: check for git in create-uninstalled-setup.sh as well

8 years agoWhenever we include windows.h, also define WIN32_LEAN_AND_MEAN
Nirbheek Chauhan [Fri, 19 Feb 2016 14:56:26 +0000 (20:26 +0530)]
Whenever we include windows.h, also define WIN32_LEAN_AND_MEAN

This reduces the number of symbols and code pulled in drastically

8 years agoprintf: On MSVC, also define HAVE_STDINT_H_WITH_UINTMAX
Nirbheek Chauhan [Sat, 13 Feb 2016 01:23:24 +0000 (06:53 +0530)]
printf: On MSVC, also define HAVE_STDINT_H_WITH_UINTMAX

MSVC provides stdint.h but not inttypes.h, and we need to include stdint.h to
get intmax_t

8 years agoptpclock: Only include unistd.h if found
Nirbheek Chauhan [Sat, 13 Feb 2016 01:12:06 +0000 (06:42 +0530)]
ptpclock: Only include unistd.h if found

unistd.h is not provided by the  Microsoft Visual C++ compiler. It instead
provides the necessary defines through io.h

8 years agoplugin: Only check for S_IFBLK if it is defined
Nirbheek Chauhan [Sat, 13 Feb 2016 00:49:52 +0000 (06:19 +0530)]
plugin: Only check for S_IFBLK if it is defined

Windows does not define S_IFBLK since it doesn't have block devices

8 years agowin32: update README and remove outdated build cruft
Tim-Philipp Müller [Fri, 19 Feb 2016 20:17:02 +0000 (20:17 +0000)]
win32: update README and remove outdated build cruft

This hasn't been touched for generations, doesn't work,
and is just causing confusion. We also don't want to
maintain these files manually.

8 years agomanual: Explain what happens if upstream elements are removed from the pipeline witho...
George Yunaev [Fri, 19 Feb 2016 08:43:00 +0000 (08:43 +0000)]
manual: Explain what happens if upstream elements are removed from the pipeline without draining them first

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

8 years agotests: fix indentation
Tim-Philipp Müller [Fri, 19 Feb 2016 14:41:55 +0000 (14:41 +0000)]
tests: fix indentation

8 years agoBack to development
Sebastian Dröge [Fri, 19 Feb 2016 10:38:21 +0000 (12:38 +0200)]
Back to development

8 years agoRelease 1.7.2
Sebastian Dröge [Fri, 19 Feb 2016 09:47:52 +0000 (11:47 +0200)]
Release 1.7.2

8 years agopo: Update translations
Sebastian Dröge [Fri, 19 Feb 2016 08:29:40 +0000 (10:29 +0200)]
po: Update translations

8 years agouninstalled.pc: add support for non libtool build systems
Julien Isorce [Thu, 18 Feb 2016 14:20:17 +0000 (14:20 +0000)]
uninstalled.pc: add support for non libtool build systems

Currently the .la path is provided which requires to use libtool as
mentioned in the GStreamer manual section-helloworld-compilerun.html.
It is fine as long as the application is built using libtool.

So currently it is not possible to compile a GStreamer application
within gst-uninstalled with CMake or other build system different
than autotools.

This patch allows to do the following in gst-uninstalled env:
gcc test.c -o test $(pkg-config --cflags --libs gstreamer-1.0)
Previously it required to prepend libtool --mode=link

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

8 years agoRevert "pad: PULL probes are called without a buffer so don't require any of the...
Sebastian Dröge [Thu, 18 Feb 2016 09:43:22 +0000 (11:43 +0200)]
Revert "pad: PULL probes are called without a buffer so don't require any of the data flags to be set"

This reverts commit b89fa4786b3df6cb79f662c037dee74b3f7428d6.

The changes break various tests.

8 years agoRevert "pad: Add test for blocking pull probe"
Sebastian Dröge [Thu, 18 Feb 2016 09:43:04 +0000 (11:43 +0200)]
Revert "pad: Add test for blocking pull probe"

This reverts commit 17d30e944be0425ebb4fb6046f82d1f61701fe8f.

The PULL probe changes break various tests.

8 years agobuffer: Protect against failing to map input memory when merging memories
Sebastian Dröge [Thu, 18 Feb 2016 09:09:36 +0000 (11:09 +0200)]
buffer: Protect against failing to map input memory when merging memories

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

8 years agopad: Add test for blocking pull probe
Matej Knopp [Thu, 28 Jan 2016 15:22:17 +0000 (16:22 +0100)]
pad: Add test for blocking pull probe

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

8 years agopad: PULL probes are called without a buffer so don't require any of the data flags...
Sebastian Dröge [Wed, 17 Feb 2016 14:57:27 +0000 (16:57 +0200)]
pad: PULL probes are called without a buffer so don't require any of the data flags to be set

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

8 years agoRevert "element: Don't hold state lock all the time while sending an event"
Sebastian Dröge [Wed, 17 Feb 2016 14:41:02 +0000 (16:41 +0200)]
Revert "element: Don't hold state lock all the time while sending an event"

This reverts commit b427997119a2b6aacbeb550f729936f8b963e24b.

It breaks things that used to work before, even if the change by itself is
correct and the previous code is just working around deeper bugs in the async
state change code. Let's go back to what previously worked and then fix async
state changes in general.

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

8 years agoRevert "ghostpad: Do nothing in _internal_activate_push_default"
Edward Hervey [Wed, 17 Feb 2016 14:26:49 +0000 (15:26 +0100)]
Revert "ghostpad: Do nothing in _internal_activate_push_default"

That commit would break scheduling reconfiguration with ghostpads

This reverts commit ab55ad7eaad4fa2c0b16c789350e882cf70a27ed.

8 years agocheck: Add test for checking scheduling reconfiguration with ghostpads
Edward Hervey [Wed, 17 Feb 2016 14:25:08 +0000 (15:25 +0100)]
check: Add test for checking scheduling reconfiguration with ghostpads

Showcases the regression introduced by this commit:
Commit: ab55ad7eaad4fa2c0b16c789350e882cf70a27ed

Author: Stian Selnes <stian@pexip.com>
Date:   Wed Jan 27 13:20:23 2016 +0100

ghostpad: Do nothing in _internal_activate_push_default

8 years agoghostpad: add some tests for activation
Havard Graff [Wed, 17 Feb 2016 10:02:34 +0000 (11:02 +0100)]
ghostpad: add some tests for activation

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

8 years agoghostpad: Do nothing in _internal_activate_push_default
Stian Selnes [Wed, 27 Jan 2016 12:20:23 +0000 (13:20 +0100)]
ghostpad: Do nothing in _internal_activate_push_default

When calling gst_pad_activate_mode() on a ghostpad
gst_ghost_pad_activate_push_default() will be called. This will call
gst_pad_activate_mode() on the proxypad (which is internal of the
ghostpad), calling gst_ghost_pad_internal_activate_push_default(), which
again will call gst_pad_activate_mode() on the original ghostpad.

By simply returning TRUE in
gst_ghost_pad_internal_activate_push_default() the redundant call to
gst_pad_activate_mode() (for the same pad) is avoided.

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

8 years agoregistrychunks: remove unused macro
Thiago Santos [Tue, 16 Feb 2016 20:53:10 +0000 (17:53 -0300)]
registrychunks: remove unused macro

macro was added in 2011 and isn't used anymore

8 years agoRevert "queue2: add overrun signal"
Sebastian Dröge [Tue, 16 Feb 2016 17:11:59 +0000 (19:11 +0200)]
Revert "queue2: add overrun signal"

This reverts commit 8ae8b2723d0cf179a4f09b2f6c5f797e2d97034d.

It's not used anymore by anything and was considered a bad idea in general.

8 years agodocs: annotate C examples as such
Evan Nemerson [Thu, 5 Jun 2014 20:27:28 +0000 (13:27 -0700)]
docs: annotate C examples as such

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

8 years agobenchmark: tracerserialize: add missing return statement
Thiago Santos [Mon, 15 Feb 2016 14:13:40 +0000 (11:13 -0300)]
benchmark: tracerserialize: add missing return statement

tracerserialize.c:117:1: error: control reaches end of
non-void function [-Werror=return-type]

8 years agoprotection/harness/systemclock: move declaration out of for loop initialization
Thiago Santos [Mon, 15 Feb 2016 13:06:09 +0000 (10:06 -0300)]
protection/harness/systemclock: move declaration out of for loop initialization

C90 compilers complain about it
error: ‘for’ loop initial declarations are only allowed in C99 or C11 mode

Also run gst-indent on systemclock tests.

8 years agoharness: always set our test-clock on the harnessed element
Havard Graff [Wed, 27 Jan 2016 14:16:03 +0000 (15:16 +0100)]
harness: always set our test-clock on the harnessed element

The integration is already so tight, there is no reason to
not further formalize it!

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

8 years agoharness: fix compilation
Tim-Philipp Müller [Sat, 13 Feb 2016 16:10:27 +0000 (16:10 +0000)]
harness: fix compilation

Was supposed to be squashed with 336c7bb6

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

8 years agoharness: fix indentation
Tim-Philipp Müller [Sat, 13 Feb 2016 16:10:11 +0000 (16:10 +0000)]
harness: fix indentation

8 years agoharness: Fix MT issues when forwarding event/query to sink harness
Stian Selnes [Thu, 21 Jan 2016 12:33:15 +0000 (13:33 +0100)]
harness: Fix MT issues when forwarding event/query to sink harness

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

8 years agogst-uninstalled: add new -bad libraries audio, player and wayland to paths
Tim-Philipp Müller [Sat, 13 Feb 2016 10:04:42 +0000 (10:04 +0000)]
gst-uninstalled: add new -bad libraries audio, player and wayland to paths

And remove egl which no longer exists.

8 years agopushsrc: fix minor typos in header
Martin Kelly [Fri, 12 Feb 2016 19:57:55 +0000 (11:57 -0800)]
pushsrc: fix minor typos in header

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

8 years agoharness: Add event stress test functions with callback
Stian Selnes [Thu, 21 Jan 2016 12:28:23 +0000 (13:28 +0100)]
harness: Add event stress test functions with callback

Similar to the stress test functions for buffers that has a callback to
create the buffer to be pushed, it's useful to have functions that use a
callback to create the event to be pushed.

API: gst_harness_stress_push_event_with_cb_start()
API: gst_harness_stress_push_event_with_cb_start_full()
API: gst_harness_stress_send_upstream_event_with_cb_start()
API: gst_harness_stress_push_upstream_event_with_cb_start_full()

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

8 years agotestclock: add crank method
Havard Graff [Thu, 14 Jan 2016 20:54:42 +0000 (21:54 +0100)]
testclock: add crank method

And use it inside GstHarness

API: gst_test_clock_crank()

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

8 years agoharness: enable empty harness creation and refactor around this
Havard Graff [Wed, 9 Dec 2015 02:43:38 +0000 (13:43 +1100)]
harness: enable empty harness creation and refactor around this

Also make the testclock a member of the harness, allowing some
more interactions with the clock prior to adding elements.

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

8 years agocheck: fix unused parameter compiler warning
Stian Selnes [Fri, 12 Feb 2016 14:12:43 +0000 (15:12 +0100)]
check: fix unused parameter compiler warning

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

8 years agoharness: fix the race in blocking push mode
Mikhail Fludkov [Tue, 4 Aug 2015 15:09:35 +0000 (17:09 +0200)]
harness: fix the race in blocking push mode

Depending on when gst_harness_pull was called - before the buffer reached
gst_harness_chain or after we can get different behaviors of the test
with enabled blocking push mode. The fix makes the behavior always the
same. In pull function we get the buffer first, thus making sure
gst_harness_chain waits for the signal, and emitting the signal after.

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

8 years agocheck: Add tcase_skip_broken_loop_test
Stian Selnes [Thu, 4 Feb 2016 14:16:41 +0000 (15:16 +0100)]
check: Add tcase_skip_broken_loop_test

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

8 years agoharness: Fix docs for stress test functions
Stian Selnes [Thu, 21 Jan 2016 12:25:40 +0000 (13:25 +0100)]
harness: Fix docs for stress test functions

notify is not called per buffer, but when the thread is freed.

Comment about serialized events and OOB does not make sense for upstream
events.

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

8 years agoharness: Unset sink_forward_pad before tearing down sink_harness
Stian Selnes [Tue, 8 Dec 2015 13:18:21 +0000 (14:18 +0100)]
harness: Unset sink_forward_pad before tearing down sink_harness

Set the sink_forward_pad to NULL before tearing down sink_harness to
avoid that the harness tries to forward events/queries to it while it's
tearing down.

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

8 years agoharness: fix up docs to reference functions properly
Havard Graff [Tue, 29 Sep 2015 10:12:24 +0000 (12:12 +0200)]
harness: fix up docs to reference functions properly

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

8 years agobufferpool: pass acquire params to alloc_buffer
Víctor Manuel Jáquez Leal [Wed, 10 Feb 2016 13:01:54 +0000 (14:01 +0100)]
bufferpool: pass acquire params to alloc_buffer

When allocating a new buffer in the pool, both the do_alloc_buffer() and the
vmethod, alloc_buffer(), receive the parameter GstBufferPoolAcquireParams.

Nonetheless, when default_acquire_buffer() calls the do_alloc_buffer() it does
not pass the received GstBufferPoolAcquireParams, so when the user pass those
parameters they are ignored by alloc_buffer() vmethod.

This one-liner patch pass the received acquire params to do_alloc_buffer().

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

8 years agosystemclock: Fix wait/unschedule race
Stian Selnes [Wed, 10 Feb 2016 08:09:29 +0000 (09:09 +0100)]
systemclock: Fix wait/unschedule race

Fixes a race where an entry is set to BUSY in
gst_system_clock_id_wait_jitter() and is UNSCHEDULED before
gst_system_clock_id_wait_jitter_unlocked() starts processing it. The
wakeup added by gst_system_clock_id_unschedule() must be cleaned up.

Two stress tests are added. One test that triggers the specific issue
described above. The second stresses the code path where a wait is
rescheduled because the poll returned early.

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

8 years agosystemclock: handle unschedule of late entries
Thiago Santos [Fri, 5 Feb 2016 18:34:47 +0000 (15:34 -0300)]
systemclock: handle unschedule of late entries

If the clockentry is too late and is unscheduled before it gets
a change to detect its lateness the wakeup count and the poll are
used but never properly cleaned up. This leaves it in a dirty state
that is going to mess with the next clock entry waiting requests.

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

8 years agotests: extend the AM_TESTS_ENVIRONMENT from check.mak
Thiago Santos [Fri, 5 Feb 2016 22:08:18 +0000 (19:08 -0300)]
tests: extend the AM_TESTS_ENVIRONMENT from check.mak

To get the CK_DEFAULT_TIMEOUT defined for all tests

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

8 years agoAutomatic update of common submodule
Thiago Santos [Fri, 5 Feb 2016 21:01:52 +0000 (18:01 -0300)]
Automatic update of common submodule

From 86e4663 to b64f03f

8 years agobaseparse: fix stray discont flag set on outgoing buffers in push mode
Tim-Philipp Müller [Thu, 4 Feb 2016 10:07:22 +0000 (10:07 +0000)]
baseparse: fix stray discont flag set on outgoing buffers in push mode

We have no guarantees about what flags are set on buffers we take
out of the GstAdapter. If we push out multiple buffers from the
first input buffer (which will have discont set), only the first
buffer we push out should be flagged as discont, not all of the
buffers produced from that first initial input buffer.

Fixes issue where the first few mp3 frames/seconds of data in push
mode were skipped or garbled in some cases, and the discont flags
would also trip up decoders which were getting drained/flushed for
every buffer. This was a regression introduced in 1.6 apparently.

8 years agocontroller: Do not unset uninitiallized GValue
Thibault Saunier [Tue, 2 Feb 2016 15:35:34 +0000 (16:35 +0100)]
controller: Do not unset uninitiallized GValue

In case the property was not interpollable we might never initialize
the GValue, we should thus never unset it.

8 years agodocs: pwg: fix missing end of line semicolon in custom meta example
Tim-Philipp Müller [Tue, 2 Feb 2016 16:34:51 +0000 (16:34 +0000)]
docs: pwg: fix missing end of line semicolon in custom meta example

8 years agouri: add guard to make sure gstreamer is initialized
Tim-Philipp Müller [Tue, 2 Feb 2016 10:56:35 +0000 (10:56 +0000)]
uri: add guard to make sure gstreamer is initialized

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

8 years agogst-uninstalled: add new rtsp server plugin location to plugins path
Tim-Philipp Müller [Mon, 1 Feb 2016 18:41:55 +0000 (18:41 +0000)]
gst-uninstalled: add new rtsp server plugin location to plugins path

8 years agobaseparse: Try to generate caps on the srcpad before forwarding GAP event
HoonHee Lee [Mon, 25 Jan 2016 07:30:04 +0000 (16:30 +0900)]
baseparse: Try to generate caps on the srcpad before forwarding GAP event

To configure downstream elements and complete initial pre-rolling,
ensure we have default output caps before forwarding GAP event.

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

8 years agotypefindelement: Improve English grammar
Alex Henrie [Fri, 29 Jan 2016 03:18:55 +0000 (20:18 -0700)]
typefindelement: Improve English grammar

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

8 years agodocs: fix an other typo in clock chapter
Luis de Bethencourt [Wed, 27 Jan 2016 12:45:20 +0000 (12:45 +0000)]
docs: fix an other typo in clock chapter

Shockingly I missed this bigger typo in the previos fix

8 years agodocs: fix typo in clock chapter
Luis de Bethencourt [Wed, 27 Jan 2016 12:24:57 +0000 (12:24 +0000)]
docs: fix typo in clock chapter

8 years agotests:gstinfo: Fix string memory leak
Vineeth TM [Mon, 25 Jan 2016 03:09:54 +0000 (12:09 +0900)]
tests:gstinfo: Fix string memory leak

info_fourcc test leaks string.

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

8 years agotests: info: make work without registry
Tim-Philipp Müller [Sat, 23 Jan 2016 16:00:48 +0000 (16:00 +0000)]
tests: info: make work without registry

8 years agotracer: rename GST_TRACER_PLUGINS env var to GST_TRACERS
Tim-Philipp Müller [Fri, 22 Jan 2016 12:50:08 +0000 (12:50 +0000)]
tracer: rename GST_TRACER_PLUGINS env var to GST_TRACERS

8 years agotracerrecord: don't leak the spec structures
Stefan Sauer [Thu, 21 Jan 2016 07:12:01 +0000 (08:12 +0100)]
tracerrecord: don't leak the spec structures

Change the gst_tracer_record_new() api to take the parameters the make the
spec structure directly. This allows us to own the top-level structure and
also collect the args so that we can take ownership of the sub-structures.

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

8 years agodevice: fix comparison in _has_classesv()
Tim-Philipp Müller [Thu, 21 Jan 2016 15:45:30 +0000 (15:45 +0000)]
device: fix comparison in _has_classesv()

We're comparing a pointer type with '\0' here, which
probably isn't right, and the loop condition made sure
that classes[0] is != NULL already, so it's pointless.
Was probaby meant to check if the string pointed to is
not empty, so make it do that instead.

8 years agotest/tracerrecord: unref objects and free string
Stefan Sauer [Wed, 20 Jan 2016 19:32:24 +0000 (20:32 +0100)]
test/tracerrecord: unref objects and free string