platform/upstream/gstreamer.git
9 years agoqueue: Illegal memory access of sink event
Vineeth TM [Mon, 14 Dec 2015 01:32:14 +0000 (10:32 +0900)]
queue: Illegal memory access of sink event

Once event is pushed to pad, then queue should not access the event.
This is leading to invalid read valgrind errors

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

9 years agoelement: Unref event if GstElement::send_event() is not implemented
Sebastian Dröge [Mon, 14 Dec 2015 09:10:04 +0000 (10:10 +0100)]
element: Unref event if GstElement::send_event() is not implemented

Otherwise we'll take ownership of the event only if it's implemented, and
leak the event in all other situations.

9 years agomanual: Fix dynamic pipeline example
Sebastian Dröge [Mon, 14 Dec 2015 09:04:19 +0000 (10:04 +0100)]
manual: Fix dynamic pipeline example

Use GST_PAD_PROBE_PASS to pass through all events other than EOS instead of
blocking on the first non-EOS event forever. Also fix a typo in a comment in
that function.

Thanks to David Jaggard for reporting this on the mailing list.

9 years agodocs: typo in the location of make_element tool
Luis de Bethencourt [Sun, 13 Dec 2015 00:37:39 +0000 (00:37 +0000)]
docs:  typo in the location of make_element tool

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

9 years agodownloadbuffer: drop unneeded macros for G_OS_WIN32
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 09:13:59 +0000 (01:13 -0800)]
downloadbuffer: drop unneeded macros for G_OS_WIN32

9 years agofdsrc: drop unneeded macros for G_OS_WIN32
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 09:09:20 +0000 (01:09 -0800)]
fdsrc: drop unneeded macros for G_OS_WIN32

9 years agofdsrc: enable large file support in Android
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 09:06:43 +0000 (01:06 -0800)]
fdsrc: enable large file support in Android

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

9 years agofdsink: enable large file support in Android
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 06:14:32 +0000 (22:14 -0800)]
fdsink: enable large file support in Android

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

9 years agoqueue2: fix some typos
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 04:52:57 +0000 (20:52 -0800)]
queue2: fix some typos

9 years agoqueue2: enable large file support on Android
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 04:42:05 +0000 (20:42 -0800)]
queue2: enable large file support on Android

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

9 years agodownloadbuffer: fix some typos
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 03:11:01 +0000 (19:11 -0800)]
downloadbuffer: fix some typos

9 years agodownloadbuffer: enable large file support on Android
Reynaldo H. Verdejo Pinochet [Sat, 12 Dec 2015 02:59:32 +0000 (18:59 -0800)]
downloadbuffer: enable large file support on Android

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

9 years agosparsefile: drop bogus reference to file descriptor
Reynaldo H. Verdejo Pinochet [Fri, 11 Dec 2015 22:36:29 +0000 (14:36 -0800)]
sparsefile: drop bogus reference to file descriptor

+fix typo on return value comment

9 years agosparsefile: enable large file support on Android
Reynaldo H. Verdejo Pinochet [Fri, 11 Dec 2015 22:07:27 +0000 (14:07 -0800)]
sparsefile: enable large file support on Android

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

9 years agoelement: unref message in _post_message when there is no implementation
Aurélien Zanelli [Thu, 10 Dec 2015 13:32:27 +0000 (14:32 +0100)]
element: unref message in _post_message when there is no implementation

'gst_element_post_message' takes the ownership of the message, so it
shall unref it when there is no post_message implementation. Otherwise
message is leaked.

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

9 years agoelement: Don't hold state lock all the time while sending an event
Sebastian Dröge [Thu, 10 Dec 2015 09:35:05 +0000 (11:35 +0200)]
element: Don't hold state lock all the time while sending an event

This lock seems to exist only to prevent elements from changing states while
events are being processed. However events are going to be processed
nonetheless in those elements if sent directly via pads, so protection must
already be implemented inside the elements for event handling if it is needed.

As such having the lock here is not very useful and is actually causing
various deadlocks in different situations as described in
https://bugzilla.gnome.org/show_bug.cgi?id=744040

9 years agofilesrc: enable large file support in Android
Reynaldo H. Verdejo Pinochet [Tue, 8 Dec 2015 04:27:45 +0000 (20:27 -0800)]
filesrc: enable large file support in Android

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

9 years agofilesink: enable large file support on Android
Reynaldo H. Verdejo Pinochet [Thu, 3 Dec 2015 23:04:32 +0000 (15:04 -0800)]
filesink: enable large file support on Android

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

