You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@apr.apache.org by Cliff Woolley <jw...@virginia.edu> on 2002/03/25 21:16:00 UTC

Re: cvs commit: apr/include/arch/win32 fileio.h

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