Re: Early warning: Major log cache code update

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