[PATCH 01 of 10 V2] discovery: minor fix to a conditional

Boris Feld boris.feld at octobus.net
Fri Jan 4 17:29:22 UTC 2019


# HG changeset patch
# User Boris Feld <boris.feld at octobus.net>
# Date 1546620599 -3600
#      Fri Jan 04 17:49:59 2019 +0100
# Node ID 01a79b3269d4bfe49306b021de35637f7016fa45
# Parent  5c68b617ba2463eb6f1372a24b139a376c6bf6bd
# EXP-Topic discovery-refactor
# Available At https://bitbucket.org/octobus/mercurial-devel/
#              hg pull https://bitbucket.org/octobus/mercurial-devel/ -r 01a79b3269d4
discovery: minor fix to a conditional

Since `size` is the upper limit of the sample, we should include it in the
check. Otherwize the `more` variable will be zero and the sampling will be
useless

diff --git a/mercurial/setdiscovery.py b/mercurial/setdiscovery.py
--- a/mercurial/setdiscovery.py
+++ b/mercurial/setdiscovery.py
@@ -146,7 +146,7 @@ def _takefullsample(repo, headrevs, revs
     _updatesample(revs, revsroots, sample, children.__getitem__)
     assert sample
     sample = _limitsample(sample, size)
-    if len(sample) < size:
+    if len(sample) <= size:
         more = size - len(sample)
         sample.update(random.sample(list(revs - sample), more))
     return sample


More information about the Mercurial-devel mailing list