[Bug 733187] integrating the tracer branch

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Jan 7 14:13:44 PST 2016


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

--- Comment #39 from Stefan Sauer (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> ---
Docs are now added:
commit 8968af3589465ad22a2e31464fef0d66e5d070d1
Author: Stefan Sauer <ensonic at users.sf.net>
Date:   Thu Jan 7 19:13:03 2016 +0100

    tracerutils: document the tracer hook functions

    Document all tracer hook function pointer together with the detail string
that
    one needs to use with gst_tracing_register_hook().

Marcin, I also cleaned the query hooks up - you'll need to update your tracer.

commit 2f41e7bc6a842044cb73bc0470601200575c378a
Author: Stefan Sauer <ensonic at users.sf.net>
Date:   Thu Jan 7 23:03:48 2016 +0100

    tracer: harmonize the query hooks

    In post hooks always pass the return value as the last param. Pass the
query
    also to post hooks since it is still alive.

Marcin, and another question for the ELEMENT_NEW we discussed why we add now
pre/post, but for element-{add,remove}-pad we did not. Any thoughts.

Finally for all. The hook-details are called e.g. "pad-push-pre" or
"pad-push-post". It is essentially the method name without the "gst_" prefix,
with '_' changed to "-" and '-pre'/'-post' added. I wonder if "pad-push:pre" or
"pad-push:post" would look better?

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