[APR] apr_file_copy() improvements

Since several of you are APR developers, I wonder if one of you would be keen
to take up the following issues with apr_file_copy(). We've now got several
reasons to re-work it:

Correctness:

We (Subversion) expect it to do a byte-for-byte copy. To achieve this on some
less-common platforms, we have:

* Paul Burba's work-around for OS400 going into Subversion.

* Max Bowsher's patch for CygWin that he makes to APR when building it.

If the APR developers can agree that that is the intended behaviour, both of
those patches would be eliminated by documenting that apr_file_copy performs a
byte-for-byte copy, and making it open the source and destination in "binary"
mode to achieve this. The behaviour would be unchanged on Unix-like platforms.

Speed:

* Implementation for MS Windows should use the OS's native "file copy" API for
much faster performance (and probably better performance in terms of preserving
permissions, etc.).

* Ditto on any other platforms that provide such an API.

* Other implementations should use a buffer bigger than BUFSIZ.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 10 14:03:46 2006