changeset 50521:f697af015683

stream-clone: bail-out earlier if pull is partial The `canperformstreamclone` function is bit messy. However it seems clearer to me to process the very generic condition about "can we consider a stream-clone at all", before checking if a stream-clone is requested and if a compatible protocol can be negotiated with the server. So I am doing some gratuitous movement so reorder conditional.
author Pierre-Yves David <pierre-yves.david@octobus.net>
date Sat, 20 May 2023 01:19:26 +0200
parents 0558866957fa
children 58e4842fbfc1
files mercurial/streamclone.py
diffstat 1 files changed, 4 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/streamclone.py	Sat May 20 01:17:27 2023 +0200
+++ b/mercurial/streamclone.py	Sat May 20 01:19:26 2023 +0200
@@ -73,6 +73,10 @@
     if len(repo):
         return False, None
 
+    # Streaming clone only works if all data is being requested.
+    if pullop.heads:
+        return False, None
+
     bundle2supported = False
     if pullop.canusebundle2:
         if b'v2' in pullop.remotebundle2caps.get(b'stream', []):
@@ -88,10 +92,6 @@
     elif bundle2 and not bundle2supported:
         return False, None
 
-    # Streaming clone only works if all data is being requested.
-    if pullop.heads:
-        return False, None
-
     streamrequested = pullop.streamclonerequested
 
     # If we don't have a preference, let the server decide for us. This