httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: cvs commit: apache-2.0/src/main http_protocol.c
Date Sun, 08 Oct 2000 16:50:31 GMT
rbb@locus.apache.org wrote:
> 
>                length = toss - temp + 1;
>   -            e->split(e, length + (temp[length] == 0));
>   +            e->split(e, length + (temp[length] == '\0' ? 1 : 0));
>                apr_cpystrn(pos, temp, length + 1);
>    	    

certainly the above is clearer, but couldn't that reduce to

	e->split(e, length + (temp[length] == '\0'));

By ANSI definition, the results of '==' are either 1 or 0, so
the '? 1 : 0' is redundant...

Or maybe I'm missing something :)
-- 
===========================================================================
   Jim Jagielski   [|]   jim@jaguNET.com   [|]   http://www.jaguNET.com/
                "Are you suggesting coconuts migrate??"

Mime
View raw message