apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Fritsch ...@sfritsch.de>
Subject Re: svn commit: r1240475 - in /apr/apr/branches/1.4.x: ./ include/apr_errno.h
Date Sat, 04 Feb 2012 17:11:08 GMT
On Saturday 04 February 2012, Graham Leggett wrote:
> On 04 Feb 2012, at 11:47 AM, sf@apache.org wrote:
> > URL: http://svn.apache.org/viewvc?rev=1240475&view=rev
> > Log:
> > Reserve a range of USERERR values for HTTPD
> > 
> > --- apr/apr/branches/1.4.x/include/apr_errno.h (original)
> > +++ apr/apr/branches/1.4.x/include/apr_errno.h Sat Feb  4
> > 09:47:57 2012 @@ -158,6 +158,8 @@ APR_DECLARE(char *)
> > apr_strerror(apr_sta
> > 
> >  * Subversion - Defined ranges, of less than 100, at intervals of
> >  5000 *              starting at an offset of 5000, e.g.
> >  *               +5000 to 5100,  +10000 to 10100
> > 
> > + *
> > + * Apache HTTPD - +2000 to 2999
> > 
> >  */
> > 
> > #define APR_OS_START_USERERR    (APR_OS_START_STATUS +
> > APR_OS_ERRSPACE_SIZE) /**
> 
> Would it be possible to have this as "reserved for the
> application", rather than "reserved for httpd"?
> 
> Ideally APR shouldn't have any httpd specific code in it, however
> this error range would be useful for other external apps,
> including httpd.

Read the full comment in the file (the diff context is too small). 
There is a huge range of error codes reserved for the application. Out 
of this range, subversion already had some ranges reserved. Now HTTPD 
has some other range reserved. There is still plenty of space for 
other applications.

Mime
View raw message