Re: spice-server, logging and style

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

 



Hi

----- Original Message -----
> On Fri, Nov 20, 2015 at 01:01:30PM -0500, Frediano Ziglio wrote:
> > > 
> > > On Tue, Nov 03, 2015 at 08:15:14AM -0500, Frediano Ziglio wrote:
> > > > Problems:
> > > > - the Spice_style.odt is outdated;
> > > > - spice_assert recall the standard assert but instead it mainly
> > > >   call spice_error which like GLib is fatal and should call exit/abort.
> > > >   It is not affected by NDEBUG or other definitions. Why using it
> > > >   instead
> > > >   of spice_error?
> > > 
> > > Note that spice-common has verify.h (from gnulib) which can be used for
> > > truly static checks. I did not notice that there is a
> > > spice_static_assert before ;)
> > > 
> > > Christophe
> > > 
> > 
> > I think you are confusing static assertions here with dynamic ones.
> 
> verify.h does compile-time assertions, which is what I call 'static
> assertions':
> 
> lz_common.h:62:1: erreur: static assertion failed: "verify (1 == 0)"
>  verify(1 == 0);

so does G_STATIC_ASSERT(), I guess we could remove verify.h from source tree today.
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]