Re: [Spice-devel] [spice-gtk v3] vmcstream, gtask: Do idle ourself instead of leaving it to GTask's heuristic

2016-03-22 Thread Victor Toso
Hi, On Tue, Mar 22, 2016 at 10:33:56PM +0100, Fabiano Fidêncio wrote: > Seems that GTask heuristic only makes sense in a non-coroutine case. > After opening a bug[0] on spice-gtk and a few discussions in the mailing [0] ? > list, seems that is completely fine for coroutine code to deal with the

Re: [Spice-devel] [spice-gtk v3] vmcstream, gtask: Do idle ourself instead of leaving it to GTask's heuristic

2016-03-22 Thread Fabiano Fidêncio
Pushed, thanks! On Tue, Mar 22, 2016 at 11:01 PM, Fabiano Fidêncio wrote: > On Tue, Mar 22, 2016 at 10:57 PM, Marc-André Lureau > wrote: >> ack >> >> - Original Message - >>> Seems that GTask heuristic only makes sense in a non-coroutine case. >>> After opening a bug[0] on spice-gtk and

Re: [Spice-devel] [spice-gtk v3] vmcstream, gtask: Do idle ourself instead of leaving it to GTask's heuristic

2016-03-22 Thread Fabiano Fidêncio
On Tue, Mar 22, 2016 at 10:57 PM, Marc-André Lureau wrote: > ack > > - Original Message - >> Seems that GTask heuristic only makes sense in a non-coroutine case. >> After opening a bug[0] on spice-gtk and a few discussions in the mailing >> list, seems that is completely fine for coroutine

Re: [Spice-devel] [spice-gtk v3] vmcstream, gtask: Do idle ourself instead of leaving it to GTask's heuristic

2016-03-22 Thread Marc-André Lureau
ack - Original Message - > Seems that GTask heuristic only makes sense in a non-coroutine case. > After opening a bug[0] on spice-gtk and a few discussions in the mailing > list, seems that is completely fine for coroutine code to deal with the > idle explicitly. > > Signed-off-by: Fabian

[Spice-devel] [spice-gtk v3] vmcstream, gtask: Do idle ourself instead of leaving it to GTask's heuristic

2016-03-22 Thread Fabiano Fidêncio
Seems that GTask heuristic only makes sense in a non-coroutine case. After opening a bug[0] on spice-gtk and a few discussions in the mailing list, seems that is completely fine for coroutine code to deal with the idle explicitly. Signed-off-by: Fabiano Fidêncio --- Changes since v2, as per Marc-