[Bug 518857] [API] GstBaseParse: new base class for parsers

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Sat Apr 2 07:04:28 PDT 2011


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

--- Comment #26 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-04-02 14:04:17 UTC ---
Ok, mostly done with this I think.

Still need to fix up the DRAIN and SYNC macros though. On IRC we talked about
just making them flags on GstBaseParse (instead of the frame). That would work
for me, but I wonder if just adding gbooleans wouldn't be more straight-forward
(all this just to avoid the overhead of one or two function calls to per frame
- is that really worth it?).

Thing is that we'll probably need to add _get_type() functions etc. for all the
different flags, and we'll likely also need to register a boxed type for
GstBaseParseFrame, for bindings (that's going to be awkward enough.

For the format flags I'm also wondering (again) if doing set_passthrough(),
set_syncable() etc. wouldn't be nicer and more straight-forward after all.

-- 
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.


More information about the gstreamer-bugs mailing list