[Bug 757047] tracer: [API]: add register_hook_for_target.

[GStreamer (GNOME Bugzilla)]

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

--- Comment #3 from Thibault Saunier <tsaunier at gnome.org> ---
(In reply to Sebastian Dröge (slomo) from comment #2)
> Review of attachment 314005 [details] [review]:
> 
> Just out of curiosity, do you have a tracer that uses this? What does it do?
> :)

Yes, we have one tracer per pad as GstValidatePadMonitor (and so on for other
components).

> ::: gst/gsttracerutils.c
> @@ +286,3 @@
>  
> +static void
> +gst_tracing_register_hook_full (GstTracer * tracer, GQuark detail,
> 
> Should this one be public then maybe, and be called
> gst_tracing_register_hook() without the _full?

We already have;

void 
gst_tracing_register_hook (GstTracer * tracer, const gchar * detail, 
     GCallback func) 

I do not think that needs to be public.

> @@ -291,3 @@
> - * @func: (scope async): the callback
> - *
> - * Register @func to be called when the trace hook @detail is getting
> invoked.
> 
> Since: 1.8 everywhere

The Tracers are Since 1.8, I thinking adding it once is enough no?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.