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

Semantics of svn_error_t->apr_err

From: 'Daniel Shahaf' <d.s_at_daniel.shahaf.name>
Date: Tue, 9 Jul 2013 23:40:37 +0300

Bert Huijben wrote on Tue, Jul 09, 2013 at 22:32:00 +0200:
> > -----Original Message-----
> > From: Daniel Shahaf [mailto:d.s_at_daniel.shahaf.name]
> > Bert Huijben wrote on Tue, Jul 09, 2013 at 20:11:57 +0200:
> > > How is this going to help?
> >
> > I already told you how: it is going to help because API users can't do
> > anything with the value 120171 that they presently receive. The
> > outermost error must be defined by APR or by Subversion. 120171 is
> > neither.
>
> *Why* ?
>
> There is no rule that apr_err must be set to something that is defined by
> APR or Subversion.
>

I've nothing new to say. I think every svn_error_t we return from our
APIs should have an APR_ERR member which is either:

- equal to SVN_WARNING;
or
- in some SVN_ERR_CATEGORY_*_START, in the sense defined by
  SVN_ERROR_IN_CATEGORY();
or
- outside the [APR_OS_START_USERERR, APR_OS_START_USERERR+500000]
  range, *and* valid according to APR's concept of validity for
  apr_status_t values outside that range (which in particular means:
  be non-zero).

Can someone please weigh in on this issue?
Received on 2013-07-09 22:41:12 CEST

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