9 years agofilesink: indentation fixes
Reynaldo H. Verdejo Pinochet [Thu, 3 Dec 2015 23:01:27 +0000 (15:01 -0800)]
filesink: indentation fixes

9 years agomultiqueue: Don't use random segment.position from upstream
Edward Hervey [Wed, 9 Dec 2015 16:40:02 +0000 (17:40 +0100)]
multiqueue: Don't use random segment.position from upstream

segment.position is meant for internal usage only, but the various
GST_EVENT_SEGMENT creationg/parsing functions won't clear that field.

Use the appropriate segment boundary as an initial value instead

9 years agobin: Post a LATENCY message with async-handling=TRUE if the PLAYING state is reached
Sebastian Dröge [Mon, 7 Dec 2015 15:25:02 +0000 (17:25 +0200)]
bin: Post a LATENCY message with async-handling=TRUE if the PLAYING state is reached

Otherwise each bin might have a different latency in the end, causing
synchronization problems.

The bin will still first handle latency internally as before, but gives the
overall pipeline the opportunity to update the latency of the whole pipeline
afterwards.

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

9 years agobaseparse: post tag list when avg bitrate changes at least 2%
Athanasios Oikonomou [Mon, 7 Dec 2015 16:20:35 +0000 (18:20 +0200)]
baseparse: post tag list when avg bitrate changes at least 2%

Watching videos with variant bitrate is common to have delta
more than 10 kbps, resulting in tag list spam.

Instead of relying on fixed 10 kpbs delta, it is better to
calculale the difference in percentage and update tag list
only when bitrate changes more than 2%.

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

9 years agoAutomatic update of common submodule
Nicolas Dufresne [Mon, 7 Dec 2015 14:08:01 +0000 (09:08 -0500)]
Automatic update of common submodule

From b319909 to 86e4663

9 years agoelement: Check for an activated pad before adding, not a non-flushing one
Sebastian Dröge [Mon, 7 Dec 2015 10:59:39 +0000 (12:59 +0200)]
element: Check for an activated pad before adding, not a non-flushing one

The pad could be activated but flushing because of a FLUSH_START event. That's
not what we're looking for here, we want to check for activated pads.

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

9 years agotests: funnel: remove state change from stress tests
Thiago Santos [Thu, 19 Nov 2015 18:33:06 +0000 (15:33 -0300)]
tests: funnel: remove state change from stress tests

Changing states up and down while buffers are being pushed is not
a valid use case. If a pad is deactivated and reactivated during
a buffer push it is racy with the check of pushed sticky events
and the actual chainfunction call. As it might call the chain
without noticing the peer pad lost its previous sticky events.

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

9 years agogst-launch: Fix process return value on error
Nicolas Dufresne [Fri, 4 Dec 2015 15:22:56 +0000 (10:22 -0500)]
gst-launch: Fix process return value on error

In case of a run-time error message, the process return value was left
unset. This would lead to error not being caught at shell level.

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

9 years agoRevert "tools: gst-launch: return non-0 exit code on async error"
Nicolas Dufresne [Fri, 4 Dec 2015 15:45:25 +0000 (10:45 -0500)]
Revert "tools: gst-launch: return non-0 exit code on async error"

This reverts commit 2ee4cba2485d7f1646d48e4559426aed4ba99c85.

9 years agotools: gst-launch: return non-0 exit code on async error
Tim-Philipp Müller [Fri, 4 Dec 2015 15:09:39 +0000 (15:09 +0000)]
tools: gst-launch: return non-0 exit code on async error

When an error happens in playing state, still return a
non-0 exit code.

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

9 years agohelpers: really fix install race
Ross Burton [Fri, 4 Dec 2015 14:39:29 +0000 (14:39 +0000)]
helpers: really fix install race

My previous fix for #758029 wasn't quite right and simply made the race rarer.
Some of the files are installed by install-exec and others by install-exec, so
the hooks need to be split too.

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

9 years agoquery: fix docs
Wim Taymans [Fri, 4 Dec 2015 10:56:45 +0000 (11:56 +0100)]
query: fix docs

The allocation query has parameters with the meta API, not flags.

9 years agoDrop usage of deprecated g-ir-scanner --strip-prefix flag
Reynaldo H. Verdejo Pinochet [Wed, 2 Dec 2015 23:02:25 +0000 (15:02 -0800)]
Drop usage of deprecated g-ir-scanner --strip-prefix flag

