[Bug 785035] debug: Add a memory ringbuffer based debug logger

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Mon Jul 17 20:19:00 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=785035

--- Comment #2 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Cool stuff. As I mentioned on IRC, since we do the g_strdup_printf() anyway, we
might just as well not copy the result into a single ringbuffer allocation
area, but instead just keep a queue/list/array of allocated strings and their
length around and retire old ones if the size exceeds the allowed size.

If we can somehow use g_vasprintf() we can also avoid that strlen(), but that's
all implementation details I suppose.

Would it make sense to make the per-thread size config in kB units instead of
bytes?

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