Re: svn commit: rev 181 - trunk/subversion/include trunk/subversion/libsvn_subr trunk/subversion/libsvn_ra_dav

On Fri, Sep 28, 2001 at 06:11:17PM -0500, brane@tigris.org wrote:
>...
> +/* Return a POSIX-like file descriptor from FILE.
> +
> + We need this because on some platforms, notably Windows, apr_file_t
> + is not based on a file descriptor; but we have to pass an FD to neon.
> +
> + FIXME: This function will hopefully go away if/when neon gets
> + replaced by apr-serf. */

Using apr-serf isn't even up for discussion right now. Yes, that would fix
it one day, but really: the whole API would be different (bucket brigades
rather than files).

For now, the *real* and proper solution is to use Neon's callbacks to read
from the apr_file_t rather than passing in a file descriptor. In fact, that
is probably the long-term solution, too. (as long as we Neon)

Cheers,
-g

p.s. for those who are wondering wtf "apr-serf" is about... the APR project
is starting a new HTTP client library. And I mean *start* -- no design docs
or other info available yet other than what is in people's minds and some
comments on the APR dev list. And certainly no code (altho we have plenty to
steal)

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org