[Bug 757046] tracer: [API] Add factory convenience functions.

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sat Oct 24 20:02:54 PDT 2015


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

--- Comment #8 from Stefan Sauer (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> ---
Review of attachment 314003:
 --> (https://bugzilla.gnome.org/review?bug=757046&attachment=314003)

The reason I did no do this, is that the tracing subsystem should be
initialized *before* pipelines run in a sequential fashion. This way the
registration data is immutable when the tracers run and hence I can avoid
expensive locking. Now you are exposing this to everyone.
Why would you need to create tracers from GstValidate instead of cosuming the
trace output? What we *can* do is to rewrite _priv_gst_tracing_init and expose
api to pass the tracer initialisation from a 'spec', so that you can call this
instead of relying on the env-var. These _init() functions can ensure that
things are only initilaized once.

::: gst/gsttracer.h
@@ +77,3 @@
+GstTracer *        gst_tracer_factory_make              (const gchar *, const
gchar *);
+GstTracer *        gst_tracer_factory_create            (GstTracerFactory *
factory, const gchar * params);
+GstTracerFactory * gst_tracer_factory_find              (const gchar * name);

this should be in tracerutils.h then, right?

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