Re: [PATCH] Reduce disk access when browsing non-versioned directories

On Tuesday 27 February 2007 19:59, Stefan Küng wrote:

> We really need to check both, but of course if entriesFilePath does not
> exist, *then* we can avoid checking the filetime of propsDirPath.

ok, I now understand that both are required.

> So, I suggest this:
>
> if (m_entriesFileTime == entriesFilePath.GetLastWriteTime() &&
> entriesFilePath.Exists() && ...)

Do you suggest the following?

if (m_entriesFileTime == entriesFilePath.GetLastWriteTime() &&
entriesFilePath.Exists() &&
m_propsFileTime == propsDirPath.GetLastWriteTime())

This changes the behaviour because if .svn\entries does not exist the whole
expression will evaluate to "false".

What do you think about this patch?

Tobias

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net

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