> On Aug 11, 2015, at 15:34 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > > On Tue, Aug 11, 2015 at 02:22:54PM +0200, Christophe Fergeau wrote: >> Hey, >> >> Not clear at all why this is needed, even when looking at the code in >> its final state (after all patches are applied). > > Ah, got it actually, it's because of > g_simple_async_result_run_in_thread(simple, > spice_usbredir_channel_disconnect_device, > G_PRIORITY_DEFAULT, > cancellable); > in the next patch. I think I'd just introduce a wrapper function in the > next patch rather than making all direct _disconnect_device() callers > uglier. Agree. Fixing this... > > Christohpe _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel