Re: [Spice-devel] [PATCH 13/15] worker: simplify RedCompressBuf

2015-11-09 Thread Marc-André Lureau
Hi - Original Message - > > > > On Mon, Nov 9, 2015 at 3:03 PM, Frediano Ziglio wrote: > > > No, it's from Chiristophe > > > > ah? > > > > http://cgit.freedesktop.org/~alon/spice/commit/?h=wip/replay&id=1531f5935dfdc6e54b6fc8c99676869c1c0f808d > > > > That's weird > > http://cgit.fr

Re: [Spice-devel] [PATCH 13/15] worker: simplify RedCompressBuf

2015-11-09 Thread Frediano Ziglio
> > On Mon, Nov 9, 2015 at 3:03 PM, Frediano Ziglio wrote: > > No, it's from Chiristophe > > ah? > > http://cgit.freedesktop.org/~alon/spice/commit/?h=wip/replay&id=1531f5935dfdc6e54b6fc8c99676869c1c0f808d > That's weird http://cgit.freedesktop.org/~fziglio/spice-server/commit/?h=refactory&i

Re: [Spice-devel] [PATCH 13/15] worker: simplify RedCompressBuf

2015-11-09 Thread Marc-André Lureau
On Mon, Nov 9, 2015 at 3:03 PM, Frediano Ziglio wrote: > No, it's from Chiristophe ah? http://cgit.freedesktop.org/~alon/spice/commit/?h=wip/replay&id=1531f5935dfdc6e54b6fc8c99676869c1c0f808d -- Marc-André Lureau ___ Spice-devel mailing list Spice-de

Re: [Spice-devel] [PATCH 13/15] worker: simplify RedCompressBuf

2015-11-09 Thread Frediano Ziglio
> > hi > > Although I don't remember writing this, it seems git history attributes this > to me. I guess a "based on initial patch from" could be useful some day. > No, it's from Chiristophe Frediano > > - Original Message - > > From: Christophe Fergeau > > > > Make sure an allocat

Re: [Spice-devel] [PATCH 13/15] worker: simplify RedCompressBuf

2015-11-09 Thread Marc-André Lureau
hi Although I don't remember writing this, it seems git history attributes this to me. I guess a "based on initial patch from" could be useful some day. - Original Message - > From: Christophe Fergeau > > Make sure an allocated buffer is correctly referenced by the marshaller, > and c

[Spice-devel] [PATCH 13/15] worker: simplify RedCompressBuf

2015-11-09 Thread Frediano Ziglio
From: Christophe Fergeau Make sure an allocated buffer is correctly referenced by the marshaller, and can't be free and reused by mistake. Simplify the code by using GSlice --- server/display-channel.h | 3 -- server/red_worker.c | 117 --- 2 fi