# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 1684538247 -7200
# Node ID 0558866957fa44bc614c2a28848fe099b4015b48
# Parent  3c0da0fa0eeddf0908795f0f40268e578880e042
stream-clone: bail-out earlier if destination repo is not empty

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.

diff -r 3c0da0fa0eed -r 0558866957fa mercurial/streamclone.py
--- a/mercurial/streamclone.py	Sun May 21 00:00:57 2023 +0200
+++ b/mercurial/streamclone.py	Sat May 20 01:17:27 2023 +0200
@@ -69,6 +69,10 @@
     repo = pullop.repo
     remote = pullop.remote
 
+    # Streaming clone only works on an empty destination repository
+    if len(repo):
+        return False, None
+
     bundle2supported = False
     if pullop.canusebundle2:
         if b'v2' in pullop.remotebundle2caps.get(b'stream', []):
@@ -84,10 +88,6 @@
     elif bundle2 and not bundle2supported:
         return False, None
 
-    # Streaming clone only works on empty repositories.
-    if len(repo):
-        return False, None
-
     # Streaming clone only works if all data is being requested.
     if pullop.heads:
         return False, None