apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cliff Woolley <jwool...@virginia.edu>
Subject Re: cvs commit: apr/include/arch/win32 fileio.h
Date Mon, 25 Mar 2002 20:16:00 GMT
On 25 Mar 2002 wrowe@apache.org wrote:

>     Not certain this patch is strictly necessary, but I recall some abnormal
>     behavior that was fixed at the time I changed this code.
>
>    #define IS_FNCHAR(c) (apr_c_is_fnchar[(unsigned char)(c)] & 1)
>   -#define IS_SHCHAR(c) (apr_c_is_fnchar[(unsigned char)(c)] & 2 == 2)
>   +#define IS_SHCHAR(c) ((apr_c_is_fnchar[(unsigned char)(c)] & 2) == 2)

It *is* necessary... == has a higher precedence than & does.  Good catch.

--Cliff

--------------------------------------------------------------
   Cliff Woolley
   cliffwoolley@yahoo.com
   Charlottesville, VA



Mime
View raw message