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

Greg Stein wrote:

>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).
>
Sure. I wrote that comment mostly to clear my conscience after the
horrible hacks in that code.

>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)
>
Can that be done now? Or should I wait for neon-0.17.0?

-- 
Brane �ibej   <brane_at_xbc.nu>            http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org