[Spice-devel] [spice-server v2 2/3] tests: Port test-qxl-parsing to GTest

Christophe Fergeau cfergeau at redhat.com
Tue Mar 21 13:46:52 UTC 2017


On Mon, Mar 20, 2017 at 02:05:13PM -0400, Frediano Ziglio wrote:
> > 
> > On Fri, Mar 17, 2017 at 02:10:22PM +0100, Pavel Grunt wrote:
> > > > +
> > > > +static void deinit_qxl_surface(QXLSurfaceCmd *qxl)
> > > > +{
> > > > +    free((void *)qxl->u.surface_create.data);
> > > 
> > > `make syntax-check` is failing because of this
> > 
> > Ah, thanks, it also fails after this patch because of the #include
> > <assert.h>. I've removed the cast and the include.
> > 
> > However the @VALGRIND_CHECK_RULES@ added in the next patch also breaks
> > make syntax-check, I'll address this too :)
> > 
> > Christophe
> > 
> 
> Sorry, was going to say, this line also will fail due to a
> warning using 32 bit, should be
> 
>    free((void *)(uintptr_t)qxl->u.surface_create.data);
> 
> or even define a from_physical (like to_physical) even to avoid
> the syntax-check problem

Yes, I went with a helper function 

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170321/df3c85e7/attachment.sig>


More information about the Spice-devel mailing list