comparison mercurial/branchmap.py @ 51489:659f766629c8

branchcache: stop using `copy(…)` in `replace(…)`

The `copy` method is mostly used for a filter level to inherit the branchmap
from a subset. So we stop using (abusing) it in "replace" to ensure `copy` is
used only for inheritance purposes.

Since `replace` is a method of the BranchMapCache, it seems fine to do lower
level operation there.

51488:94f821490645

            # Try to stick it as low as possible
            # filter above served are unlikely to be fetch from a clone
            for candidate in (b'base', b'immutable', b'served'):
                rview = repo.filtered(candidate)
                if cache.validfor(rview):
                    cache = self._per_filter[candidate] = cache.copy(rview)
                    cache.write(rview)
                    return

    def clear(self):
        self._per_filter.clear()

51489:659f766629c8

            # Try to stick it as low as possible
            # filter above served are unlikely to be fetch from a clone
            for candidate in (b'base', b'immutable', b'served'):
                rview = repo.filtered(candidate)
                if cache.validfor(rview):
                    cache._filtername = candidate
                    self._per_filter[candidate] = cache
                    cache._dirty = True
                    cache.write(rview)
                    return

    def clear(self):
        self._per_filter.clear()