[gstreamer-bugs] [Bug 625396] gst_debug_remove_log_function doesn't remove default log handler

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #10 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-10-08 17:21:31 UTC ---
Anyone got any comments on this? (deprecation + useless destroy notify
callback)

>     API: deprecate gst_debug_add_log_function()
>     API: deprecate gst_debug_remove_log_function()
>     API: deprecate gst_debug_remove_log_function_by_data()
> 
> Just like _add_log_function() we also don't allow for a DestroyNotify callback
> in _set_log_function(), because we want to avoid locking in the log handler
> code path, and I don't see how we can trivially implement this without (doesn't
> seem worth putting effort into more sophisticated approaches just for this
> either IMHO). We could of course just add the DestroyNotify callback to the
> API, but document that it will never be called at the moment. Not sure if that
> makes things better for bindings / gobject-introspection or if it doesn't
> really matter.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.