9 years agomultiqueue: Fix set/get property
Edward Hervey [Wed, 2 Dec 2015 15:28:23 +0000 (16:28 +0100)]
multiqueue: Fix set/get property

Blame it on the rebasing :)

9 years agopad: Implement GstPadEventFullFunction
Edward Hervey [Thu, 12 Nov 2015 16:15:37 +0000 (17:15 +0100)]
pad: Implement GstPadEventFullFunction

API: GstPadEventFullFunction

Returns a GstFlowReturn, allows asynchronous elements to properly
propagate flow returns

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

9 years agomultiqueue: Use signed clock values for running time calculation
Edward Hervey [Mon, 30 Nov 2015 16:09:43 +0000 (17:09 +0100)]
multiqueue: Use signed clock values for running time calculation

This improves the accuracy of queue levels and when to push buffers
for buffers falling outside of the segment

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

9 years agomultiqueue: Add an extra cache time for unlinked streams
Edward Hervey [Fri, 27 Nov 2015 08:45:29 +0000 (09:45 +0100)]
multiqueue: Add an extra cache time for unlinked streams

When synchronizing the output by time, there are some use-cases (like
allowing gapless playback downstream) where we want the unlinked streams
to stay slightly behind the linked streams.

The "unlinked-cache-time" property allows the user to specify by how
much time the unlinked streams should wait before pushing again.

9 years agomultiqueue: Optimize multiqueue sizes based on interleave
Edward Hervey [Mon, 26 Oct 2015 07:06:01 +0000 (08:06 +0100)]
multiqueue: Optimize multiqueue sizes based on interleave

Multiqueue should only be used to cope with:
* decoupling upstream and dowstream threading (i.e. having separate threads
  for elementary streams).
* Ensuring individual queues have enough space to cope with upstream interleave
  (distance in stream time between co-located samples). This is to guarantee
  that we have enough room in each individual queues to provide new data in
  each, without being blocked.
* Limit the queue sizes to that interleave distance (and an extra minimal
  buffering size). This is to ensure we don't consume too much memory.

Based on that, multiqueue now continuously calculates the input interleave
(per incoming streaming thread). Based on that, it calculates a target
interleave (currently 1.5 x real_interleave + 250ms padding).

If the target interleave is greater than the current max_size.time, it will
update it accordingly (to allow enough margin to not block).
If the target interleave goes down by more than 50%, we re-adjust it once
we know we have gone past a safe distance (2 x current max_size.time).

This mode can only be used for incoming streams that are guaranteed to be
properly timestamped.

Furthermore, we ignore sparse streams when calculating interleave and maximum
size of queues.

For the simplest of use-cases (single stream), multiqueue acts as a single
queue with a time limit of 250ms.
If there are multiple inputs, but each come from a different streaming thread,
the maximum time limit will also end up being 250ms.

On regular files (more than one input stream from the same upstream streaming
thread), it can reduce the total memory used as much as 10x, ending up with
max_size.time around 500ms.

Due to the adaptive nature, it can also cope with changing interleave (which
can happen commonly on some files at startup/pre-roll time)

9 years agomultiqueue: use new stream-start event sparse flag to avoid overreading subtitles
Vincent Penquerc'h [Tue, 8 Jan 2013 21:16:42 +0000 (21:16 +0000)]
multiqueue: use new stream-start event sparse flag to avoid overreading subtitles

This will mean a much lower delay before a subtitles track changes take
effect. Also avoids excessive memory usage in many cases.

This will also consider sparse streams as (individually) never full, so
as to avoid blocking all playback due to one sparse stream.

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

9 years agomultiqueue: Fix high_time computation
Edward Hervey [Fri, 30 Oct 2015 09:22:20 +0000 (10:22 +0100)]
multiqueue: Fix high_time computation

* Avoid the computation completely if we know we don't need it (not in
  sync time mode)
* Make sure we don't override highest time with GST_CLOCK_TIME_NONE on
  unlinked pads
* Ensure the high_time gets properly updated if all pads are not linked
* Fix the comparision in the loop whether the target high time is the same
  as the current time
* Split wake_up_next_non_linked method to avoid useless calculation

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

9 years agoqueue2: Don't report 0% unless empty
Jan Schmidt [Thu, 5 Nov 2015 16:02:42 +0000 (03:02 +1100)]
queue2: Don't report 0% unless empty

