Re: EINVAL vs APR_EINVAL in apr_xlate_open()

On Thu, Jul 18, 2002 at 05:04:33PM -0500, Karl Fogel wrote:
> Well, this is *us* generating the EINVAL by fiat, not getting it from
> some lower-down system call.
>
> Maybe we should still go with APR_EINVAL, on the principle of least
> surprise, though. I'm not sure; trusting people with more APR
> experience to know a good answer here.

Right, the system is free to use EINVAL. APR should not produce
EINVAL itself.

I'm assuming you mean this code fragment in xlate.c:

return rv ? rv : EINVAL;

By all means that should be:

return rv ? rv : APR_EINVAL;

HTH. -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jul 19 00:20:46 2002