Re: [common] canvas: canvas_fix_alignment() is unused so remove it.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2016-04-19 at 10:59 +0200, Francois Gouget wrote:
> On Tue, 19 Apr 2016, Pavel Grunt wrote:
> 
> > Hi Francois,
> > 
> > what error did you get? The function is used in canvas_get_lz()
> > and canvas_get_lz4().
> 
>   CC       sw-canvas.lo
> In file included from ../spice-common/common/sw_canvas.c:29:0,
>                  from sw-canvas.c:22:
> ../spice-common/common/canvas_base.c:518:13: error: 
> 'canvas_fix_alignment' defined but not used [-Werror=unused-function]
>  static void canvas_fix_alignment(uint8_t *bits,
>              ^
> cc1: all warnings being treated as errors
> 
> 
> This is with the 2eaf7c9572746bcc0edc52871974d4085618f3e5 spice-
> common 
> checkout.
> 
> Looking into this I noticed that:
>  * SW_CANVAS_CACHE is not defined anywhere in the server (all I can
> find 
>    are #ifdef checks, spice-gtk does define it however).
>  * And apparently I don't have the liblz4 development files.
> Installing 
>    those would certainly fix this issue but spice-common should
> still 
>    compile in this configuration.
> 

Yeah, but spice-common compiles fine. imo the problem lies in our usage
of server/sw-canvas.[ch]. SW_CANVAS_IMAGE_CACHE is defined but not used
anywhere. To me it looks like some leftover code from the old client
(spicec).

Thanks for the report,
Pavel




_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]