The initial thought was you might have LDAP success, OS status success, and possibly multiple return codes that were considered successes.
Nothing was ever done with this.
What about the win32 definition of the macro:
#define APR_STATUS_IS_SUCCESS(s) ((s) == APR_SUCCESS \
|| (s) == APR_OS_START_SYSERR + ERROR_SUCCESS)If you just compare against APR_SUCCESS won't you miss the ERROR_SUCCESS part there?
-garrett
