On Fri, 2010-09-24 at 13:45 +0100, Philip Martin wrote: > julianf...@apache.org writes: > > > Author: julianfoad > > Date: Fri Sep 24 11:27:36 2010 > > New Revision: 1000816 > > > + /* ### Huh? This looks like it's expected to overwrite the temp > > file > > + *DST_ABSPATH, but it would return an error if the > > destination > > + exists. And same for svn_io_dir_make() below. What gives? */ > > The file has already been closed and deleted because NULL is passed to > svn_io_open_unique_file3.
Philip, thanks! Even after you said this it took me a while to see how the file got deleted, so I've updated the doc string of svn_io_open_unique_file3() to mention that behaviour. r1000874. - Julian
