Re: [Spice-devel] [spice v15 02/21] streaming: Remove the width/height encode_frame() parameters

2016-05-31 Thread Christophe Fergeau
Hi, On Mon, May 30, 2016 at 07:12:48PM +0200, Francois Gouget wrote: > On Fri, 27 May 2016, Christophe Fergeau wrote: > > I'm afraid this bit is not going to work as expected with older clients > > (ones not supporting SPICE_DISPLAY_CAP_SIZED_STREAM). > > This chunk does not change whether we

Re: [Spice-devel] [spice v15 02/21] streaming: Remove the width/height encode_frame() parameters

2016-05-30 Thread Francois Gouget
On Fri, 27 May 2016, Christophe Fergeau wrote: > Hey, > > On Thu, May 26, 2016 at 05:15:56PM +0200, Francois Gouget wrote: > > encode_frame() needs the QXL_DRAW_COPY operation's SpiceCopy.src_area > > field anyway, so the width and height parameters were redundant. > > > > Signed-off-by:

Re: [Spice-devel] [spice v15 02/21] streaming: Remove the width/height encode_frame() parameters

2016-05-27 Thread Christophe Fergeau
Hey, On Thu, May 26, 2016 at 05:15:56PM +0200, Francois Gouget wrote: > encode_frame() needs the QXL_DRAW_COPY operation's SpiceCopy.src_area > field anyway, so the width and height parameters were redundant. > > Signed-off-by: Francois Gouget > --- > server/dcc-send.c

[Spice-devel] [spice v15 02/21] streaming: Remove the width/height encode_frame() parameters

2016-05-26 Thread Francois Gouget
encode_frame() needs the QXL_DRAW_COPY operation's SpiceCopy.src_area field anyway, so the width and height parameters were redundant. Signed-off-by: Francois Gouget --- server/dcc-send.c | 26 -- server/mjpeg-encoder.c | 1 -