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

Re: Early warning: Major log cache code update

From: Alexander Klenin <klenin_at_gmail.com>
Date: 2007-10-05 11:30:01 CEST

On 10/5/07, Stefan Küng <tortoisesvn@gmail.com> wrote:
> > * Refresh in log dialog shall update the head revision
> > (e.g. "any new revisions?") as well as update authors and
> > comments in the revisions shown.
> I think to update authors/comments, the users can use the 'clear log
> cache' in the settings dialog. A refresh in the log dialog should not
> invalidate the cache IMHO.
 I disagree due to several reasons:
1) 'Clear log cache' is not good interface for this, since it is
inconveniently located and,
AFAIK, clears all cache for all working copies instead of just current one.
2) I think that correctness should come before speed here, if user
requested a refresh,
he expects to see current information, not stale copy of it.
3) Finally, what would be the purpose of refreshing if it will just
re-read the cache?
If you think cache invalidation aspect should be emphasized in the name,
how about calling it 'Refetch'?

Alexander S. Klenin
Insight Experts Ltd.
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Fri Oct 5 11:30:09 2007

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

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