[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