# HG changeset patch
# User Kyle Lippincott <spectral@google.com>
# Date 1650395843 25200
# Node ID f45e1618cbf68aca4e72b6fed8dd2acef6f39a10
# Parent  e55ce61891e3a2c65844cebb645973a48bf6933a
amend: move "return None for removed files" into block handling filestoamend

This is just a small logic cleanup from D12573, no change in behavior.

Differential Revision: https://phab.mercurial-scm.org/D12576

diff -r e55ce61891e3 -r f45e1618cbf6 mercurial/cmdutil.py
--- a/mercurial/cmdutil.py	Tue Apr 12 17:35:25 2022 +0200
+++ b/mercurial/cmdutil.py	Tue Apr 19 12:17:23 2022 -0700
@@ -2933,16 +2933,15 @@
 
             def filectxfn(repo, ctx_, path):
                 try:
-                    # Return None for removed files.
-                    if path in wctx.removed() and path in filestoamend:
-                        return None
-
                     # If the file being considered is not amongst the files
                     # to be amended, we should use the file context from the
                     # old changeset. This avoids issues when only some files in
                     # the working copy are being amended but there are also
                     # changes to other files from the old changeset.
                     if path in filestoamend:
+                        # Return None for removed files.
+                        if path in wctx.removed():
+                            return None
                         fctx = wctx[path]
                     else:
                         fctx = old.filectx(path)