r2180 - trunk/varnish-cache/lib/libvarnishapi

Poul-Henning Kamp phk at phk.freebsd.dk
Mon Oct 29 16:28:07 CET 2007


In message <ujr1wbeasnb.fsf at false.linpro.no>, =?iso-8859-1?Q?Dag-Erling_Smørg
rav?= writes:
>phk at projects.linpro.no writes:
>> Log:
>> Mask negative characters to 8 bit before formatting.
>
>A cast would be more appropriate...  and we should probably use
>isprint() instead of hardcoding (*ptr >= ' ' && *ptr <= '~').

A cast we can discuss, but isprint() would be wrong because it would
take the locale into consideration and HTTP is specifically specified
in ASCII with no extension.

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