Thiago Santos [Sat, 14 Feb 2015 14:11:30 +0000 (11:11 -0300)]
qtdemux: do not use sparse streams in push-based seeking
Using the sparse streams can make the push-based seeking return
too far in the stream. It also can lead to issues as the
sparse streams will be ignored when restarting playback and,
if the sparse stream is the one that has the earliest sample,
it will confuse qtdemux's offsets as one stream will have
an earlier offset than the demuxer's one which might lead to
early EOS.
https://bugzilla.gnome.org/show_bug.cgi?id=742661
Jimmy Ohn [Fri, 13 Feb 2015 10:43:16 +0000 (19:43 +0900)]
pulsesink: Enhance code readability in pulsesink_query
In pulsesink_query function, we use a switch for the query
type. In the CAPS case, there is no 'break', instead we
return right away. Use a break and return at the end of
the function instead for better code readability.
https://bugzilla.gnome.org/show_bug.cgi?id=744461
Tim-Philipp Müller [Fri, 13 Feb 2015 20:40:48 +0000 (20:40 +0000)]
splitmuxsink: flag as sink from the start
Philippe Normand [Wed, 11 Feb 2015 14:30:44 +0000 (15:30 +0100)]
qtdemux: Initial 'sidx' atom parsing support
Parse the 'sidx' atom and update the total duration according to the
parser result. The isoff parser code is imported from
gst-plugins-bad's dashdemux and a gst_isoff_sidx_parser_add_data()
function was factored out of the gst_isoff_sidx_parser_add_buffer()
function.
https://bugzilla.gnome.org/show_bug.cgi?id=743578
Jan Schmidt [Tue, 10 Feb 2015 18:06:45 +0000 (05:06 +1100)]
flvdemux: Use gst_video_guess_framerate()
Use gst_video_guess_framerate() from libgstvideo to guess
sensible common framerates where possible from the
floating point fps in the stream.
Sebastian Dröge [Wed, 11 Feb 2015 12:53:02 +0000 (13:53 +0100)]
Improve and fix LATENCY query handling
This now follows the design docs everywhere, especially the maximum latency
handling.
https://bugzilla.gnome.org/show_bug.cgi?id=744106
Sebastian Dröge [Wed, 11 Feb 2015 09:29:55 +0000 (10:29 +0100)]
rtpsession: Handle first RTCP packet and early feedback correctly
According to RFC 4585 section 3.5.3 step 1 we are not allowed to send
an early RTCP packet for the very first one. It must be a regular one.
Also make sure to not use last_rtcp_send_time in any calculations until
we actually sent an RTCP packet already. In specific this means that we
must not use it for forward reconsideration of the current RTCP send time.
Instead we don't do any forward reconsideration for the first RTCP packet.
Wim Taymans [Tue, 10 Feb 2015 17:53:53 +0000 (18:53 +0100)]
rtph263depay: fix compilation with gcc 5.0
Tim-Philipp Müller [Tue, 10 Feb 2015 16:00:07 +0000 (16:00 +0000)]
splitmuxsink: fix example pipeline properly
x264enc might not have a max-key-int property, but it
has a key-int-max property...
Luis de Bethencourt [Tue, 10 Feb 2015 14:57:55 +0000 (14:57 +0000)]
splitmux: fix typo
Luis de Bethencourt [Tue, 10 Feb 2015 14:56:23 +0000 (14:56 +0000)]
splitmux: update example pipeline
Element x264enc doesn't have a max-key-int property
Luis de Bethencourt [Tue, 10 Feb 2015 13:29:32 +0000 (13:29 +0000)]
splitmux: fix memory leak
If execution goes to the beach in line 981, buf_info goes out of scope without
the memory being free'd. Handle this case.
CID #1268403
Tim-Philipp Müller [Sun, 8 Feb 2015 12:03:10 +0000 (12:03 +0000)]
rtspsrc: fix awkward if clause
Jan Schmidt [Fri, 6 Feb 2015 14:41:49 +0000 (01:41 +1100)]
splitmux: Add unit test for file splitting
Add a unit test for file splitting, and fix the leaks in the
splitmuxsink it found
Luis de Bethencourt [Fri, 6 Feb 2015 14:43:22 +0000 (14:43 +0000)]
wavparse: fix which stop variable is used in assignment
Assignment is done to variable segment.stop when the intention was to assign to
local variable stop. Instead of overwriting it, the value is now clamped and
segment.stop is set to it soon after.
CID #1265773
Jan Schmidt [Fri, 6 Feb 2015 13:19:36 +0000 (00:19 +1100)]
splitmux: Fix memory leaks until the test valgrinds clean
Jan Schmidt [Thu, 5 Feb 2015 19:42:17 +0000 (06:42 +1100)]
splitmux: Handle early EOS during part preparation
Handle the case where a short file reaches EOS while we're still
waiting for no-more-pads, and make sure we continue to the internal
READY state for real playback to work properly later.
Jan Schmidt [Thu, 5 Feb 2015 18:03:19 +0000 (05:03 +1100)]
tests: Change splitmux test video files
Avoid test failure by changing the stored video resolution
from 80x60 to 80x64, which needs bug 741030 to be fixed.
Jan Schmidt [Thu, 31 Jul 2014 14:07:53 +0000 (00:07 +1000)]
splitmux: Implement new elements for splitting files at mux level.
Implement 2 new elements - splitmuxsink and splitmuxsrc.
splitmuxsink is a bin which wraps a muxer and takes 1 video stream,
plus audio/subtitle streams, and starts a new file
whenever necessary to avoid overrunning a threshold of either bytes
or time. New files are started at a keyframe, and corresponding audio
and subtitle streams are split at packet boundaries to match
video GOP timestamps.
splitmuxsrc is a corresponding source element which handles
the splitmux:// URL and plays back all component files,
reconstructing the original elementary streams as it goes.
Thiago Santos [Wed, 4 Feb 2015 19:32:14 +0000 (16:32 -0300)]
tests: souphttpsrc: update ssl key/cert pair
Our ones were expired. The new ones were copied from libsoup's
tests files.
Also sets the property to use our own cert to validate the
server, otherwise the default system certs would be used
and it would fail.
Thiago Santos [Wed, 4 Feb 2015 05:25:44 +0000 (02:25 -0300)]
rtph264depay: prevent trying to get 0 bytes from adapter
This causes an assertion and would lead to getting a NULL instead
of a buffer. Without proper checking this would easily lead to
a segfault
https://bugzilla.gnome.org/show_bug.cgi?id=737199
Jan Schmidt [Wed, 4 Feb 2015 10:50:51 +0000 (21:50 +1100)]
qtdemux: Simple implementation of GST_SEGMENT_FLAG_TRICKMODE_KEY_UNITS
When the trickmode key-units flag is set on the segment, simply skip
any sample on a video stream that isn't a keyframe
Wim Taymans [Tue, 3 Feb 2015 16:35:52 +0000 (17:35 +0100)]
rtspsrc: fix container handling
We detect a container correctly now so we need to revert the weird
check there was before.
Use gst_rtspsrc_stream_push_event() to push the caps event on the
right pad.
See https://bugzilla.gnome.org/show_bug.cgi?id=739391
Thiago Santos [Mon, 2 Feb 2015 22:46:27 +0000 (19:46 -0300)]
matroskamux: store and write stream tags
Separate global from stream tags storage and write them to the
appropriate tags entry in the output
Thiago Santos [Mon, 2 Feb 2015 16:35:59 +0000 (13:35 -0300)]
qtdemux: parse stream tags
Keep global and stream tags separately and parse the udta node
that can be found under the trak atom. The udta will contain
stream specific tags and will be pushed as such
https://bugzilla.gnome.org/show_bug.cgi?id=692473
Thiago Santos [Sat, 31 Jan 2015 17:32:34 +0000 (14:32 -0300)]
qtmux: store stream and container tags separately
Tags received via events, when marked as stream tags, will
be stored on that stream's trak atom instead of being stored
in the main tags atom. This allows the resulting file to have
global and stream tags stored.
https://bugzilla.gnome.org/show_bug.cgi?id=692473
Thiago Santos [Sat, 31 Jan 2015 16:14:44 +0000 (13:14 -0300)]
qtmux: refactor tags functions to accomodata UDTA at trak level
Refactor the functions that were bound to the 'moov' atom to
directly pass the desired 'udta' that should receive the tags.
This allows the tags to be written to 'udta' at the 'moov' or
the 'trak' level, creating tags that are for the container or
for a stream only.
https://bugzilla.gnome.org/show_bug.cgi?id=692473
Thiago Santos [Sat, 31 Jan 2015 13:47:40 +0000 (10:47 -0300)]
qtmux: map application name to _swr tag
It refers to the application name and version used to create the
file
https://bugzilla.gnome.org/show_bug.cgi?id=692473
Jan Schmidt [Fri, 30 Jan 2015 15:30:40 +0000 (02:30 +1100)]
matroska: Fix seeking past the end of the file in reverse mode.
Snap to the end of the file when seeking past the end in reverse mode,
and also fix GST_SEEK_TYPE_END and GST_SEEK_TYPE_NONE handling
for the stop position by always seeking on a segment in stream time
Sebastian Dröge [Fri, 30 Jan 2015 17:22:31 +0000 (18:22 +0100)]
rtpsession: Fix signal name
This wasn't meant to be pushed at all yet, but now that it's there
already it won't hurt to make it correct at least.
Sebastian Dröge [Fri, 30 Jan 2015 15:56:35 +0000 (16:56 +0100)]
rtpstats: Fix typo in documentation
Sebastian Dröge [Fri, 30 Jan 2015 15:50:36 +0000 (16:50 +0100)]
rtpsession: Add new on-receiving-rtcp signal
This will be emitted whenever an RTCP packet is received. Different to
on-feedback-rtcp, this signal gets every complete RTCP packet and not
just the individual feedback packets.
Thiago Santos [Wed, 28 Jan 2015 17:02:15 +0000 (14:02 -0300)]
qtdemux: simplify segment.base math
Remove a fix for heavily edited files added for fixing
https://bugzilla.gnome.org/show_bug.cgi?id=345830 to work
with seeks and proper gaps playback. The fix was replaced
for a more general solution that bases on using previous
segment's duration, just like it works for media segments
playback.
https://bugzilla.gnome.org/show_bug.cgi?id=743518
Luis de Bethencourt [Tue, 27 Jan 2015 14:00:35 +0000 (14:00 +0000)]
videomixer: update orc files
Thiago Santos [Mon, 26 Jan 2015 20:08:12 +0000 (17:08 -0300)]
qtdemux: Fix data dropping for fragmented streams
For fragmented streams with extra data at the end of the mdat
qtdemux was not dropping those bytes and would try to use
that extra data as the beginning of a new atom, causing the
stream to fail.
https://bugzilla.gnome.org/show_bug.cgi?id=743407
Sebastian Dröge [Sun, 25 Jan 2015 16:30:33 +0000 (17:30 +0100)]
rtpsession: Deprecate rtcp-immediate-feedback-threshold property
It had no effect since quite some time and also is not needed in general,
especially not to switch between immediate feedback mode and early feedback
mode. The latest understanding of the RFC is that from the endpoint point of
view, both modes are exactly the same. RTCP is only allowed to use the
bandwidth as given by the RFC constraints, as such it is only ever possible
to schedule a RTCP packet early but it's against the RFC to schedule more RTCP
packets.
The difference between immediate feedback mode and early feedback mode is that
the former guarantees that an RTCP packet can be sent for every event
"immediately", which means that the bandwidth calculations from the RFC have
resulted in an RTCP scheduling interval that is small enough. Early feedback
mode on the other hand means that we can schedule some packets early to make
that happen, but it's not guaranteed at all that it's possible to schedule
an RTCP packet per event (i.e. they need to be accumulated or dropped).
Sebastian Dröge [Thu, 22 Jan 2015 09:29:39 +0000 (10:29 +0100)]
rtpsession: Delay the next regular RTCP packet after early RTCP
This is required to not exceed the short term average RTCP bitrate when
using early feedback as compared to without early feedback.
Sebastian Dröge [Thu, 22 Jan 2015 09:28:52 +0000 (10:28 +0100)]
rtpsession: Add new send-rtcp-full signal
This indicates with a boolean return value if scheduling a new RTCP packet
within the requested delay was possible. Otherwise it behaves exactly like
send-rtcp. The only reason for adding a new signal is ABI compatibility.
Jimmy Ohn [Tue, 20 Jan 2015 00:32:00 +0000 (00:32 +0000)]
pulsesink: Free format_info in query_getcaps
If we can not create probe stream in query_getcaps function, it will appear
memory leakage from format info.
The following patch prevent memory leakage in pulsesink.
https://bugzilla.gnome.org/show_bug.cgi?id=743178
Luis de Bethencourt [Fri, 23 Jan 2015 17:35:51 +0000 (17:35 +0000)]
matroskademux: remove unnecessary check
No matter if gst_matroska_read_common_parse_index_cuetrack () returns that the
flow is OK or not, the check there will be a break from the switch. Removing the
check since the outcome is the same.
CID #1265762
Edward Hervey [Fri, 23 Jan 2015 14:16:25 +0000 (15:16 +0100)]
matroskamux: Avoid using freed variable
the name variable might have been attributed to pad_name, make sure we
free it only *after* pad_name has been used.
Coverity CID : 1265774
Edward Hervey [Fri, 23 Jan 2015 14:13:55 +0000 (15:13 +0100)]
avimux: Avoid using freed variable
the name variable might have been attributed to pad_name, make sure we
free it only *after* pad_name has been used.
Coverity CID : 1265775
Peter Seiderer [Fri, 14 Nov 2014 11:59:31 +0000 (12:59 +0100)]
v4l2object: reuse caps framerate if not overwritten by v4l2 device
Enables duration setting in v4l2src.
https://bugzilla.gnome.org/show_bug.cgi?id=740403
Sebastian Dröge [Thu, 22 Jan 2015 09:29:24 +0000 (10:29 +0100)]
rtpsession: Fix indention
Edward Hervey [Wed, 21 Jan 2015 16:36:26 +0000 (17:36 +0100)]
qtdemux_dump: Bypass even more code if debugging is disabled
And avoid using variables that won't exist when debugging is disabled
Edward Hervey [Wed, 21 Jan 2015 14:30:33 +0000 (15:30 +0100)]
qtdemux: Only traverse/dump nodes if guaranteed to be used
__gst_debug_min is the "global" lowest debug level set. There's no
guarantee the qtdemux debug category is actually set at that level.
Edward Hervey [Sat, 20 Dec 2014 16:09:14 +0000 (17:09 +0100)]
matroska: Avoid debugging below category threshold
This part alone was what made the matroska thread take a full core
on an android phone ...
Sebastian Dröge [Wed, 21 Jan 2015 08:55:30 +0000 (09:55 +0100)]
Constify some static arrays everywhere
Vincent Penquerc'h [Mon, 19 Jan 2015 17:49:54 +0000 (17:49 +0000)]
qtdemux: fix deadlock seeking in files without seek entries
A mutex unlock was missing.
https://bugzilla.gnome.org/show_bug.cgi?id=739975
Vincent Penquerc'h [Mon, 19 Jan 2015 12:34:25 +0000 (12:34 +0000)]
videomixer: fix illegal memory access in blend function with negative ypos
https://bugzilla.gnome.org/show_bug.cgi?id=741115
Nicolas Dufresne [Tue, 13 Jan 2015 21:49:34 +0000 (16:49 -0500)]
v4l2videodec: Proxy getcaps
Replace the sink_query with new getcaps() virtual and use the proxy
helper with the probed caps. This allow upstream element taking decision
base on what is supported downstream.
Sebastian Dröge [Tue, 13 Jan 2015 18:05:20 +0000 (19:05 +0100)]
qtmux: Add support for v210
Sebastian Dröge [Tue, 13 Jan 2015 17:58:01 +0000 (18:58 +0100)]
qtdemux: v210 is v210, not UYVY and yuv2 is YUY2, not I420
Also add a few other raw video formats we support: v308, v216
and add comments for a few others we don't support yet.
https://developer.apple.com/library/mac/technotes/tn2162/
Stefan Sauer [Mon, 12 Jan 2015 14:56:29 +0000 (15:56 +0100)]
Automatic update of common submodule
From f2c6b95 to bc76a8b
Sebastian Dröge [Sat, 10 Jan 2015 14:51:16 +0000 (15:51 +0100)]
osxvideosink: Disable hack for NSApp iteration with a special #define
The hack causes deadlocks and other interesting problems and it really
can only be fixed properly inside GLib. We will include a patch for
GLib in our builds for now that handles this, and hopefully at some
point GLib will also merge a proper solution.
A proper solution would first require to refactor the polling in
GMainContext to only provide a single fd, e.g. via epoll/kqueue
or a thread like the one added by our patch. Then this single
fd could be retrieved from the GMainContext and directly integrated
into a NSRunLoop.
https://bugzilla.gnome.org/show_bug.cgi?id=741450
https://bugzilla.gnome.org/show_bug.cgi?id=704374
Mark Nauwelaerts [Thu, 8 Jan 2015 20:07:05 +0000 (21:07 +0100)]
pulsesink: uncork if needed upon commit
... to provide for a running clock.
Nicolas Dufresne [Fri, 9 Jan 2015 21:59:53 +0000 (16:59 -0500)]
v4l2videodec: Prevent renegotiation
Renegotiation isn't supported, simply prevent it the way we do in
v4l2src.
Nicolas Dufresne [Tue, 6 Jan 2015 18:54:25 +0000 (13:54 -0500)]
v4l2videodec: Don't unlock the stream lock twice
Thiago Santos [Fri, 9 Jan 2015 14:40:40 +0000 (11:40 -0300)]
qtdemux: fix stream time conversion
Use the right macro to convert to the correct scale or the
segment information will be wrong
https://bugzilla.gnome.org/show_bug.cgi?id=742572
Nicolas Dufresne [Wed, 7 Jan 2015 23:48:58 +0000 (18:48 -0500)]
v4l2allocator: Add protection against driver bug
v4l2loopback driver has a this nasty bug that if the queue is larger
then 2 buffers, it returns random index on dqbuf. So far we assumed
that the index was always right, which would lead to memory being
unref twice, and eventually crash.
Nicolas Dufresne [Wed, 7 Jan 2015 22:58:05 +0000 (17:58 -0500)]
v4l2: Don't use allocator size to iterate
As the buffer array is fixed size and small, it's safer to simply
use this static size to cleanup the buffers. This is also more
consistent with the rest. The associated method is no longer
required and can be dropped.
Nicolas Dufresne [Wed, 7 Jan 2015 22:55:14 +0000 (17:55 -0500)]
v4l2bufferpool: Don't clean buffer array in dispose
This should already have been done, plus this code is incorrect
and may lead to crash.
https://bugzilla.gnome.org/show_bug.cgi?id=742074
Nicolas Dufresne [Wed, 7 Jan 2015 22:48:31 +0000 (17:48 -0500)]
v4l2bufferpool: Don't ref queued output buffer
This partly revert to the old 1.2 behavior. Instead of keeping a
reference to the output buffer queued, we simply release them but
don't forward it to GstBufferPool. This way, the buffer pool don't
need to be flushed to be stopped.
https://bugzilla.gnome.org/show_bug.cgi?id=742074
Nicolas Dufresne [Thu, 8 Jan 2015 16:37:23 +0000 (11:37 -0500)]
v4l2bufferpool: Never fail on streamoff
Failing streamoff prevents allocator from being disposed hence
lead to device FD leak. There is no known cases where streamoff
may fails for which we'd still be streaming. streamoff is known
to fail when a device is being unplugged (in which case errno
19/ENODEV is set).
https://bugzilla.gnome.org/show_bug.cgi?id=732734
Brad Smith [Thu, 8 Jan 2015 02:52:17 +0000 (21:52 -0500)]
v4l2: Add support for detecting the presence of V4L2 support on OpenBSD
https://bugzilla.gnome.org/review?bug=742503
Matej Knopp [Sun, 4 Jan 2015 14:57:10 +0000 (15:57 +0100)]
ac3parse: request at least 8 bytes to properly parse header
https://bugzilla.gnome.org/show_bug.cgi?id=742325
Michael Smith [Thu, 8 Jan 2015 00:20:03 +0000 (16:20 -0800)]
wavparse: skip an additional uninteresting chunk type before the fmt chunk.
Luis de Bethencourt [Wed, 7 Jan 2015 18:16:12 +0000 (18:16 +0000)]
audiodynamic: assert func_index is inside bounds
Bringing back the check removed in the previous commit but have that check be a
g_assert. Changing the function to static void since return can never be False,
because audio format will never be unkown.
Luis de Bethencourt [Wed, 7 Jan 2015 17:31:39 +0000 (17:31 +0000)]
audiodynamic: remove always-true conditional
func_index is set by the sum of three ternary operators which add, 0:4, 0:2,
and 1:0. Minimum value would be 0+0+0=0, and maximum would be 4+2+1=7.
The conditional checking if func_index is >= 0 and < 8 will always be true.
Removing it.
CID 1226442
Sebastian Dröge [Wed, 7 Jan 2015 17:05:18 +0000 (18:05 +0100)]
rtpjitterbuffer: If we get a gap with a buffer without DTS, error out
We (currently?) can't really handle gaps between RTP packets if they're not
properly timestamped. The current code would go into calculations with
GST_CLOCK_TIME_NONE and then cause assertions everywhere. It's probably
better to error out cleanly instead.
Aleix Conchillo Flaqué [Fri, 21 Nov 2014 19:39:19 +0000 (11:39 -0800)]
rtspsrc: set PLAYING state after configuring caps
We set to PLAYING after we have configured the caps, otherwise we
might end up calling request_key (with SRTP) while caps are still
being configured, ending in a crash.
https://bugzilla.gnome.org/show_bug.cgi?id=740505
Tim-Philipp Müller [Tue, 30 Dec 2014 18:03:22 +0000 (18:03 +0000)]
tests: gdkpixbufoverlay-test: remove outdated FIXME
Tim-Philipp Müller [Tue, 30 Dec 2014 17:19:42 +0000 (17:19 +0000)]
tests: rtpcollision: use alawenc/dec in these tests instead of Speex
They should always be built, while the speex elements are not.
Need to check for a smaller number of buffers then (7->4) because
speexenc will add 3 header buffers while alawenc will just output
as many buffers as it receives as input.
https://bugzilla.gnome.org/show_bug.cgi?id=742098
Tim-Philipp Müller [Tue, 30 Dec 2014 16:36:02 +0000 (16:36 +0000)]
tests: simple-launch-lines: only run jpeg/png tests if elements are available
Sebastian Dröge [Tue, 30 Dec 2014 15:26:58 +0000 (16:26 +0100)]
souphttpsrc: Don't return a buffer when returning not GST_FLOW_OK
basesrc assumes that we don't return a buffer if
something else than OK is returned. It will just
leak any buffer we might accidentially provide
here.
This can potentially happen during flushing.
Maybe fixes https://bugzilla.gnome.org/show_bug.cgi?id=741993
Tim-Philipp Müller [Tue, 30 Dec 2014 14:52:42 +0000 (14:52 +0000)]
tests: rtpaux: use alawenc/dec in these tests instead of Speex
They should always be built, while the speex elements are not.
https://bugzilla.gnome.org/show_bug.cgi?id=742098
Sebastian Dröge [Mon, 29 Dec 2014 14:35:19 +0000 (15:35 +0100)]
matroskademux: Improve detection of being stuck at the same offset
Only error out if we read from the same position again and got the
same length. Just the same position is not necessarily enough.
Sebastian Dröge [Mon, 29 Dec 2014 14:00:02 +0000 (15:00 +0100)]
matroskademux: Don't get stuck at the same offset when searching for clusters
This could happen if there is an invalid cluster with size 0, and in that
case just error out instead of looping forever.
Tim-Philipp Müller [Thu, 25 Dec 2014 21:32:40 +0000 (21:32 +0000)]
qtmux: fix ALAC muxing
Actually copy the codec data instead of copying nothing
and then bombing out because there's no data.
Fixes: gst-launch-1.0 audiotestsrc ! avenc_alac ! qtmux ! fakesink
https://bugzilla.gnome.org/show_bug.cgi?id=741783
Tim-Philipp Müller [Thu, 25 Dec 2014 15:48:04 +0000 (15:48 +0000)]
rtpptdemux: just drop invalid rtp packets instead of erroring out
Apparently linphone sends an invalid RTP packet as very
first packet. We want to ignore that instead of erroring
out (same for any other invalid packets really).
https://bugzilla.gnome.org/show_bug.cgi?id=741398
Tim-Philipp Müller [Thu, 25 Dec 2014 15:44:15 +0000 (15:44 +0000)]
rtpptdemux: fix 0.10-ism in docs
Tim-Philipp Müller [Thu, 25 Dec 2014 14:58:12 +0000 (14:58 +0000)]
tests: gdkpixbufoverlay-test: use absolute positioning to fix demo
https://bugzilla.gnome.org/show_bug.cgi?id=739566
Tim-Philipp Müller [Thu, 25 Dec 2014 14:53:09 +0000 (14:53 +0000)]
gdkpixbufoverlay: add "positioning-mode" property to allow absolute positions
Set positioning-mode=pixels-absolute to allow positioning with
absolute coordinates, meaning negative x/y offsets will be
interpreted as being to the left/above the video frame instead
of being interpreted as relative to the right/bottom edge of
the video frame (which is a silly default, but that's how it is).
This means we can nicely slide images into and out of the frame,
see gdkpixbufoverlay-test.
https://bugzilla.gnome.org/show_bug.cgi?id=739566
Sebastian Dröge [Mon, 22 Dec 2014 14:33:51 +0000 (15:33 +0100)]
osxaudio: Directly return the ringbuffer's caps if it is acquired
Sebastian Dröge [Mon, 22 Dec 2014 11:56:19 +0000 (12:56 +0100)]
osxaudio: Put all audio formats into the template caps
We report the proper caps later from the get_caps() vfunc implementation after
probing the selected device.
Sebastian Dröge [Mon, 22 Dec 2014 11:56:05 +0000 (12:56 +0100)]
osxaudio: Also set the big endian flag for floating point samples
Sebastian Dröge [Mon, 22 Dec 2014 10:45:59 +0000 (11:45 +0100)]
MAINTAINERS: Update my mail address
Sebastian Dröge [Mon, 22 Dec 2014 09:23:01 +0000 (10:23 +0100)]
osxaudio: Fix deadlock and property change notification in device selection code
After creating the ringbuffer we have to set the device on the ringbuffer as
it defaults to kAudioDeviceUnknown. At this point it can't have changed to
anything else yet and we don't have to notify about changes to the sink/src
"device" property. It's also not a good idea because GstAudioBaseSrc has the
object lock taken while the ringbuffer is created, which might cause a
deadlock if something calls back into the element from "notify::device".
Once the base class is done with the NULL_TO_READY state change, it has opened
the device via the ringbuffer and this might have chosen a different device.
Especially if we initially used kAudioDeviceUnknown. Also notify about this
property change as initially intended by this code.
Nicolas Dufresne [Fri, 19 Dec 2014 17:30:03 +0000 (12:30 -0500)]
v4l2pool: Update configuration size
We already update our copy of VideoInfo.size to proper size, now also
the configuration so the size matches on release.
https://bugzilla.gnome.org/show_bug.cgi?id=741420
Edward Hervey [Fri, 19 Dec 2014 09:57:29 +0000 (10:57 +0100)]
matroska-demux: Cache upstream length
Instead of constantly querying upstream, just cache the last duration,
and in the unlikelyness we might have gone over query again before
deciding we are EOS.
Cut 15% cpu off matroskademux streaming thread (srsly...)
Vincent Penquerc'h [Wed, 17 Dec 2014 17:36:18 +0000 (17:36 +0000)]
matroska: mux/demux the OpusHead header
This is meant to be so (https://wiki.xiph.org/MatroskaOpus - while
it is marked as a draft, this part was confirmed to be correct on
IRC), and allows one to determine whether a demuxed stream is
multistream or not, and thus set the multistream caps field
accordingly. In turn, this means downstream does not have to guess.
https://bugzilla.gnome.org/show_bug.cgi?id=740744
Sebastian Dröge [Thu, 18 Dec 2014 10:50:33 +0000 (11:50 +0100)]
rtspsrc: Don't dereference NULL if a suitable stream for the AUX element can't be found
CID 1258717
Sebastian Dröge [Thu, 18 Dec 2014 09:53:39 +0000 (10:53 +0100)]
Automatic update of common submodule
From ef1ffdc to f2c6b95
Tim-Philipp Müller [Fri, 12 Dec 2014 23:06:07 +0000 (23:06 +0000)]
udpsink: allocate scratch space for render functions on the heap
and not the stack. Our allocations could get a bit too large
to be sure it's not going to cause trouble using the stack.
Tim-Philipp Müller [Tue, 24 Jun 2014 00:16:37 +0000 (01:16 +0100)]
multiudpsink: re-use send_buffers() code path for render() function
It's like rendering a buffer list, just with one buffer.
Has the added advantage that if there are multiple clients
we can send the buffer to all the clients in one go.
Tim-Philipp Müller [Tue, 24 Jun 2014 00:15:25 +0000 (01:15 +0100)]
multiudpsink: keep client list consistent during removals
We unlock and re-lock the client lock while emitting the
removed signal, which causes inconsistencies in the client
list vs. the client counts. Instead, remove the client from
the list already before emitting the signal and put it into
a temporary list of clients to be removed. That way things
look consistent to the streaming thread, but signal callbacks
can still do things like get stats from removed clients.
Tim-Philipp Müller [Mon, 23 Jun 2014 23:56:27 +0000 (00:56 +0100)]
multiudpsink: fix client count after removal
Tim-Philipp Müller [Mon, 23 Jun 2014 17:43:21 +0000 (18:43 +0100)]
multiudpsink: keep client list sorted by socket family
We make use of in the send_buffers() function if we
need to use different sockets to send to IPv4 and
IPv6 destinations.
Tim-Philipp Müller [Fri, 20 Jun 2014 10:36:19 +0000 (11:36 +0100)]
multiudpsink: add sendmmsg-ready render_list function prototype
Add prototype for a render_list() function that can use a
sendmmsg-style g_socket_send_messages() function once it lands
in GLib. We can use this infrastructure to send multiple buffers
made up by multiple memories to multiple clients in one go, which
drastically reduces the number of syscalls made when sending
high-bitrate video streams.
https://bugzilla.gnome.org/show_bug.cgi?id=732152
Tim-Philipp Müller [Thu, 19 Jun 2014 18:16:01 +0000 (19:16 +0100)]
multiudpsink: make udp client structure refcounted
Use the refcount for memory management and keep track
of the number of duplicate clients in a separate
variable. This will be useful later, and means we
don't have to hold the OBJECT_LOCK all the time.
https://bugzilla.gnome.org/show_bug.cgi?id=732866