On Mon, 2013-11-11 at 11:32 +0200, Luiz Augusto von Dentz wrote: > Hi Bastien, > > On Sat, Nov 9, 2013 at 7:04 PM, Bastien Nocera <hadess@xxxxxxxxxx> wrote: > > On Sat, 2013-11-09 at 18:02 +0100, Bastien Nocera wrote: > >> When we return FALSE from idle handlers, the source is removed. > >> This will be causing warnings in glib 2.40. > >> > >> See https://bugzilla.gnome.org/show_bug.cgi?id=710724 > > > > Note that there's plenty of other places where this should be fixed. > > I'll leave those up to somebody who doesn't get RSI ;) > > Btw, did you really got a problem or was just by looking at the code? > process_changes always call remove_pending so perhaps the problem was > just in generic_unregister, so the change to process_changes was not > really necessary. I really had the problem, and the patch fixed it. Resetting the id for idle callbacks and timeouts needs to happen at the end of each callback function, or you might be trying to remove an invalid source ID. -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
