Lucian Adrian Grijincu wrote: > Just removing some unused code. > APR_SUCCESS is deemed to be zero as I understand it, so no use in > checking for it explicitly. Untrue, because it makes things clearer to the reader. The compiler can't care less if you compare (rv == 0) or (!rv), it optimizes it the same way. Bill