When preparing a buffering message, don't report 0% if there
is any bytes left in the queue at all. We still have something
to push, so don't tell the app to start buffering - maybe
we'll get more data before actually running dry.

9 years agogstclock: Fix GST_STIME_ARGS
Edward Hervey [Mon, 30 Nov 2015 16:11:33 +0000 (17:11 +0100)]
gstclock: Fix GST_STIME_ARGS

It wasn't properly handling GST_CLOCK_STIME_NONE and always use the
sign marker (+/-) to make it easier to identify signed values in
logs

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

9 years agocontrolsource: Annotate get_value[_array] as (method)
Thibault Saunier [Mon, 30 Nov 2015 22:08:50 +0000 (23:08 +0100)]
controlsource: Annotate get_value[_array] as (method)

As the names clash with gst_object_get_value[_array]

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

9 years agoplugin: Add prefix support to dependencies
Nicolas Dufresne [Fri, 13 Nov 2015 22:32:57 +0000 (17:32 -0500)]
plugin: Add prefix support to dependencies

This adds a new flags, GST_PLUGIN_DEPENDENCY_FLAG_FILE_NAME_IS_PREFIX,
which allow using the names as prefix for plugin depencies.

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

9 years agoplugin: Allow device nodes as dependency
Nicolas Dufresne [Fri, 13 Nov 2015 21:32:31 +0000 (16:32 -0500)]
plugin: Allow device nodes as dependency

This is useful for feature that are produced after probing a specific
node. You want to reload this plugin if the specific node(s) have been
removed, added, or reloaded.

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

9 years agoplugin: Don't do lossy shift on hash
Nicolas Dufresne [Fri, 13 Nov 2015 21:20:42 +0000 (16:20 -0500)]
plugin: Don't do lossy shift on hash

In plugin is responsible for calculating a hash of the dependencies
in order to determine if the cache should be invalidated or not.
Currently, the hash combining method removes a bit of the original
have before combining with an addition. As we use 32bits for our hash
and shift 1 bit for each file and directory, that resulting hash only
account for the last 32 files. And is more affected by the last file.

Rotating technique (shifting, and adding back the ending bit), can be
use to make the addition non-commutative. In a way that different order
gives different hashes. In this case, I don't preserve this behaviour
because the order in which the files are provided by the OS is
irrelevant.

In most cases, the XOR operation is used to combine hashes. In this
code we use the addition. I decided to preserve the addition because
we make use of non-random hash ((guint) -1) in the algorithm for
matching files that are not really part of the hash (symlinks, special
files). Doing successive XOR on this value, will simply switch from
full ones, to full zero. The XOR used with whitelist has been preserved
as it's based on a fairly randomized hash (g_str_hash).

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

9 years agoinfo: fix compiler warning with -Wpedantic and gcc 5
Lukasz Forynski [Mon, 23 Nov 2015 21:40:34 +0000 (21:40 +0000)]
info: fix compiler warning with -Wpedantic and gcc 5

Gstreamer compiled with gcc 5.2 and -Wpedantic produces the
following warning:

'ISO C does not support '__FUNCTION__' predefined identifier [-Wpedantic]
  const char *s = __FUNCTION__;'

Since gcc 5 enables C99 by default, use __func__ if it's available
instead of the non-standard __FUNCTION__ (as suggested in [2]).

[1]: https://gcc.gnu.org/gcc-5/changes.html
[2]: https://gcc.gnu.org/gcc-5/porting_to.html

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

9 years agocontroller: fix annotation to make g-ir-scanner happy
Tim-Philipp Müller [Fri, 20 Nov 2015 19:45:39 +0000 (19:45 +0000)]
controller: fix annotation to make g-ir-scanner happy

gstinterpolationcontrolsource.h:59: Warning: GstController: missing ":" at column 51:
 * @GST_INTERPOLATION_MODE_CUBIC_MONO: (Since 1.8) monotonic cubic interpolation, will not
gstinterpolationcontrolsource.h:59: Warning: GstController: unknown annotation: since

9 years agotags: add GST_TAG_PRIVATE_DATA
Ravi Kiran K N [Mon, 29 Sep 2014 08:33:13 +0000 (14:03 +0530)]
tags: add GST_TAG_PRIVATE_DATA

Can be used to represent private data that may be
contained in tags, such as ID3v2 PRIV frames.

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

9 years agoRevert "baseparse: do not overwrite header buffer timestamps"
Thiago Santos [Thu, 19 Nov 2015 03:51:44 +0000 (00:51 -0300)]
Revert "baseparse: do not overwrite header buffer timestamps"

