[Spice-devel] [PATCH spice-server] Use desired image compression for the first image
Pavel Grunt
pgrunt at redhat.com
Mon Jul 13 02:57:10 PDT 2015
On Fri, 2015-06-26 at 16:31 +0200, Pavel Grunt wrote:
> On Fri, 2015-06-26 at 15:45 +0200, Christophe Fergeau wrote:
> > On Thu, Jun 25, 2015 at 01:28:42PM +0200, Pavel Grunt wrote:
> > > Hi Christophe,
> > >
> > > On Mon, 2015-06-22 at 14:20 +0200, Christophe Fergeau wrote:
> > > > Hey,
> > > >
> > > > Seems fine, though it's not clear to me why this would only impact the
> > > > first image?
> > > It has impact on the first image because other images are compressed by
> > > red_compress_image(). Maybe the mechanism of choosing the compression
> > > should
> > > be
> > > the same for both functions (or have just one function for compression ?)
> > > -
> > > I
> > > will look into it.
> >
> > Ah indeed, I was wondering what's up with red_compress_image VS
> > red_marshall_image (_marshall_image seems to be used a bit more than
> > first image, but yeah, seldomly used). Feel free to push this, maybe
> > with a mention of red_compress_image() in the commit log.
> >
> Thank you Christophe, I am going to send a patch which replaces the "mechanism
> of choosing the compression" by calling red_compress_image() before/instead of
> pushing this now.
I just pushed this patch.
Pavel
More information about the Spice-devel
mailing list