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

Re: [PATCH] #7 OS400/EBCDIC Port: Make svn_io_copy_file() CCSID insensitive.

From: Mark Phippard <markp_at_softlanding.com>
Date: 2006-02-25 01:54:27 CET

> APR_BINARY has no effect on Windows or Unix, as far as I know. APR does
> _not_ do end-of-line conversion on file contents, so APR_BINARY and its
> documentation is a bit misleading.

We are looking at the Unix implementation, the one Max referred to. It
just opens both files (without a binary flag) and then just reads and
writes. Does opening a file on Unix without the binary flag have any
meaning?

> The question is, then, whether the OS/400 port of APR is correct in not
> using APR_BINARY in apr_file_copy or not.

I suspect this is another case of a function that they (IBM) do not really
need for httpd and the Unix code compiles and nominally works. Also, the
APR documentation does not really specify what the function ought to do.
It seems to me that APR ought to be specifying the APR_BINARY flag the way
that Paul's proposed patch does. The only reply I got from IBM was that
they are using the APR code for this function without modification.

Mark

_____________________________________________________________________________
Scanned for SoftLanding Systems, Inc. and SoftLanding Europe Plc by IBM Email Security Management Services powered by MessageLabs.
_____________________________________________________________________________

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Feb 25 01:55:10 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.