# HG changeset patch
# User Matt Harbison <matt_harbison@yahoo.com>
# Date 1676407526 18000
# Node ID a8d71a6ba2050c1689c6c4227b9ae8f55fa1f409
# Parent  a3c856e2ea2f410c2bec2a58f62041e7f5d77378
tag: move the prohibition of tagging the `null` rev up to the `wdir()` check

It makes sense to do these together, and avoid another revision lookup.

diff -r a3c856e2ea2f -r a8d71a6ba205 mercurial/commands.py
--- a/mercurial/commands.py	Fri Feb 17 17:04:41 2023 +0100
+++ b/mercurial/commands.py	Tue Feb 14 15:45:26 2023 -0500
@@ -13,6 +13,7 @@
 from .i18n import _
 from .node import (
     hex,
+    nullid,
     nullrev,
     short,
     wdirrev,
@@ -7500,8 +7501,11 @@
                 )
         node = logcmdutil.revsingle(repo, rev_).node()
 
+        # don't allow tagging the null rev or the working directory
         if node is None:
             raise error.InputError(_(b"cannot tag working directory"))
+        elif not opts.get(b'remove') and node == nullid:
+            raise error.InputError(_(b"cannot tag null revision"))
 
         if not message:
             # we don't translate commit messages
@@ -7522,13 +7526,6 @@
             editform=editform, **pycompat.strkwargs(opts)
         )
 
-        # don't allow tagging the null rev
-        if (
-            not opts.get(b'remove')
-            and logcmdutil.revsingle(repo, rev_).rev() == nullrev
-        ):
-            raise error.InputError(_(b"cannot tag null revision"))
-
         tagsmod.tag(
             repo,
             names,