Re: CVS update: subversion/subversion/libsvn_wc/tests checkout-test.c

On Tue, Nov 07, 2000 at 07:26:58AM -0600, Ben Collins-Sussman wrote:
> gstein@tigris.org writes:
> > Log:
> > don't use apr_canonical_error() -- its continued existence is questionable
>
> Why?

The APR_STATUS_IS_*() macros are intended to replace uses of
apr_canonical_error(). What was happening is that canonical_error had this
huge switching, which was growing and growing. From an efficiency
standpoint, the macros are much better.

While I can't say that apr_canonical_error is definitely going away, I know
that every single usage has been wiped from Apache. IOW, it certainly has no
value right now, and it will probably suffer bit-rot over time (so it should
just go away now).

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/