r753 - trunk/varnish-cache

Poul-Henning Kamp phk at phk.freebsd.dk
Tue Aug 8 08:06:38 CEST 2006


In message <ujrhd0n93lm.fsf at cat.linpro.no>, Dag-Erling =?iso-8859-1?Q?Sm=F8rgra
v?= writes:
>phk at projects.linpro.no writes:
>> Log:
>> Add -Wformat and remove -fno-inline which disables it.
>>
>>
>> Modified: trunk/varnish-cache/configure.ac
>> ===================================================================
>> --- trunk/varnish-cache/configure.ac	2006-08-07 21:01:09 UTC (rev 752)
>> +++ trunk/varnish-cache/configure.ac	2006-08-07 21:01:40 UTC (rev 753)
>> @@ -21,7 +21,7 @@
>>  	CFLAGS="${CFLAGS} ${DEVELOPER_CFLAGS}")
>>  AC_ARG_ENABLE(debugging-symbols,
>>  	AS_HELP_STRING([--enable-debugging-symbols],[enable debugging symbols (default is NO)]),
>> -	CFLAGS="${CFLAGS} -O -g -fno-inline -fno-builtin")
>> +	CFLAGS="${CFLAGS} -O -g -fno-inline")
>>  AC_ARG_ENABLE(werror,
>>  	AS_HELP_STRING([--enable-werror],[use -Werror (default is NO)]),
>>  	CFLAGS="${CFLAGS} -Werror")
>
>The log message and the patch don't seem to agree...

Right, I meant -fno-builtin in the message too.  Without built-ins
it doesn't get to the code which checks printf args.

-- 
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