This reverts commit 2c475a035543efc0202ecdc52070295a421ed4b4.

This causes issues with h264parse. It breaks timestamps as
there are headers in the middle of the stream and this patch
makes the timestamps for those differ from the ones that
are adjusted, creating a discontinuity and leading to sync
issues.

9 years agoRevert "baseparse: simplify code a bit"
Thiago Santos [Tue, 17 Nov 2015 21:47:48 +0000 (18:47 -0300)]
Revert "baseparse: simplify code a bit"

This reverts commit 3984f7159a72c2eebe01905ee53716e3b2abcb30.

9 years agoqueue2: don't print criticals when receiving custom events in ring buffer mode
Tim-Philipp Müller [Wed, 18 Nov 2015 11:46:45 +0000 (11:46 +0000)]
queue2: don't print criticals when receiving custom events in ring buffer mode

Downgrade from g_warning to GST_WARNING log message.

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

9 years agoptp-helper: Disable multicast loopback
Sebastian Dröge [Wed, 18 Nov 2015 07:21:23 +0000 (09:21 +0200)]
ptp-helper: Disable multicast loopback

We're not really interested in our own packets and ignore them anyway.

9 years agodebugutils: Fix string memory leak
Vineeth TM [Tue, 17 Nov 2015 00:17:58 +0000 (09:17 +0900)]
debugutils: Fix string memory leak

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

9 years agobaseparse: simplify code a bit
Thiago Santos [Mon, 16 Nov 2015 11:22:14 +0000 (08:22 -0300)]
baseparse: simplify code a bit

Avoid repeated checks for testing if a buffer is a header

9 years agocollectpads: handle buffer with dts-only when mapping to running time
Thiago Santos [Fri, 13 Nov 2015 23:44:57 +0000 (20:44 -0300)]
collectpads: handle buffer with dts-only when mapping to running time

Otherwise the buffer was left with the original values and later would
be compared with other buffers that were converted to runninn time,
leading to bad interleaving of multiple streams.

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

9 years agobaseparse: do not overwrite header buffer timestamps
Thiago Santos [Fri, 13 Nov 2015 19:31:06 +0000 (16:31 -0300)]
baseparse: do not overwrite header buffer timestamps

baseparse tries to preserve timestamps from upstream if
it is running on a time segment and write that to
output buffers. It assumes the first DTS is going to be
segment.start and sets that to the first buffers. In case
the buffer is a header buffer, it had no timestamps and
will have only the DTS set due to this mechanism.

This patch prevents this by skipping this behavior for
header buffers.

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

9 years agoidentity: add drop-buffer-flags property
Aleix Conchillo Flaqué [Thu, 18 Jun 2015 20:56:53 +0000 (13:56 -0700)]
identity: add drop-buffer-flags property

New property drop-buffer-flags that will discard buffers that have the
given flags set.

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

9 years agohelpers: fix install race
Ross Burton [Thu, 12 Nov 2015 19:46:44 +0000 (19:46 +0000)]
helpers: fix install race

The install hook needs to be a install-data-hook not an install-exec-hook as the
helpers are installed into helperdir which is considered data (only path
variables with "exec" in are considered executables).

