[Bug 613081] [API] add GST_LOG_CAPS etc.

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue May 24 09:35:29 PDT 2011


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

--- Comment #6 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-05-24 16:35:22 UTC ---
> Can't we just have a *big* static array of *big* string buffers, a counter and
> a couple of functions like GST_DEBUG_FORMAT_CAPS(caps).

That's possible, and could probably even be done without the artificial
limitation using GPrivate or GStaticPrivate or so), but I think I'd still
prefer dedicated API for this, for at least two reasons:

 - dedicated API (macros) makes this much more
   discoverable than GST_PTR_FORMAT or
   GST_DEBUG_FORMAT_CAPS(). A counter-argument
   to this would be the fact that many newbies just
   do GST_LOG("%s", gst_caps_to_string (caps)), so
   offering something along those lines, maybe a
   gst_caps_to_static_string() could be seen as
   more intuitive.

 - we log a lot of caps. This is really frequently
   used. It would be nice to have nice API for that,
   so the code looks nice. IMHO GST_FORMAT_TIME /
   GST_TIME_ARGS() often make things quite ugly,
   would be nice to avoid adding more of those.


> Any progress?

Not really, still not sure about the right API yet I think.

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