Mercurial > hg-stable
changeset 51335:c9ab452e5611
delta-find: check DELTA_BASE_REUSE_FORCE in the _DeltaSearch.__init__
Now that we have an object we can check that DELTA_BASE_REUSE_FORCE cases does not reach this code at in a more suitable location.
author | Pierre-Yves David <pierre-yves.david@octobus.net> |
---|---|
date | Mon, 20 Nov 2023 05:05:29 +0100 |
parents | 047358417d01 |
children | 01db705bbe32 |
files | mercurial/revlogutils/deltas.py |
diffstat | 1 files changed, 8 insertions(+), 9 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/revlogutils/deltas.py Mon Nov 20 05:04:23 2023 +0100 +++ b/mercurial/revlogutils/deltas.py Mon Nov 20 05:05:29 2023 +0100 @@ -693,6 +693,14 @@ target_rev=None, snapshot_cache=None, ): + # the DELTA_BASE_REUSE_FORCE case should have been taken care of sooner + # so we should never end up asking such question. Adding the assert as + # a safe-guard to detect anything that would be fishy in this regard. + assert ( + cachedelta is None + or cachedelta[2] != DELTA_BASE_REUSE_FORCE + or not revlog.delta_config.general_delta + ) self.revlog = revlog self.textlen = textlen self.p1 = p1 @@ -725,15 +733,6 @@ yield None return - # the DELTA_BASE_REUSE_FORCE case should have been taken care of sooner - # so we should never end up asking such question. Adding the assert as - # a safe-guard to detect anything that would be fishy in this regard. - assert ( - self.cachedelta is None - or self.cachedelta[2] != DELTA_BASE_REUSE_FORCE - or not self.revlog.delta_config.general_delta - ) - deltalength = self.revlog.length deltaparent = self.revlog.deltaparent sparse = self.revlog.delta_config.sparse_revlog