branch: migrate `opts` to native kwargs
authorMatt Harbison <matt_harbison@yahoo.com>
Sun, 20 Aug 2023 00:55:52 -0400
changeset 50828 13ad1b2ad3b4
parent 50827 aad6e62d64f8
child 50829 3bd013e8e024
branch: migrate `opts` to native kwargs
mercurial/cmdutil.py
mercurial/commands.py
--- a/mercurial/cmdutil.py	Sun Aug 20 00:44:15 2023 -0400
+++ b/mercurial/cmdutil.py	Sun Aug 20 00:55:52 2023 -0400
@@ -1007,7 +1007,7 @@
     raise error.UnknownCommand(cmd, allcmds)
 
 
-def changebranch(ui, repo, revs, label, opts):
+def changebranch(ui, repo, revs, label, **opts):
     """Change the branch name of given revs to label"""
 
     with repo.wlock(), repo.lock(), repo.transaction(b'branches'):
@@ -1026,7 +1026,7 @@
         root = repo[roots.first()]
         rpb = {parent.branch() for parent in root.parents()}
         if (
-            not opts.get(b'force')
+            not opts.get('force')
             and label not in rpb
             and label in repo.branchmap()
         ):
--- a/mercurial/commands.py	Sun Aug 20 00:44:15 2023 -0400
+++ b/mercurial/commands.py	Sun Aug 20 00:55:52 2023 -0400
@@ -1339,12 +1339,11 @@
 
     Returns 0 on success.
     """
-    opts = pycompat.byteskwargs(opts)
-    revs = opts.get(b'rev')
+    revs = opts.get('rev')
     if label:
         label = label.strip()
 
-    if not opts.get(b'clean') and not label:
+    if not opts.get('clean') and not label:
         if revs:
             raise error.InputError(
                 _(b"no branch name specified for the revisions")
@@ -1353,7 +1352,7 @@
         return
 
     with repo.wlock():
-        if opts.get(b'clean'):
+        if opts.get('clean'):
             label = repo[b'.'].branch()
             repo.dirstate.setbranch(label, repo.currenttransaction())
             ui.status(_(b'reset working directory to branch %s\n') % label)
@@ -1361,9 +1360,9 @@
 
             scmutil.checknewlabel(repo, label, b'branch')
             if revs:
-                return cmdutil.changebranch(ui, repo, revs, label, opts)
-
-            if not opts.get(b'force') and label in repo.branchmap():
+                return cmdutil.changebranch(ui, repo, revs, label, **opts)
+
+            if not opts.get('force') and label in repo.branchmap():
                 if label not in [p.branch() for p in repo[None].parents()]:
                     raise error.InputError(
                         _(b'a branch of the same name already exists'),