platform/upstream/gstreamer.git
13 years agonet: Use G_DEFINE_TYPE
Sebastian Dröge [Sat, 16 Apr 2011 13:23:46 +0000 (15:23 +0200)]
net: Use G_DEFINE_TYPE

13 years agogst: Don't use base_init and use G_DEFINE_TYPE instead of GST_BOILERPLATE
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

13 years agoelement: Inherit element metadata and pad templates from parent classes
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.

13 years agoMerge branch 'master' into 0.11
Sebastian Dröge [Sat, 16 Apr 2011 07:33:06 +0000 (09:33 +0200)]
Merge branch 'master' into 0.11

13 years agowin32: Add exports for the GstParseContext and GstBufferListIterator types
Sebastian Dröge [Sat, 16 Apr 2011 07:32:17 +0000 (09:32 +0200)]
win32: Add exports for the GstParseContext and GstBufferListIterator types

13 years agoMerge branch 'master' into 0.11
Sebastian Dröge [Sat, 16 Apr 2011 06:59:58 +0000 (08:59 +0200)]
Merge branch 'master' into 0.11

13 years agopluginloader: only run gst-plugin-scanner with /usr/bin/arch wrapper on OS X >= 10.5
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

13 years agobaseparse: expose gst_base_parse_frame_free() for completeness
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()

13 years agobaseparse: init frames on the stack with gst_base_parse_frame_init()
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.

13 years agobaseparse: more debug logging, minor clean-up
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.

13 years agobaseparse: change gst_base_parse_frame_init() to not take a GstBaseParse argument
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

13 years agobaseparse: make GstBaseParseFrame handling more bindings-friendly
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()

13 years agobaseparse: register boxed type for GstBaseFrameParse
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

13 years agoqueue2: Add missing ) to the ring-buffer-max-size property description
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

13 years agobaseparse: Remove unused but set variable
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.

13 years agobufferlist: Add boxed type for GstBufferListIterator for gobject-introspection
Sebastian Dröge [Thu, 14 Apr 2011 14:06:16 +0000 (16:06 +0200)]
bufferlist: Add boxed type for GstBufferListIterator for gobject-introspection

13 years agoparse: Add boxed type for GstParseContext 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

13 years agogst: Add some more gobject-introspection annotations
Sebastian Dröge [Thu, 14 Apr 2011 13:51:24 +0000 (15:51 +0200)]
gst: Add some more gobject-introspection annotations

13 years agomultiqueue: Don't leak the sinkpad name
Sebastian Dröge [Thu, 14 Apr 2011 07:07:48 +0000 (09:07 +0200)]
multiqueue: Don't leak the sinkpad name

13 years agomultiqueue: Don't leak pads in the named pads unit test
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

13 years agoutils: Fix caps leaks in gst_element_factory_can_accept_{any,all}_caps_in_direction()
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()

13 years agoparser: Allow element names to begin with digits
David Schleef [Wed, 13 Apr 2011 16:20:13 +0000 (09:20 -0700)]
parser: Allow element names to begin with digits

13 years agotests: Add test for greatest common divisor
David Schleef [Wed, 13 Apr 2011 17:24:33 +0000 (10:24 -0700)]
tests: Add test for greatest common divisor

13 years agoelements: Fix pad callbacks so they handle when parent goes away
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.

13 years agobase: Fix pad callbacks so they handle when parent goes away
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.

13 years agoghostpad: Fix pad callbacks so they handle when parent goes away
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.

13 years agoqueue2: prevent calculation with GST_CLOCK_TIME_NONE in update_time_level()
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()

13 years agotests: fix unusued-but-assigned-variable warnings with gcc 4.6
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

13 years agotests: disable test_many_bins unit test for now
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.

13 years agopluginloader: fix compiler warnings
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.

13 years agotests: allow more time for the test_many_bins pipeline to preroll
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

13 years agopluginloader: make sure gst-plugin-scanner is called with the right arch on OSX
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

13 years agopad: Allow tracking of buffers in GST_SCHEDULING debug output
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.

13 years agobaseparse: port to 0.11
Wim Taymans [Mon, 11 Apr 2011 08:53:39 +0000 (10:53 +0200)]
baseparse: port to 0.11

13 years agoMerge branch 'master' into 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

13 years agobuffer: add FIXME
Wim Taymans [Mon, 11 Apr 2011 08:20:10 +0000 (10:20 +0200)]
buffer: add FIXME

13 years agoandroid: make it ready for androgenizer
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

