| hg-reviewers |
We should not be accessing private attributes on the manifestlog,
since its behavior is governed by an interface.
| Lint Skipped |
| Unit Tests Skipped |
| reporef = weakref.ref(repo) | reporef = weakref.ref(repo) | ||||
| manifestrevlogref = weakref.ref(self) | manifestrevlogref = weakref.ref(self) | ||||
| def persistmanifestcache(): | def persistmanifestcache(): | ||||
| repo = reporef() | repo = reporef() | ||||
| self = manifestrevlogref() | self = manifestrevlogref() | ||||
| if repo is None or self is None: | if repo is None or self is None: | ||||
| return | return | ||||
| if repo.manifestlog._revlog is not self: | if repo.manifestlog.getstorage(b'') is not self: | ||||
| # there's a different manifest in play now, abort | # there's a different manifest in play now, abort | ||||
| return | return | ||||
| self._fulltextcache.write() | self._fulltextcache.write() | ||||
| if repo._currentlock(repo._lockref) is not None: | if repo._currentlock(repo._lockref) is not None: | ||||
| repo._afterlock(persistmanifestcache) | repo._afterlock(persistmanifestcache) | ||||
| @property | @property | ||||
| @interfaceutil.implementer(repository.imanifestrevisionwritable) | @interfaceutil.implementer(repository.imanifestrevisionwritable) | ||||
| class memmanifestctx(object): | class memmanifestctx(object): | ||||
| def __init__(self, manifestlog): | def __init__(self, manifestlog): | ||||
| self._manifestlog = manifestlog | self._manifestlog = manifestlog | ||||
| self._manifestdict = manifestdict() | self._manifestdict = manifestdict() | ||||
| def _storage(self): | def _storage(self): | ||||
| return self._manifestlog._revlog | return self._manifestlog.getstorage(b'') | ||||
| def new(self): | def new(self): | ||||
| return memmanifestctx(self._manifestlog) | return memmanifestctx(self._manifestlog) | ||||
| def copy(self): | def copy(self): | ||||
| memmf = memmanifestctx(self._manifestlog) | memmf = memmanifestctx(self._manifestlog) | ||||
| memmf._manifestdict = self.read().copy() | memmf._manifestdict = self.read().copy() | ||||
| return memmf | return memmf | ||||
| # TODO: We eventually want p1, p2, and linkrev exposed on this class, | # TODO: We eventually want p1, p2, and linkrev exposed on this class, | ||||
| # but let's add it later when something needs it and we can load it | # but let's add it later when something needs it and we can load it | ||||
| # lazily. | # lazily. | ||||
| #self.p1, self.p2 = store.parents(node) | #self.p1, self.p2 = store.parents(node) | ||||
| #rev = store.rev(node) | #rev = store.rev(node) | ||||
| #self.linkrev = store.linkrev(rev) | #self.linkrev = store.linkrev(rev) | ||||
| def _storage(self): | def _storage(self): | ||||
| return self._manifestlog._revlog | return self._manifestlog.getstorage(b'') | ||||
| def node(self): | def node(self): | ||||
| return self._node | return self._node | ||||
| def new(self): | def new(self): | ||||
| return memmanifestctx(self._manifestlog) | return memmanifestctx(self._manifestlog) | ||||
| def copy(self): | def copy(self): | ||||
| @interfaceutil.implementer(repository.imanifestrevisionwritable) | @interfaceutil.implementer(repository.imanifestrevisionwritable) | ||||
| class memtreemanifestctx(object): | class memtreemanifestctx(object): | ||||
| def __init__(self, manifestlog, dir=''): | def __init__(self, manifestlog, dir=''): | ||||
| self._manifestlog = manifestlog | self._manifestlog = manifestlog | ||||
| self._dir = dir | self._dir = dir | ||||
| self._treemanifest = treemanifest() | self._treemanifest = treemanifest() | ||||
| def _storage(self): | def _storage(self): | ||||
| return self._manifestlog._revlog | return self._manifestlog.getstorage(b'') | ||||
| def new(self, dir=''): | def new(self, dir=''): | ||||
| return memtreemanifestctx(self._manifestlog, dir=dir) | return memtreemanifestctx(self._manifestlog, dir=dir) | ||||
| def copy(self): | def copy(self): | ||||
| memmf = memtreemanifestctx(self._manifestlog, dir=self._dir) | memmf = memtreemanifestctx(self._manifestlog, dir=self._dir) | ||||
| memmf._treemanifest = self._treemanifest.copy() | memmf._treemanifest = self._treemanifest.copy() | ||||
| return memmf | return memmf | ||||