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

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

From: Greg Stein <gstein_at_lyra.org>
Date: 2001-10-01 11:36:14 CEST

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
Received on Sat Oct 21 14:36:43 2006

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