r2272 - in trunk/varnish-cache: bin/varnishd include

Poul-Henning Kamp phk at phk.freebsd.dk
Tue Nov 20 16:02:30 CET 2007


In message <20071120145851.9EB551EC2B4 at projects.linpro.no>, des at projects.linpro
.no writes:

>Modified:
>   trunk/varnish-cache/bin/varnishd/cache.h
>   trunk/varnish-cache/include/libvarnish.h
>Log:
>Many of our assertions have side effects, so the condition needs to be
>evaluated even if we don't bother to check the result.  We should trust
>the compiler to eliminate code that has no effect.

Are we going to allow the asserts to be compiled out ?

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.



More information about the varnish-commit mailing list