On Monday 01 August 2005 05:42, Branko Čibej wrote:
> Peter N. Lundblad wrote:
> >On Sun, 31 Jul 2005, [UTF-8] Branko �^Libej wrote:
> >>Peter N. Lundblad wrote:
> >>
> >>One of the things we could do, that I don't see mentioned in the wc
> >>enhancements doc, is to get rid of empty-file. Instead of one empty file
> >>in every .svn directory, I'd put one somewhere in ~/.subversion and use
> >>that. If it's not there, and can't be created. we can still create a
> >>temporary one in .svn/tmp or /tmp. This would save yet another inode per
> >>directory (on those FSes that care about inodes, that is).
> >
> >I haven't checked, but I'd guess empty-file is used to generate diffs,
> >true? In that creating an empty file on demand couldn't be that of a
> >performance bottle-neck... Shy bother with an empty-file in .subversion in
> >that case? We just stop using empty-file and stop creating it in the next
> >WC format.
>
> We put emtpy-file in there because IIRC it was shown that creating it
> for every diff did have a significant impact on speed. It might be that
> "it was shown" incorrectly; but if it wasn't, it certainly makes no
> sense to have one such file per .svn directory.
Why not use an empty or null stream instead? It looks like the file handles
are being converted to stream object anyways.
-John
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Aug 1 12:30:28 2005