13 years agotrace: don't put code with side effects into g_return_if_fail()
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()

13 years agodocs: minor fixes for baseparse docs
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.

13 years agoelement: unref event in default_send_event in case element has no pads
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.

13 years agocheck: Ignore new gstmeta binary
Edward Hervey [Sat, 9 Apr 2011 02:07:04 +0000 (04:07 +0200)]
check: Ignore new gstmeta binary

13 years agodesign: draft-buffer2.txt no longer exists
Edward Hervey [Sat, 9 Apr 2011 02:05:48 +0000 (04:05 +0200)]
design: draft-buffer2.txt no longer exists

13 years agogst: Don't forget to dist gstelementmetadata.h
Edward Hervey [Sat, 9 Apr 2011 02:05:20 +0000 (04:05 +0200)]
gst: Don't forget to dist gstelementmetadata.h

13 years agobaseparse: minor variable name clean-up
Tim-Philipp Müller [Fri, 8 Apr 2011 18:07:02 +0000 (19:07 +0100)]
baseparse: minor variable name clean-up

13 years agobaseparse: rename _set_frame_props() to _set_frame_rate()
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

13 years agobaseparse: replace format flags with gst_base_parse_set_{passthrough,syncable,has_tim...
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

13 years agobaseparse: make DRAIN and SYNC flags on baseparse, not the frame, and change to DRAIN...
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

13 years agobaseparse: add some padding to GstBaseParseFrame
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

13 years agobaseparse: fix typo in docs for GST_BASE_PARSE_FORMAT_FLAG_PASSTHROUGH
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

13 years agobaseparse: use GQueue instead of GList for queued frames
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

13 years agobaseparse: add GST_BASE_PARSE_FLOW_QUEUED to queue buffers until caps are known
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

13 years agobaseparse: add to docs and fix up gtk-doc markup a little
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.

13 years agobaseparse: replace set_seek() with _set_average_bitrate() and FLAG_SYNCABLE
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

13 years agobaseparse: rename GstBaseFormat to GstBaseFormatFlags and fix up associated API
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.

13 years agobaseparse: don't expose GstAdapter in public header
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

13 years agobaseparse: move various segment-related members into the private instance struct
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

13 years agobaseparse: remove unused GST_BASE_PARSE_{SINK,SRC}_NAME
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

13 years agobaseparse: re-indent header
Tim-Philipp Müller [Sat, 12 Mar 2011 16:16:22 +0000 (16:16 +0000)]
baseparse: re-indent header

13 years agobaseparse: fix up GType name and make _get_type() function thread-safe
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.

13 years agolibs: add GstBaseParse which was moved from -bad
Tim-Philipp Müller [Sat, 12 Mar 2011 15:29:38 +0000 (15:29 +0000)]
libs: add GstBaseParse which was moved from -bad

13 years agobaseparse: make_metadata_writable() fix
David Schleef [Thu, 24 Feb 2011 01:24:14 +0000 (17:24 -0800)]
baseparse: make_metadata_writable() fix

13 years agobaseparse: rename GType from GstAudioBaseParseBad to GstBaseParseBad
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.

13 years agobaseparse: trim trailing whitespace
Stefan Kost [Fri, 18 Feb 2011 13:05:31 +0000 (15:05 +0200)]
baseparse: trim trailing whitespace

13 years agobaseparse: use delta-unit flags instead of none
Stefan Kost [Fri, 18 Feb 2011 13:05:03 +0000 (15:05 +0200)]
baseparse: use delta-unit flags instead of none

13 years agobaseparse: update documentation for API changes
David Schleef [Thu, 17 Feb 2011 21:22:28 +0000 (13:22 -0800)]
baseparse: update documentation for API changes

13 years agobaseparse: Create baseparse library
David Schleef [Wed, 13 Oct 2010 22:39:55 +0000 (15:39 -0700)]
baseparse: Create baseparse library

13 years agobaseparse: tune QUERY_SEEKING response
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.

13 years agobaseparse: Update min/max bitrate before first posting them
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

13 years agobaseparse: tune default duration estimate update interval
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.

13 years agobaseparse: reverse playback; mind keyframes for fragment boundary
Mark Nauwelaerts [Fri, 14 Jan 2011 14:16:04 +0000 (15:16 +0100)]
baseparse: reverse playback; mind keyframes for fragment boundary

13 years agobaseparse: ensure non-empty candidate frames
Mark Nauwelaerts [Wed, 12 Jan 2011 13:40:37 +0000 (14:40 +0100)]
baseparse: ensure non-empty candidate frames

13 years agobaseparse: clarify some debug statements
Mark Nauwelaerts [Tue, 11 Jan 2011 14:24:23 +0000 (15:24 +0100)]
baseparse: clarify some debug statements

13 years agobaseparse: properly track upstream timestamps
Mark Nauwelaerts [Tue, 11 Jan 2011 14:24:02 +0000 (15:24 +0100)]
baseparse: properly track upstream timestamps

... rather than with a delay.

13 years agobaseparse: need proper frame duration to obtain sensible frame bitrate
Mark Nauwelaerts [Tue, 11 Jan 2011 14:23:29 +0000 (15:23 +0100)]
baseparse: need proper frame duration to obtain sensible frame bitrate

13 years agobaseparse: proper initial values for index tracking variables
Mark Nauwelaerts [Tue, 11 Jan 2011 14:22:51 +0000 (15:22 +0100)]
baseparse: proper initial values for index tracking variables

13 years agobaseparse: arrange for consistent event handling
Mark Nauwelaerts [Tue, 11 Jan 2011 11:05:13 +0000 (12:05 +0100)]
baseparse: arrange for consistent event handling

13 years agobaseparse: header style cleaning
Mark Nauwelaerts [Mon, 10 Jan 2011 15:59:59 +0000 (16:59 +0100)]
baseparse: header style cleaning

13 years agobaseparse: provide some more initial frame metadata in parse_frame
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.

13 years agobaseparse: refactor passthrough into format flags
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.

13 years agobaseparse: introduce a baseparse frame to serve as context
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

13 years agobaseparse: restrict duration scanning to pull mode and avoid extra set_caps call
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

13 years agobaseparse: update some documentation
Mark Nauwelaerts [Fri, 7 Jan 2011 14:58:49 +0000 (15:58 +0100)]
baseparse: update some documentation

Also add some more debug.

13 years agobaseparse: allow increasing min_size for current frame parsing only
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.

13 years agobaesparse: fix refactor regression in loop based parsing
Mark Nauwelaerts [Fri, 14 Jan 2011 14:26:37 +0000 (15:26 +0100)]
baesparse: fix refactor regression in loop based parsing

13 years agobaseparse: pass all available data to subclass rather than minimum
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.

13 years agobaseparse: fix reverse playback handling
Mark Nauwelaerts [Fri, 10 Dec 2010 14:59:49 +0000 (15:59 +0100)]
baseparse: fix reverse playback handling

13 years agobaseparse: minor typo and debug statement cleanup
Mark Nauwelaerts [Fri, 10 Dec 2010 13:56:13 +0000 (14:56 +0100)]
baseparse: minor typo and debug statement cleanup

13 years agobaseparse: reduce locking
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.

13 years agobaseparse: avoid loop in frame locating interpolation
Mark Nauwelaerts [Fri, 14 Jan 2011 13:08:38 +0000 (14:08 +0100)]
baseparse: avoid loop in frame locating interpolation

13 years agoaudioparsers: baseparse: Be careful to not lose the event ref
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

13 years agodocs: minor baseparse docs/comment fixes
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.

13 years agobaseparse: increase keyframe awareness
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).

