[gstreamer-bugs] [Bug 599870] [API] micro-optimisation: add GST_DEBUG_FUNCPTR_FULL

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Nov 6 09:02:15 PST 2009


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

--- Comment #12 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2009-11-06 17:02:13 UTC ---
> > Maybe we could adjust the macro to be a G_STMT with the volatile inside.
> 
> That's probably the best solution... any reason not to do that? :)

I didn't know it was possible to do that in a portable way? (afaik this would
rely on gcc extensions)

(Also, while it's cleaner API wise it's really a bit ugly under the hood, with
it adding 5-20 static booleans per function where just a single one would do
just as well in practice...)

FWIW, my plan was to just put the macro into gstprivate.h for now and commit
the gst/*pad* bits, which takes care of 90% of the calls and presumably
contention already, unless we can make the macro suggestion work in a portable
way.

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