The explicit dependency on install-helpersPROGRAMS was an attempt at solving
this, but this causes occasional races where install-helpersPROGRAMS can run
twice in parallel (once via install-all, once via the hook's dependency).

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

9 years agosystemclock: Add test for gst_clock_get_resolution
Heinrich Fink [Thu, 12 Nov 2015 15:14:18 +0000 (16:14 +0100)]
systemclock: Add test for gst_clock_get_resolution

In a series of time measurements, the diff between now and previous
timestamps is either 0 or at least as long as get_resolution returned.

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

9 years agosystemclock: Use mach_time on Apple platforms
Heinrich Fink [Thu, 12 Nov 2015 10:26:56 +0000 (11:26 +0100)]
systemclock: Use mach_time on Apple platforms

On iOS/OSX g_get_current_time was used by default. However, mach_time is
the preferred high-resolution monotonic clock to be used on Apple
platforms.

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

9 years agotracers: log: add missing hooks
Thiago Santos [Wed, 11 Nov 2015 14:11:23 +0000 (11:11 -0300)]
tracers: log: add missing hooks

Log all possible hooks

9 years agotracer: add element-change-state-pre/post hook
Thiago Santos [Thu, 29 Oct 2015 11:40:32 +0000 (08:40 -0300)]
tracer: add element-change-state-pre/post hook

Helps catching when a state change is starting and ending.

It is also possible to track the end of state changes by checking the
async-done or state-change messages.
This is particularly important for elements that do async state changes.

9 years agopad: test for checking the order of the probe calls
Miguel París Díaz [Wed, 11 Nov 2015 13:32:44 +0000 (14:32 +0100)]
pad: test for checking the order of the probe calls

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

9 years agocontext: fix some copy and paste leftover in docs
Thiago Santos [Tue, 10 Nov 2015 17:41:35 +0000 (14:41 -0300)]
context: fix some copy and paste leftover in docs

9 years agocontext: add some more documentation
Thiago Santos [Tue, 10 Nov 2015 17:37:27 +0000 (14:37 -0300)]
context: add some more documentation

Add a short paragraph on what means for a context to be persistent

9 years agoutils: use 'static inline' instead of 'inline static' for gtk-doc
Tim-Philipp Müller [Mon, 9 Nov 2015 18:02:09 +0000 (18:02 +0000)]
utils: use 'static inline' instead of 'inline static' for gtk-doc

gtk-doc doesn't seem to recognise the former variant.

9 years agodocs: remove dummy function declarations with G_INLINE_FUNCTION for gtk-doc
Tim-Philipp Müller [Mon, 9 Nov 2015 17:59:16 +0000 (17:59 +0000)]
docs: remove dummy function declarations with G_INLINE_FUNCTION for gtk-doc

gtk-doc can handle static inline functions just fine these days,
there's no need for this stuff any more.

9 years agoqueue2: Add the avg-in-rate property.
Duncan Palmer [Mon, 23 Feb 2015 03:16:19 +0000 (13:16 +1000)]
queue2: Add the avg-in-rate property.

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

9 years agoqueue2: add overrun signal
Philippe Normand [Wed, 4 Nov 2015 11:02:51 +0000 (12:02 +0100)]
queue2: add overrun signal

Notifies that the queue2 is full, same as queue does

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

9 years agopad: tests for accept-caps handling with proxy pads
Thiago Santos [Wed, 30 Sep 2015 00:14:19 +0000 (21:14 -0300)]
pad: tests for accept-caps handling with proxy pads

Adds 3 new tests for testing accept-caps behavior with
proxy-caps pads.

1) A scenario where there is no proxy. The caps should be compared to the
template caps of the pad

2) A scenario where there is a compatible pad. The caps should be compared
to the proxied pad caps (and also with the template)

3) A scenario where there is an incompatible proxy pad. No caps should be
possible at all.

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

9 years agopad: check template caps for proxy pads in accept-caps
Thiago Santos [Thu, 5 Nov 2015 19:15:22 +0000 (16:15 -0300)]
pad: check template caps for proxy pads in accept-caps

Validate that the proxy pad indeed accepts the caps by also
comparing with the pad template caps, otherwise when the pad
had no internally linked pads it would always return true.

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

9 years agoFix build with -Werror=maybe-uninitialized
Thibault Saunier [Fri, 6 Nov 2015 11:58:42 +0000 (12:58 +0100)]
Fix build with -Werror=maybe-uninitialized

9 years agopreset: Add a GST_PRESET_PATH env variable for presets to be usable uninstalled
Thibault Saunier [Fri, 6 Nov 2015 11:05:18 +0000 (12:05 +0100)]
preset: Add a GST_PRESET_PATH env variable for presets to be usable uninstalled

And start setting the various uninstalled presets paths.

9 years agofilesink: continue element cleanup even if fclose fails
Anton Bondarenko [Thu, 5 Nov 2015 07:56:43 +0000 (08:56 +0100)]
filesink: continue element cleanup even if fclose fails

Sometimes filesink cleanup during stop may fail due to fclose error.
In this case object left partial cleanup with no file opened
but still holding old file descriptor.

It's not possible to change location property in a such state,
so next start will cause old file overwrite if 'append' does not set.

According to man page and POSIX standard about fclose behavior(extract):
------------------------------------------------------------------------
The fclose() function shall cause the stream pointed to by stream
to be flushed and the associated file to be closed.
...
Whether or not the call succeeds, the stream shall be disassociated
from the file and any buffer set by the setbuf() or setvbuf()
function shall be disassociated from the stream.
...
The fclose() function shall perform the equivalent of a close()
on the file descriptor that is associated with the stream
pointed to by stream.

