Code

Fix branch -m @{-1} newname
authorJunio C Hamano <gitster@pobox.com>
Sat, 21 Mar 2009 20:23:27 +0000 (13:23 -0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 23 Mar 2009 06:52:11 +0000 (23:52 -0700)
The command is supposed to rename the branch we were on before switched
from to a new name, but was not aware of the short-hand notation we added
recently.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-branch.c

index 7452db13c89f0d025700949605e983087df85961..0df82bf96d7f06426ac0ff89f3410714257c20fd 100644 (file)
@@ -468,18 +468,18 @@ static void rename_branch(const char *oldname, const char *newname, int force)
        if (!oldname)
                die("cannot rename the current branch while not on any.");
 
-       strbuf_addf(&oldref, "refs/heads/%s", oldname);
-
+       strbuf_branchname(&oldref, oldname);
+       strbuf_splice(&oldref, 0, 0, "refs/heads/", 11);
        if (check_ref_format(oldref.buf))
-               die("Invalid branch name: %s", oldref.buf);
-
-       strbuf_addf(&newref, "refs/heads/%s", newname);
+               die("Invalid branch name: '%s'", oldname);
 
+       strbuf_branchname(&newref, newname);
+       strbuf_splice(&newref, 0, 0, "refs/heads/", 11);
        if (check_ref_format(newref.buf))
-               die("Invalid branch name: %s", newref.buf);
+               die("Invalid branch name: '%s'", newname);
 
        if (resolve_ref(newref.buf, sha1, 1, NULL) && !force)
-               die("A branch named '%s' already exists.", newname);
+               die("A branch named '%s' already exists.", newref.buf + 11);
 
        strbuf_addf(&logmsg, "Branch: renamed %s to %s",
                 oldref.buf, newref.buf);