[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Greg Stein <gstein_at_lyra.org>
Date: 2000-11-07 02:09:34 CET

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


Greg Stein, http://www.lyra.org/
Received on Sat Oct 21 14:36:14 2006

This is an archived mail posted to the Subversion Dev mailing list.