[Bug 734716] bus: signal watched added in a new thread-default context can't be removed

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #2 from Philippe Normand <phil at base-art.net> 2014-08-13 11:38:43 UTC ---
(In reply to comment #1)
> (In reply to comment #0)
> 
> > I think the solution would be to get the thread-default context in
> > _remove_signal_watch(), look for the source in that context and destroy it.
> > I'll post a patch :)
> 
> No, you'll have to remember the main context that was the default when the
> watch was added :)

Oh right, in my code I remove the signal watch before popping the
thread-default context but we can't assume everyone would do that :)

> To get around this I just called g_source_destroy() in the
> past.

Alright, do you know why that code was changed?

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