After the call to fclose(), any use of stream results
in undefined behavior.
------------------------------------------------------------------------

So file is in 'closed' state no matter if fclose succeed or not.
And cleanup could be continued.

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

9 years agopad: Mark sticky events as sent on not-linked
Jan Schmidt [Sat, 31 Oct 2015 13:04:27 +0000 (00:04 +1100)]
pad: Mark sticky events as sent on not-linked

Instead of re-sending sticky events over and over to a not-linked
pad, mark them as sent the first time. If the not-linked came from
downstream, it already received the events. If the pad is actually
not-linked, the sticky events will be rescheduled when the
pad is linked anyway.

9 years agoghostpad: Allow deactivation with no peer.
Jan Schmidt [Thu, 29 Oct 2015 07:53:29 +0000 (18:53 +1100)]
ghostpad: Allow deactivation with no peer.

Allow deactivation in pull-mode, since that implies we
had a peer, activated in pull mode, then the peer disa-peer-ed ;)

9 years agoelement: emit tracer's element-new hook from 'constructed'
Thiago Santos [Wed, 28 Oct 2015 20:31:07 +0000 (17:31 -0300)]
element: emit tracer's element-new hook from 'constructed'

It allows to properly emitting it for all newly created elements

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

9 years agostats: log the element-new hook properly
Thiago Santos [Wed, 28 Oct 2015 09:03:39 +0000 (06:03 -0300)]
stats: log the element-new hook properly

To be able to get the time the elements were created instead of
just logging them without a time

9 years agopad: Unblock blocking pad probes when receiving FLUSH_START in send_event() too
Sebastian Dröge [Wed, 28 Oct 2015 16:04:46 +0000 (18:04 +0200)]
pad: Unblock blocking pad probes when receiving FLUSH_START in send_event() too

Without this, flushing might not unblock the streaming thread and cause deadlocks.

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

9 years agopad: Document the order in which pad probes are called
Sebastian Dröge [Wed, 28 Oct 2015 11:04:25 +0000 (13:04 +0200)]
pad: Document the order in which pad probes are called

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

9 years agopad: Append hooks instead of prepending to call them in the order they were added
Sebastian Dröge [Tue, 27 Oct 2015 16:05:05 +0000 (18:05 +0200)]
pad: Append hooks instead of prepending to call them in the order they were added

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

9 years agonetaddressmeta: gst_buffer_get_net_address_meta() as function
Stian Selnes [Mon, 26 Oct 2015 11:50:12 +0000 (12:50 +0100)]
netaddressmeta: gst_buffer_get_net_address_meta() as function

Implement gst_buffer_get_net_address_meta() as a function instead
of a macro in order to get gobject-introspection to work.

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

9 years agobuffer: flesh out docs for gst_buffer_make_writable() a little
Nirbheek Chauhan [Mon, 26 Oct 2015 12:37:11 +0000 (18:07 +0530)]
buffer: flesh out docs for gst_buffer_make_writable() a little

There is a similar explanation in gst_caps_make_writable, but the existing
documentation can be misleading since it does not define what 'is already
writable' means.

Also note when this function is meant to be used.

9 years agocaps: clarify docs for a few functions that they don't update things in-place
Nirbheek Chauhan [Mon, 26 Oct 2015 12:05:34 +0000 (17:35 +0530)]
caps: clarify docs for a few functions that they don't update things in-place

It is not necessarily clear from the existing introspection tags and
documentation alone.

9 years agoqueue/queue2: Use GST_BUFFER_DTS_OR_PTS
Edward Hervey [Tue, 27 Oct 2015 07:48:07 +0000 (08:48 +0100)]
queue/queue2: Use GST_BUFFER_DTS_OR_PTS

The input of queue/queue2 might have DTS set, in which cas we want
to take that into account (instead of the PTS) to calculate position
and queue levels.

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

9 years agomultiqueue: Use buffer DTS if present, else PTS
Edward Hervey [Tue, 13 Oct 2015 15:20:26 +0000 (17:20 +0200)]
multiqueue: Use buffer DTS if present, else PTS

In order to accurately determine the amount (in time) of data
travelling in queues, we should use an increasing value.

If buffers are encoded and potentially reordered, we should be
using their DTS (increasing) and not PTS (reordered)

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

