Aaron Bannert wrote:
> p.s. why would "!!" fix bugs on some platforms if ap_exists_config_define
> can only return 0 or 1? I can only see using !! to normalize some non-bool
> return to 0/1 for some weird logic, and even then it's not readable.
As I was responsible for !!, I guess I should explain. The original
version was flag=!!getenv("SOMETHING").
I agree it isn't totally readable, and asked permission before including
it.
Without checking the code it looks to be redundant.
Cheers,
Ben.
--
http://www.apache-ssl.org/ben.html
In Boston 'til 1st July.
- [PATCH] Add back -X functionality Justin Erenkrantz
- Re: [PATCH] Add back -X functionality rbb
- Re: [PATCH] Add back -X functionality Cliff Woolley
- Re: [PATCH] Add back -X functionality David N. Welton
- Re: [PATCH] Add back -X functionality Aaron Bannert
- Re: [PATCH] Add back -X functionality Ben Laurie
- Re: [PATCH] Add back -X functionalit... William A. Rowe, Jr.
- Re: [PATCH] Add back -X functionality Justin Erenkrantz
- Re: [PATCH] Add back -X functionality rbb
- Re: [PATCH] Add back -X functionalit... Cliff Woolley
- Re: [PATCH] Add back -X functio... Bill Stoddard
- Re: [PATCH] Add back -X functionality rbb
- Re: [PATCH] Add back -X functionalit... David N. Welton
- Re: [PATCH] Add back -X functio... David N. Welton
- Re: [PATCH] Add back -X functionality dean gaudet
- Re: [PATCH] Add back -X functionality Roy T. Fielding
