[Bug 735195] bus: Missing API to remove a bus watch

[GStreamer (bugzilla.gnome.org)]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEEDINFO

--- Comment #9 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2014-08-23 11:35:42 UTC ---
Olivier, g_main_context_get_thread_default() is widely used in gio, and was
added for it. (Anyway...)

Linus, does this already fix your issues?

commit b1f23598099da82835df60bc2fb09501233f9118
Author: Philippe Normand <philn at igalia.com>
Date:   Wed Aug 13 14:12:00 2014 +0200

    bus: destroy signal watch from the context it was mapped to

    Don't rely on g_source_remove() because it operates on the main
    context. If a signal watch was added to a new thread-default context
    g_source_remove() would have no effect. So simply use
    g_source_destroy() to avoid this problem.

    Additionally the source_id was removed from GstBusPrivate because it
    was redundant with the signal watch GSource also stored in that
    structure.

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

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