Sebastian Dröge [Sat, 16 Apr 2011 13:23:46 +0000 (15:23 +0200)]
net: Use G_DEFINE_TYPE
Sebastian Dröge [Sat, 16 Apr 2011 13:23:19 +0000 (15:23 +0200)]
gst: Don't use base_init and use G_DEFINE_TYPE instead of GST_BOILERPLATE
Sebastian Dröge [Sat, 16 Apr 2011 13:03:33 +0000 (15:03 +0200)]
element: Inherit element metadata and pad templates from parent classes
This allows to add pad templates and set metadata in class_init instead of
base_init. base_init is a concept that is not supported by almost all
languages and copying the templates/metadata for subclasses is the more
intuitive way of doing things.
Subclasses can override pad templates of parent classes by adding a new
template with the same now.
Also gst_element_class_add_pad_template() now takes ownership of the
pad template, which was assumed by all code before anyway.
Fixes bug #491501.
Sebastian Dröge [Sat, 16 Apr 2011 07:33:06 +0000 (09:33 +0200)]
Merge branch 'master' into 0.11
Sebastian Dröge [Sat, 16 Apr 2011 07:32:17 +0000 (09:32 +0200)]
win32: Add exports for the GstParseContext and GstBufferListIterator types
Sebastian Dröge [Sat, 16 Apr 2011 06:59:58 +0000 (08:59 +0200)]
Merge branch 'master' into 0.11
Tim-Philipp Müller [Fri, 15 Apr 2011 19:58:51 +0000 (20:58 +0100)]
pluginloader: only run gst-plugin-scanner with /usr/bin/arch wrapper on OS X >= 10.5
Based on patch by: Daniel Macks <dmacks@netspace.org>
Earlier versions of OSX don't support proper multiarch and
trying to use /usr/bin/arch -foo with those versions would
just break things.
https://bugzilla.gnome.org/show_bug.cgi?id=615357
Tim-Philipp Müller [Fri, 15 Apr 2011 18:07:55 +0000 (19:07 +0100)]
baseparse: expose gst_base_parse_frame_free() for completeness
API: gst_base_parse_frame_free()
Tim-Philipp Müller [Fri, 15 Apr 2011 17:52:18 +0000 (18:52 +0100)]
baseparse: init frames on the stack with gst_base_parse_frame_init()
Frames must now be inited this way, can't just zero them
out and use them.
Tim-Philipp Müller [Fri, 15 Apr 2011 17:38:46 +0000 (18:38 +0100)]
baseparse: more debug logging, minor clean-up
Trace frames, split out code to queue a frame for later.
Tim-Philipp Müller [Fri, 15 Apr 2011 17:00:21 +0000 (18:00 +0100)]
baseparse: change gst_base_parse_frame_init() to not take a GstBaseParse argument
Tim-Philipp Müller [Fri, 15 Apr 2011 16:41:02 +0000 (17:41 +0100)]
baseparse: make GstBaseParseFrame handling more bindings-friendly
Change semantics of gst_base_parse_push_frame() and make it take
ownership of the whole frame, not just the frame contents. This
is more in line with how gst_pad_push() etc. work. Just transfering
the content, but not the container of something that's not really
known to be a container is hard to annotate properly and probably
won't work. We mark frames allocated on the stack now with a private
flag in gst_base_parse_frame_init(), so gst_base_parse_frame_free()
only frees the contents in that case but not the frame struct itself.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
API: gst_base_parse_frame_new()
Tim-Philipp Müller [Fri, 15 Apr 2011 14:02:20 +0000 (15:02 +0100)]
baseparse: register boxed type for GstBaseFrameParse
To make this usable for bindings.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Sebastian Dröge [Fri, 15 Apr 2011 11:57:47 +0000 (13:57 +0200)]
queue2: Add missing ) to the ring-buffer-max-size property description
Robert Swain [Fri, 15 Apr 2011 08:53:56 +0000 (10:53 +0200)]
baseparse: Remove unused but set variable
GCC 4.6.0 spits warnings about these.
Sebastian Dröge [Thu, 14 Apr 2011 14:06:16 +0000 (16:06 +0200)]
bufferlist: Add boxed type for GstBufferListIterator for gobject-introspection
Sebastian Dröge [Thu, 14 Apr 2011 13:59:28 +0000 (15:59 +0200)]
parse: Add boxed type for GstParseContext for gobject-introspection
Sebastian Dröge [Thu, 14 Apr 2011 13:51:24 +0000 (15:51 +0200)]
gst: Add some more gobject-introspection annotations
Sebastian Dröge [Thu, 14 Apr 2011 07:07:48 +0000 (09:07 +0200)]
multiqueue: Don't leak the sinkpad name
Sebastian Dröge [Thu, 14 Apr 2011 07:07:25 +0000 (09:07 +0200)]
multiqueue: Don't leak pads in the named pads unit test
Sebastian Dröge [Thu, 14 Apr 2011 06:59:14 +0000 (08:59 +0200)]
utils: Fix caps leaks in gst_element_factory_can_accept_{any,all}_caps_in_direction()
David Schleef [Wed, 13 Apr 2011 16:20:13 +0000 (09:20 -0700)]
parser: Allow element names to begin with digits
David Schleef [Wed, 13 Apr 2011 17:24:33 +0000 (10:24 -0700)]
tests: Add test for greatest common divisor
Ole André Vadla Ravnås [Thu, 6 Jan 2011 17:11:31 +0000 (18:11 +0100)]
elements: Fix pad callbacks so they handle when parent goes away
1) We need to lock and get a strong ref to the parent, if still there.
2) If it has gone away, we need to handle that gracefully.
This is necessary in order to safely modify a running pipeline. Has been
observed when a streaming thread is doing a buffer_alloc() while an
application thread sends an event on a pad further downstream, and from
within a pad probe (holding STREAM_LOCK) carries out the pipeline plumbing
while the streaming thread has its buffer_alloc() in progress.
Ole André Vadla Ravnås [Thu, 6 Jan 2011 17:11:31 +0000 (18:11 +0100)]
base: Fix pad callbacks so they handle when parent goes away
1) We need to lock and get a strong ref to the parent, if still there.
2) If it has gone away, we need to handle that gracefully.
This is necessary in order to safely modify a running pipeline. Has been
observed when a streaming thread is doing a buffer_alloc() while an
application thread sends an event on a pad further downstream, and from
within a pad probe (holding STREAM_LOCK) carries out the pipeline plumbing
while the streaming thread has its buffer_alloc() in progress.
Ole André Vadla Ravnås [Thu, 6 Jan 2011 17:11:31 +0000 (18:11 +0100)]
ghostpad: Fix pad callbacks so they handle when parent goes away
1) We need to lock and get a strong ref to the parent, if still there.
2) If it has gone away, we need to handle that gracefully.
This is necessary in order to safely modify a running pipeline. Has been
observed when a streaming thread is doing a buffer_alloc() while an
application thread sends an event on a pad further downstream, and from
within a pad probe (holding STREAM_LOCK) carries out the pipeline plumbing
while the streaming thread has its buffer_alloc() in progress.
Janne Grunau [Wed, 13 Apr 2011 15:26:54 +0000 (17:26 +0200)]
queue2: prevent calculation with GST_CLOCK_TIME_NONE in update_time_level()
Tim-Philipp Müller [Mon, 11 Apr 2011 14:08:30 +0000 (15:08 +0100)]
tests: fix unusued-but-assigned-variable warnings with gcc 4.6
Tim-Philipp Müller [Mon, 11 Apr 2011 12:04:32 +0000 (13:04 +0100)]
tests: disable test_many_bins unit test for now
It fails on the OSX bot (both with git and the last release), and
it doesn't really test anything useful, so may just as well disable
it for now.
Tim-Philipp Müller [Mon, 11 Apr 2011 11:51:36 +0000 (12:51 +0100)]
pluginloader: fix compiler warnings
Cast string constants to make compiler happy.
Tim-Philipp Müller [Mon, 11 Apr 2011 11:04:34 +0000 (12:04 +0100)]
tests: allow more time for the test_many_bins pipeline to preroll
Hopefully makes this test work on the OSX build bot and other
not-so-powerful machines.
https://bugzilla.gnome.org/show_bug.cgi?id=646624
Jan Schmidt [Mon, 11 Apr 2011 10:29:00 +0000 (11:29 +0100)]
pluginloader: make sure gst-plugin-scanner is called with the right arch on OSX
On OSX, GStreamer might be built as a 'fat/universal' binary containing
both 32-bit and 64-bit code. We must take care that gst-plugin-scanner
is executed with the same architecture as the GStreamer core, otherwise
bad things may happen and core/scanner will not be able to communicate
properly.
Should fix issues with (32-bit) firefox using a 32-bit GStreamer core
which then spawns a 'universal' gst-plugin-scanner binary which gets
run in 64-bit mode, causing 100% cpu usage / busy loops or just hanging
firefox until killed.
https://bugzilla.gnome.org/show_bug.cgi?id=615357
Robert Swain [Mon, 11 Apr 2011 09:05:24 +0000 (11:05 +0200)]
pad: Allow tracking of buffers in GST_SCHEDULING debug output
As GST_SCHEDULING reports when buffers pass through pads due to
gst_pad_push calls, they are a good way of tracking the progress of
buffers through pipelines. As such, adding output of the buffer pointers
to these messages allows tracking of specific buffers, easing debugging.
Wim Taymans [Mon, 11 Apr 2011 08:53:39 +0000 (10:53 +0200)]
baseparse: port to 0.11
Wim Taymans [Mon, 11 Apr 2011 08:26:54 +0000 (10:26 +0200)]
Merge branch 'master' into 0.11
Conflicts:
android/base.mk
android/controller.mk
android/dataprotocol.mk
android/elements.mk
android/gst-inspect.mk
android/gst-launch.mk
android/gst-plugin-scanner.mk
android/gst.mk
android/indexers.mk
android/net.mk
win32/common/libgstbase.def
Wim Taymans [Mon, 11 Apr 2011 08:20:10 +0000 (10:20 +0200)]
buffer: add FIXME
Alessandro Decina [Thu, 27 Jan 2011 13:33:08 +0000 (14:33 +0100)]
android: make it ready for androgenizer
Remove the android/ top dir
Fixe the Makefile.am to be androgenized
To build gstreamer for android we are now using androgenizer which generates the needed Android.mk files.
Androgenizer can be found here: http://git.collabora.co.uk/?p=user/derek/androgenizer.git
Tim-Philipp Müller [Sat, 9 Apr 2011 22:54:20 +0000 (23:54 +0100)]
trace: don't put code with side effects into g_return_if_fail()
Tim-Philipp Müller [Sat, 9 Apr 2011 21:57:46 +0000 (22:57 +0100)]
docs: minor fixes for baseparse docs
Class vfunc references still aren't right, no idea what
the correct markup for those is.
Tim-Philipp Müller [Sat, 9 Apr 2011 17:04:55 +0000 (18:04 +0100)]
element: unref event in default_send_event in case element has no pads
Spotted by Haakon Sporsheim.
Edward Hervey [Sat, 9 Apr 2011 02:07:04 +0000 (04:07 +0200)]
check: Ignore new gstmeta binary
Edward Hervey [Sat, 9 Apr 2011 02:05:48 +0000 (04:05 +0200)]
design: draft-buffer2.txt no longer exists
Edward Hervey [Sat, 9 Apr 2011 02:05:20 +0000 (04:05 +0200)]
gst: Don't forget to dist gstelementmetadata.h
Tim-Philipp Müller [Fri, 8 Apr 2011 18:07:02 +0000 (19:07 +0100)]
baseparse: minor variable name clean-up
Tim-Philipp Müller [Fri, 8 Apr 2011 14:31:14 +0000 (15:31 +0100)]
baseparse: rename _set_frame_props() to _set_frame_rate()
Seems like the best fit to what it does, and is shorter than
set_frame_properties() which might also have been confusing
because of GstBaseParseFrame.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Wed, 6 Apr 2011 16:43:27 +0000 (17:43 +0100)]
baseparse: replace format flags with gst_base_parse_set_{passthrough,syncable,has_timing_info}
This is more in line with e.g. GstBaseTransform's API, and makes for nicer
to read code. No getters for now since I don't see any use case for them,
the API is for subclasses, which usually know what format they're
dealing with already and hence know what they've set.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Mon, 4 Apr 2011 16:58:59 +0000 (17:58 +0100)]
baseparse: make DRAIN and SYNC flags on baseparse, not the frame, and change to DRAINING and LOST_SYNC
The first because it seems a better fit conceptually, the second
to express booleanness. Also change the accessor macros for subclasses
to GST_BASE_PARSE_DRAINING and GST_BASE_PARSE_LOST_SYNC.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Sat, 2 Apr 2011 13:18:57 +0000 (14:18 +0100)]
baseparse: add some padding to GstBaseParseFrame
Esp. since it's usually allocated on the stack.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Sat, 2 Apr 2011 13:08:46 +0000 (14:08 +0100)]
baseparse: fix typo in docs for GST_BASE_PARSE_FORMAT_FLAG_PASSTHROUGH
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Sat, 2 Apr 2011 13:04:42 +0000 (14:04 +0100)]
baseparse: use GQueue instead of GList for queued frames
and make buffer metadata writable before setting caps on queued
buffer.
https://bugzilla.gnome.org/show_bug.cgi?id=646341
Zaheer Abbas Merali [Sat, 2 Apr 2011 12:02:01 +0000 (13:02 +0100)]
baseparse: add GST_BASE_PARSE_FLOW_QUEUED to queue buffers until caps are known
This is useful for parser like flacparse or h264parse which may need to process
some buffers before they can construct the final caps, in which case they may
want to delay pushing the initial buffers until the full and proper caps are
known.
https://bugzilla.gnome.org/show_bug.cgi?id=646341
Tim-Philipp Müller [Thu, 31 Mar 2011 14:50:22 +0000 (15:50 +0100)]
baseparse: add to docs and fix up gtk-doc markup a little
And add Since markers.
Tim-Philipp Müller [Thu, 31 Mar 2011 13:48:47 +0000 (14:48 +0100)]
baseparse: replace set_seek() with _set_average_bitrate() and FLAG_SYNCABLE
This makes more sense conceptually, since the bitrate may be used
to estimate a seek position if there's no seek table or just for
duration reporting/estimation if we can't seek. Also, even if the
format is not syncable, we could still seek by pushing data from the
start and using the segment to make downstream clip.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Thu, 24 Mar 2011 17:30:53 +0000 (17:30 +0000)]
baseparse: rename GstBaseFormat to GstBaseFormatFlags and fix up associated API
Also change gst_base_parse_set_format(parse,flags,switch_on) to
gst_base_parse_set_format_flags(parse,flags) which is more in line
with the rest of our API and how the function is used.
Tim-Philipp Müller [Sun, 13 Mar 2011 23:43:52 +0000 (23:43 +0000)]
baseparse: don't expose GstAdapter in public header
None of the existing subclasses needs access to that, so there's
no reason to expose it for now.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Sun, 13 Mar 2011 23:38:12 +0000 (23:38 +0000)]
baseparse: move various segment-related members into the private instance struct
If none of the existing subclasses uses these, there's probably no
need to expose them at the moment. Keep the segment itself exposed
though.
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Sun, 13 Mar 2011 23:30:51 +0000 (23:30 +0000)]
baseparse: remove unused GST_BASE_PARSE_{SINK,SRC}_NAME
https://bugzilla.gnome.org/show_bug.cgi?id=518857
Tim-Philipp Müller [Sat, 12 Mar 2011 16:16:22 +0000 (16:16 +0000)]
baseparse: re-indent header
Tim-Philipp Müller [Sat, 12 Mar 2011 15:34:33 +0000 (15:34 +0000)]
baseparse: fix up GType name and make _get_type() function thread-safe
Rename GType from GstBaseParseBad to GstBaseParse.
Tim-Philipp Müller [Sat, 12 Mar 2011 15:29:38 +0000 (15:29 +0000)]
libs: add GstBaseParse which was moved from -bad
David Schleef [Thu, 24 Feb 2011 01:24:14 +0000 (17:24 -0800)]
baseparse: make_metadata_writable() fix
Tim-Philipp Müller [Mon, 21 Feb 2011 13:24:03 +0000 (13:24 +0000)]
baseparse: rename GType from GstAudioBaseParseBad to GstBaseParseBad
We use it for video as well now.
Stefan Kost [Fri, 18 Feb 2011 13:05:31 +0000 (15:05 +0200)]
baseparse: trim trailing whitespace
Stefan Kost [Fri, 18 Feb 2011 13:05:03 +0000 (15:05 +0200)]
baseparse: use delta-unit flags instead of none
David Schleef [Thu, 17 Feb 2011 21:22:28 +0000 (13:22 -0800)]
baseparse: update documentation for API changes
David Schleef [Wed, 13 Oct 2010 22:39:55 +0000 (15:39 -0700)]
baseparse: Create baseparse library
Mark Nauwelaerts [Mon, 7 Feb 2011 13:46:57 +0000 (14:46 +0100)]
baseparse: tune QUERY_SEEKING response
Even if we currently do not have a duration yet, assume seekable if
it looks like we'll likely be able to determine it later on
(which coincides with needed information to perform seeking).
Fixes #641047.
Arun Raghavan [Tue, 8 Feb 2011 18:09:24 +0000 (23:39 +0530)]
baseparse: Update min/max bitrate before first posting them
This avoids posting an initial min-bitrate of G_UINTMAX and max-bitrate
of 0.
https://bugzilla.gnome.org/show_bug.cgi?id=641857
Mark Nauwelaerts [Fri, 21 Jan 2011 13:53:39 +0000 (14:53 +0100)]
baseparse: tune default duration estimate update interval
Rather than a fixed default frame count, estimate frame count to aim for
an interval duration depending on fps if available, otherwise use old
fixed default.
Mark Nauwelaerts [Fri, 14 Jan 2011 14:16:04 +0000 (15:16 +0100)]
baseparse: reverse playback; mind keyframes for fragment boundary
Mark Nauwelaerts [Wed, 12 Jan 2011 13:40:37 +0000 (14:40 +0100)]
baseparse: ensure non-empty candidate frames
Mark Nauwelaerts [Tue, 11 Jan 2011 14:24:23 +0000 (15:24 +0100)]
baseparse: clarify some debug statements
Mark Nauwelaerts [Tue, 11 Jan 2011 14:24:02 +0000 (15:24 +0100)]
baseparse: properly track upstream timestamps
... rather than with a delay.
Mark Nauwelaerts [Tue, 11 Jan 2011 14:23:29 +0000 (15:23 +0100)]
baseparse: need proper frame duration to obtain sensible frame bitrate
Mark Nauwelaerts [Tue, 11 Jan 2011 14:22:51 +0000 (15:22 +0100)]
baseparse: proper initial values for index tracking variables
Mark Nauwelaerts [Tue, 11 Jan 2011 11:05:13 +0000 (12:05 +0100)]
baseparse: arrange for consistent event handling
Mark Nauwelaerts [Mon, 10 Jan 2011 15:59:59 +0000 (16:59 +0100)]
baseparse: header style cleaning
Mark Nauwelaerts [Mon, 10 Jan 2011 16:07:38 +0000 (17:07 +0100)]
baseparse: provide some more initial frame metadata in parse_frame
... and document accordingly.
Mark Nauwelaerts [Mon, 10 Jan 2011 15:56:36 +0000 (16:56 +0100)]
baseparse: refactor passthrough into format flags
Also add a format flag to signal baseparse that subclass/format can provide
(parsed) timestamp rather than an estimated one. In particular, such "strong"
timestamp then allows to e.g. determine duration.
Mark Nauwelaerts [Mon, 10 Jan 2011 14:34:48 +0000 (15:34 +0100)]
baseparse: introduce a baseparse frame to serve as context
... and adjust subclass parsers accordingly
Mark Nauwelaerts [Fri, 7 Jan 2011 15:39:51 +0000 (16:39 +0100)]
baseparse: restrict duration scanning to pull mode and avoid extra set_caps call
Mark Nauwelaerts [Fri, 7 Jan 2011 14:58:49 +0000 (15:58 +0100)]
baseparse: update some documentation
Also add some more debug.
Mark Nauwelaerts [Thu, 6 Jan 2011 10:41:44 +0000 (11:41 +0100)]
baseparse: allow increasing min_size for current frame parsing only
Also check that subclass actually either directs to skip bytes or
increases expected frame size to avoid going nowhere in bogus
indefinite looping.
Mark Nauwelaerts [Fri, 14 Jan 2011 14:26:37 +0000 (15:26 +0100)]
baesparse: fix refactor regression in loop based parsing
Mark Nauwelaerts [Thu, 6 Jan 2011 10:16:56 +0000 (11:16 +0100)]
baseparse: pass all available data to subclass rather than minimum
Also reduce some adapter calls and add a few debug statements.
Mark Nauwelaerts [Fri, 10 Dec 2010 14:59:49 +0000 (15:59 +0100)]
baseparse: fix reverse playback handling
Mark Nauwelaerts [Fri, 10 Dec 2010 13:56:13 +0000 (14:56 +0100)]
baseparse: minor typo and debug statement cleanup
Mark Nauwelaerts [Fri, 10 Dec 2010 13:40:05 +0000 (14:40 +0100)]
baseparse: reduce locking
... which is either already mute and/or implicitly handled by STREAM_LOCK.
Mark Nauwelaerts [Fri, 14 Jan 2011 13:08:38 +0000 (14:08 +0100)]
baseparse: avoid loop in frame locating interpolation
Thiago Santos [Fri, 14 Jan 2011 19:30:11 +0000 (16:30 -0300)]
audioparsers: baseparse: Be careful to not lose the event ref
Don't unref the event if it hasn't been handled, because the caller
assumes it is still valid and might reuse it.
I ran into this problem when transcoding an AVI (with mp3 inside)
to gpp.
https://bugzilla.gnome.org/show_bug.cgi?id=639555
Tim-Philipp Müller [Thu, 13 Jan 2011 16:27:04 +0000 (16:27 +0000)]
docs: minor baseparse docs/comment fixes
Remove copy'n'paste leftovers.
Mark Nauwelaerts [Mon, 8 Nov 2010 18:58:31 +0000 (19:58 +0100)]
baseparse: increase keyframe awareness
... which is not particular relevant for audio parsing, but more so
in video cases. In particular, auto-determine if dealing with video (caps).
Mark Nauwelaerts [Tue, 30 Nov 2010 14:41:02 +0000 (15:41 +0100)]
baseparse: avoid unexpected stray metadata
Mark Nauwelaerts [Tue, 30 Nov 2010 14:40:28 +0000 (15:40 +0100)]
baseparse: use proper _NONE output value when applicable
Edward Hervey [Thu, 25 Nov 2010 17:56:42 +0000 (18:56 +0100)]
audioparsers: Remove dead assignments
Andoni Morales Alastruey [Thu, 25 Nov 2010 16:14:23 +0000 (17:14 +0100)]
audioparse: fix possible division-by-zero
https://bugzilla.gnome.org/show_bug.cgi?id=635786
Mark Nauwelaerts [Wed, 17 Nov 2010 15:23:42 +0000 (16:23 +0100)]
baseparse: use correct offset when adding index entry
... bearing in mind that BUFFER_OFFSET is media specific and may not
reflect the basic offset after having been parsed.
Mark Nauwelaerts [Wed, 17 Nov 2010 13:30:09 +0000 (14:30 +0100)]
baseparse: enhancements for timestamp marked framed formats
That is, as such formats allow subclass to extract position from frame,
it is possible to extract duration (if not otherwise provided)
from (near) last frame, and a seek can fairly accurately target the required
position.
Fixes #631389.
Mark Nauwelaerts [Tue, 16 Nov 2010 16:06:14 +0000 (17:06 +0100)]
baseparse: refactor frame scanning peformed by _loop
Mark Nauwelaerts [Tue, 16 Nov 2010 17:04:00 +0000 (18:04 +0100)]
baseparse: slightly optimize sending of pending newsegment events