[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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.33

--- Comment #25 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-02-03 11:14:55 UTC ---
This is a KISS solution, hope it makes things work now. Please re-open the bug
if it still doesn't work:

commit 609a75eae28c770372ef1464d1e03fadf03982ea
Author: Tim-Philipp Müller <tim.muller at collabora.co.uk>
Date:   Thu Feb 3 10:53:27 2011 +0000

    info: make adding/removing of gst_debug_log_default() work properly

    Make adding/removing gst_debug_log_default() work reliably in all
    circumstances. The problem was that depending on platform and linker
    flags the function argument might resolve to different addresses,
    which made it impossible to remove the default log function added
    in gst_init() from application code (because the pointer values
    didn't match). The new approach should keep things simple by passing
    NULL for the default function, which the code in libgstreamer can
    then handle.

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

commit 74ff936752fdb8715bcafb7560726fbe4d0438ae
Author: Tim-Philipp Müller <tim.muller at collabora.co.uk>
Date:   Thu Feb 3 10:28:01 2011 +0000

    Revert "info: use the publicly visible address to fix the tests"

    This reverts commit eb56687a6dfd207507a4ca000eae53f93b5e33ea.

    While this commit may have fixed a problem on one of the build bots,
    it didn't actually fix the original bug reported for win32.

    Also, it causes other problems, such as the lookup failing when
    called from C++ code (gst-phonon, amarok).

    This needs to be fixed differently.

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

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