Re: [Spice-devel] [PATCH 2/8] Simplify send_free_list

2016-09-20 Thread Uri Lublin
On 09/19/2016 03:42 PM, Frediano Ziglio wrote: What's the simplification -- always send two headers and possibly have the wait_m one empty ? The check for free_list->wait.header.wait_count was already done some lines above so I removed as always true. Ack. ___

Re: [Spice-devel] [PATCH 2/8] Simplify send_free_list

2016-09-19 Thread Frediano Ziglio
> > What's the simplification -- always send two headers and > possibly have the wait_m one empty ? > > The check for free_list->wait.header.wait_count was already done some lines above so I removed as always true. Frediano > > On 09/19/2016 11:29 AM, Frediano Ziglio wrote: > > Signed-off-by

Re: [Spice-devel] [PATCH 2/8] Simplify send_free_list

2016-09-19 Thread Uri Lublin
What's the simplification -- always send two headers and possibly have the wait_m one empty ? On 09/19/2016 11:29 AM, Frediano Ziglio wrote: Signed-off-by: Frediano Ziglio --- server/dcc-send.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/server/dc

[Spice-devel] [PATCH 2/8] Simplify send_free_list

2016-09-19 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/dcc-send.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/server/dcc-send.c b/server/dcc-send.c index 2215f69..ab6edc2 100644 --- a/server/dcc-send.c +++ b/server/dcc-send.c @@ -263,9 +263,9 @@ static void send_fr