[Bug 650093] [baseparse] GstBaseParseFrame extensibility/flexibility needed

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=650093
  GStreamer | gstreamer (core) | git

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-05-31 14:07:16 UTC ---
> In any case, we would have to maintain frame state during a particular parsing
> round, which mainly means keeping a GstBaseParseFrame around during subsequent
> _chain calls (whereas the pull mode path does not present a problem this way).
> 
> Following commit at least goes this part of the way:
> 
> commit 35567592efcbd44267048f006c0b910bb3d2b5ca
> Author: Mark Nauwelaerts <mark.nauwelaerts at collabora.co.uk>
> Date:   Tue May 17 22:17:14 2011 +0200
> 
>     baseparse: maintain frame state during frame parsing round

Sorry, completely missed that commit. Right, so that takes care of part 1.

Now the next step would be to allocate this frame dynamically on the heap
instead of embedding it into the struct, no? 

And then somehow provide API for the sub-class to tell us how many bytes to
allocate for that struct and/or custom alloc/free funcs or so.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.