# HG changeset patch # User Adrian Buehlmann # Date 1304506739 -7200 # Node ID 3a90fb3addc139a071e312546498b4e7f0c28dff # Parent c4de166428610bd927bd5958c62b8e73187a709b store: break up reference cycle introduced in 9cbff8a39a2a see also 996c1cd8f530 diff -r c4de16642861 -r 3a90fb3addc1 mercurial/store.py --- a/mercurial/store.py Wed May 04 10:32:08 2011 -0500 +++ b/mercurial/store.py Wed May 04 12:58:59 2011 +0200 @@ -365,26 +365,27 @@ self._load() return iter(self.entries) +class _fncacheopener(scmutil.abstractopener): + def __init__(self, op, fnc, encode): + self.opener = op + self.fncache = fnc + self.encode = encode + + def __call__(self, path, mode='r', *args, **kw): + if mode not in ('r', 'rb') and path.startswith('data/'): + self.fncache.add(path) + return self.opener(self.encode(path), mode, *args, **kw) + class fncachestore(basicstore): def __init__(self, path, openertype, encode): self.encode = encode self.path = path + '/store' self.createmode = _calcmode(self.path) - - storeself = self - - class fncacheopener(openertype): - def __call__(self, path, mode='r', *args, **kw): - if mode not in ('r', 'rb') and path.startswith('data/'): - fnc.add(path) - return openertype.__call__(self, storeself.encode(path), mode, - *args, **kw) - - op = fncacheopener(self.path) + op = openertype(self.path) op.createmode = self.createmode fnc = fncache(op) self.fncache = fnc - self.opener = op + self.opener = _fncacheopener(op, fnc, encode) def join(self, f): return self.path + '/' + self.encode(f)