On Thu, May 16, 2002 at 08:11:03AM -0700, Brian Pane wrote: >... > My vote presumes, by the way, that the removal is really > a backward-compatible deprecation: > > * re#define APLOG_NOERRNO to zero for source compatibility > * mask out the bit that used to be APLOG_NOERRNO within > ap_log_*error() for binary compatibility, unless other > changes force an MMN major increment in the same release.
If we want Apache to follow APR's versioning model(*), then you shouldn't redefine a symbolic constant's value. Just mark it as deprecated, and toss the symbol in the next major release. In particular, it might be possible for a third-party module to use APLOG_NOERRNO and want to test for it. It doesn't hurt us to leave the value in there. Cheers, -g (*) http://apr.apache.org/versioning.html -- Greg Stein, http://www.lyra.org/
