William A. Rowe, Jr. wrote:
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.
It doesn't matter anyways - those aren't apr_ functions, their value of zero is particular to Netware semantics.
