[TSVN] Another commit dialog perf patch

Hi,

Here's another set of patches which improve efficiency of the commit-log
dialog.

Have saved a bit of PathIsDirectory calling by using the value from the
FindNextFile.

A few other changes mainly to try and avoid too much CString chatter.

I am still concerned about the 'check for files which differ only in case'
stuff in ListCtrl::Stat() - this loop just explodes given half a
chance. One of my test sets has about 32000 files in, and even though
nearly all of these are in unversioned folders, it still ran that loop in
Stat over 3 million times, though admittedly it didn't do much in it.

On a largeish WC I have, that loop runs about 1 million times, and did seem
to do rather more - stat() took about 10% of the run time. (70% was in the
SVN get status call).

Maybe it's now down in the noise and I shouldn't care anymore, but offends
me to see something running N-squared for a large 'N' when I think it could
probably be NLog(N) for a very small N. Something to think about for the
future, anyway.

Cheers,

Will

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org