>
> On Mon, Nov 14, 2016 at 03:11:31PM +0000, Frediano Ziglio wrote:
> > The verify macro used currently has some problem
> > as it raise a warning in RHEL6.
> > As suggested in verify.h use verify_expr macro
> > to avoid the warning.
>
>
>
> > diff --git a/server/red-common.h b/server/red-common.h
> > index 190fd9c..a457ed8 100644
> > --- a/server/red-common.h
> > +++ b/server/red-common.h
> > @@ -80,4 +80,6 @@ typedef struct GListIter {
> > #define GLIST_FOREACH_REVERSED(_list, _iter, _type, _data) \
> > GLIST_FOREACH_GENERIC(_list, _iter, _type, _data, prev)
> >
> > +#define SPICE_VERIFY(cond) verify_expr(cond, (void)0)
> > +
> > #endif
>
> Bit of a nitpick here, but I'd return 1 rather than 0, and I'd put the
> macro somewhere in spice-common as verify.h is there.
>
Well... as it's converted to void it does not change much but
I'll do.
About the position of the macro I didn't have a specific place to put it.
Surely not in verify.h as copied verbatim in spice-common.
Probably the best place is macros.h although will requite to include
verify.h.
> Looks good otherwise.
>
> Christophe
>
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel