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

Greg Stein <gstein@lyra.org> writes:
> > + else if (apr_err == APR_ENOENT) /* which is better, this test... */
> > + *kind = svn_invalid_kind;
> > + else if (finfo.filetype == APR_NOFILE) /* ... or this one? */
> > + *kind = svn_invalid_kind;
>
> Not obvious from apr_file_io.h, but APR_NOFILE is APR's way of saying
> "something is there, but I have no idea what it is."
>
> APR_ENOENT means that nothing was there.

Interesting.

I'll leave both tests, then; that gets the most useful result for
Subversion.
Received on Sat Oct 21 14:36:11 2006