Re: [Spice-devel] [PATCH 03/10] worker: move stream to display channel

2015-11-10 Thread Frediano Ziglio
> NACK > > There's a lot of stuff going on in this patch, but I think it would > be hard to split out (other than a few minor include changes I noted > below). So I'm OK with that part. But there seems to be one behavior > change that I've noted below. > > Agree that code change (if stream one)

Re: [Spice-devel] [PATCH 03/10] worker: move stream to display channel

2015-11-06 Thread Jonathon Jongsma
NACK There's a lot of stuff going on in this patch, but I think it would be hard to split out (other than a few minor include changes I noted below). So I'm OK with that part. But there seems to be one behavior change that I've noted below. On Fri, 2015-11-06 at 11:01 +, Frediano Ziglio wro

[Spice-devel] [PATCH 03/10] worker: move stream to display channel

2015-11-06 Thread Frediano Ziglio
From: Marc-André Lureau --- server/Makefile.am | 2 + server/display-channel.c | 44 +++ server/display-channel.h | 82 ++--- server/red_worker.c | 916 --- server/red_worker.h | 1 + server/stream.c | 66 server/st