Re: [Spice-devel] [PATCH 01/30] Do not release too much drawables

2016-06-08 Thread Frediano Ziglio
> > I have to admit that I'm not very familiar with this bit of the code. But > it's > clearly a behavior change and it seems like one that could have non-obvious > or > hard-to-test consequences. Can you give a bit more justification for the > change? > Does it fix an observed issue? It seems tha

Re: [Spice-devel] [PATCH 01/30] Do not release too much drawables

2016-06-07 Thread Jonathon Jongsma
I have to admit that I'm not very familiar with this bit of the code. But it's clearly a behavior change and it seems like one that could have non-obvious or hard-to-test consequences. Can you give a bit more justification for the change? Does it fix an observed issue? It seems that these glz drawa

[Spice-devel] [PATCH 01/30] Do not release too much drawables

2016-06-07 Thread Frediano Ziglio
Accumulate counter, do not reset for each client. Signed-off-by: Frediano Ziglio --- server/dcc-encoders.c| 5 ++--- server/dcc-encoders.h| 3 ++- server/display-channel.c | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/server/dcc-encoders.c b/server/dcc-encoders.c