9 years agocollectpads: Use GST_BUFFER_DTS_OR_PTS
Edward Hervey [Tue, 27 Oct 2015 07:33:41 +0000 (08:33 +0100)]
collectpads: Use GST_BUFFER_DTS_OR_PTS

Simplifies code a bit

9 years agobuffer: Add a GST_BUFFER_DTS_OR_PTS macro
Edward Hervey [Mon, 26 Oct 2015 06:59:54 +0000 (07:59 +0100)]
buffer: Add a GST_BUFFER_DTS_OR_PTS macro

API: GST_BUFFER_DTS_OR_PTS

Many scenarios/elements require dealing with streams of buffers that
might have DTS set (i.e. encoded data, potentially reordered)

To simplify getting the increasing "timestamp" of those buffers, create
a macro that will return the DTS if valid, and if not the PTS

9 years agodoc: add GST_{PTR,SEGMENT}_FORMAT
Guillaume Desmottes [Tue, 6 Oct 2015 10:21:04 +0000 (12:21 +0200)]
doc: add GST_{PTR,SEGMENT}_FORMAT

Very useful formats in debug output so best to have them in the
generated doc.

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

9 years agosegment: Return -1 if gst_segment_to_stream_time_full() considers the position not...
Sebastian Dröge [Mon, 26 Oct 2015 08:53:35 +0000 (10:53 +0200)]
segment: Return -1 if gst_segment_to_stream_time_full() considers the position not inside the segment

Fixes GstPipeline unit test.

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

9 years agouri: fix behaviour for merging uris ending in .. without following /
Florin Apostol [Sat, 24 Oct 2015 15:52:44 +0000 (16:52 +0100)]
uri: fix behaviour for merging uris ending in .. without following /

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

9 years agouri: tests: added unit test for streams ending in .. without following /
Florin Apostol [Sat, 24 Oct 2015 15:43:59 +0000 (16:43 +0100)]
uri: tests: added unit test for streams ending in .. without following /

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

9 years agodebug: Dump pad properties values
Thibault Saunier [Thu, 27 Aug 2015 10:43:28 +0000 (12:43 +0200)]
debug: Dump pad properties values

Currently we only show element properties values, we should also show
pad properties values

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

9 years agosegment: Remove leftover debug g_print()
Sebastian Dröge [Fri, 23 Oct 2015 17:04:42 +0000 (20:04 +0300)]
segment: Remove leftover debug g_print()

9 years agosegment: Add _full variants of all stream/running_time from/to segment position functions
Vivia Nikolaidou [Thu, 15 Oct 2015 11:49:37 +0000 (14:49 +0300)]
segment: Add _full variants of all stream/running_time from/to segment position functions

See formula clarifications in design docs for calculation details.

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

9 years agogst-uninstalled: Added env var for uninstalled PTP helper
Vivia Nikolaidou [Fri, 25 Sep 2015 22:29:07 +0000 (01:29 +0300)]
gst-uninstalled: Added env var for uninstalled PTP helper

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

9 years agobasesink: rename argument of PREROLL_{COND,LOCK} macros
Guillaume Desmottes [Thu, 22 Oct 2015 10:00:42 +0000 (12:00 +0200)]
basesink: rename argument of PREROLL_{COND,LOCK} macros

They take a GstBaseSink instance as argument at not a GstPad. Rename the
argument to 'obj' which is not miss leading and in line with
GST_BASE_SINK_PAD(obj).

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

9 years agogstcontrolsource: Add missing (out) annotation
Mathieu Duponchelle [Thu, 22 Oct 2015 08:05:14 +0000 (10:05 +0200)]
gstcontrolsource: Add missing (out) annotation

9 years agoAutomatic update of common submodule
Tim-Philipp Müller [Wed, 21 Oct 2015 13:34:47 +0000 (14:34 +0100)]
Automatic update of common submodule

From b99800a to b319909

9 years agopad: Fix docs/annotation of gst_pad_probe_info_get_buffer_list()
Sebastian Dröge [Wed, 21 Oct 2015 11:49:49 +0000 (14:49 +0300)]
pad: Fix docs/annotation of gst_pad_probe_info_get_buffer_list()

It's not get_bufferlist(). Because of that it was ignored by the docs and
G-I, leading to crashes because of broken ownership transfer.

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

9 years agoUse new GST_ENABLE_EXTRA_CHECKS #define
Sebastian Dröge [Tue, 20 Oct 2015 14:29:42 +0000 (17:29 +0300)]
Use new GST_ENABLE_EXTRA_CHECKS #define

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