comparison mercurial/manifest.py @ 31097:4a1486c73fdf

manifest: check 'if x is None' instead of 'if not x'

The old code here would end up executing __len__ on a tree manifest to determine
if 'not _data' was true or not. This was very expensive on large repos. Since
this function just cares about memoization, we can just check 'if _data is None'
instead and save a bunch of time.

31096:356937ea7a02

    @propertycache
    def parents(self):
        return self._revlog().parents(self._node)

    def read(self):
        if not self._data:
            if self._node == revlog.nullid:
                self._data = manifestdict()
            else:
                rl = self._revlog()
                text = rl.revision(self._node)

    def _revlog(self):
        return self._repo.manifestlog._revlog.dirlog(self._dir)

    def read(self):
        if not self._data:
            rl = self._revlog()
            if self._node == revlog.nullid:
                self._data = treemanifest()
            elif rl._treeondisk:
                m = treemanifest(dir=self._dir)

31097:4a1486c73fdf

    @propertycache
    def parents(self):
        return self._revlog().parents(self._node)

    def read(self):
        if self._data is None:
            if self._node == revlog.nullid:
                self._data = manifestdict()
            else:
                rl = self._revlog()
                text = rl.revision(self._node)

    def _revlog(self):
        return self._repo.manifestlog._revlog.dirlog(self._dir)

    def read(self):
        if self._data is None:
            rl = self._revlog()
            if self._node == revlog.nullid:
                self._data = treemanifest()
            elif rl._treeondisk:
                m = treemanifest(dir=self._dir)