On Sun, Aug 11, 2013 at 11:17 PM, Philip Martin <philip.mar...@wandisco.com>wrote:
> To see the difference consider an authz file with just one block for /. > Given two paths, /foo/bar and /zig/zag, your patch produces two cache > entries. Repeat requests for one of the paths pull the result directly > from the cache. The cache grows without limit as further distinct paths > are requested. > That's not how my patch works (the description could probably be more explicit about this aspect): we only cache the "result" of svn_config_enumerate2 if the given path corresponds to a block (section) in the authz file. (Otherwise we don't call svn_config_enumerate2 at all and cache nothing.) Hence the cache is limited by the size of the authz file (i.e. keys of apt_hash_t svn_config_t->sections. Cheers, Roderich
