Re: CVS update: subversion/subversion/libsvn_subr xml.c

On Tue, Nov 07, 2000 at 01:41:40AM +0100, Branko Cibej wrote:
>...
> * Change all tests of APR status codes (except for tests for
> APR_SUCCESS) to use APR_STATUS_IS_blabla() macros. This is
> necessary for Win32, because there are actually several different
> possible codes for some kinds of errors. The APR macros take care
> of that, and should be portable. This should also be the Right Way
> to write such tests in future.

The patch looks good. The APR_STATUS_IS_* macros are actually quite recent,
so using them now is a Good Thing.

>...
> * Two fixes in APR file_io code (already mentioned in another post),
> which Greg Stein needs to review and approve.

Will do. Thx

Cheers,
-g

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