13 years agobaseparse: avoid unexpected stray metadata
Mark Nauwelaerts [Tue, 30 Nov 2010 14:41:02 +0000 (15:41 +0100)]
baseparse: avoid unexpected stray metadata

13 years agobaseparse: use proper _NONE output value when applicable
Mark Nauwelaerts [Tue, 30 Nov 2010 14:40:28 +0000 (15:40 +0100)]
baseparse: use proper _NONE output value when applicable

13 years agoaudioparsers: Remove dead assignments
Edward Hervey [Thu, 25 Nov 2010 17:56:42 +0000 (18:56 +0100)]
audioparsers: Remove dead assignments

13 years agoaudioparse: fix possible division-by-zero
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

13 years agobaseparse: use correct offset when adding index entry
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.

13 years agobaseparse: enhancements for timestamp marked framed formats
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.

13 years agobaseparse: refactor frame scanning peformed by _loop
Mark Nauwelaerts [Tue, 16 Nov 2010 16:06:14 +0000 (17:06 +0100)]
baseparse: refactor frame scanning peformed by _loop

13 years agobaseparse: slightly optimize sending of pending newsegment events
Mark Nauwelaerts [Tue, 16 Nov 2010 17:04:00 +0000 (18:04 +0100)]
baseparse: slightly optimize sending of pending newsegment events