Re: Caching the existance of a (base) property file?

On Fri, 4 Nov 2005, Erik Huelsmann wrote:

> Dan,

Uh, I'm not Dan, but anyway... BTW, how did this discussion start? On IRC?

> I was reading through our code in preparation of r17186 and I found
> out there's no reason to add an extra field to indicate the presence
> of a base-props file:

> if (entry->prop_time)
> /* there is a base-pops-file */;
> else
> /* there's not... */

> Or am I wrong and is prop_time for wc-prop-files?

Yes it is. It is used so you can know if the working props have changed
since they were installed. So you can't use it for this purpose.

And bTW, it is going away in wc-propcaching, since it is not necessary
anymore. Then, we can check if stating baseprops will be a bottleneck and
in that case add an extra field for this.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Nov 5 13:41:47 2005