Code

Reword "your branch has diverged..." lines to reduce line length
authorAvery Pennarun <apenwarr@gmail.com>
Wed, 16 Jul 2008 19:19:27 +0000 (15:19 -0400)
committerJunio C Hamano <gitster@pobox.com>
Thu, 17 Jul 2008 00:56:59 +0000 (17:56 -0700)
The message length depends on the length of the branch name.  In my case,
the branch name "origin/add-chickens2" put the first line of the "your
branch has diverged" message over 80 characters, which triggered "less -FS"
to not exit automatically as expected.

This patch rewords the messages to make the lines generally shorter, so
that you'd need a significantly longer branch name to trigger the problem.

Signed-off-by: Avery Pennarun <apenwarr@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
remote.c

index df8bd72ba9b9728aeff0c34208a6492b7d0b692c..0d6020beb8ca9159df09cc947d920e4af8698eca 100644 (file)
--- a/remote.c
+++ b/remote.c
@@ -1308,34 +1308,28 @@ int stat_tracking_info(struct branch *branch, int *num_ours, int *num_theirs)
 int format_tracking_info(struct branch *branch, struct strbuf *sb)
 {
        int num_ours, num_theirs;
-       const char *base, *remote_msg;
+       const char *base;
 
        if (!stat_tracking_info(branch, &num_ours, &num_theirs))
                return 0;
 
        base = branch->merge[0]->dst;
        if (!prefixcmp(base, "refs/remotes/")) {
-               remote_msg = " remote";
                base += strlen("refs/remotes/");
-       } else {
-               remote_msg = "";
        }
        if (!num_theirs)
-               strbuf_addf(sb, "Your branch is ahead of the tracked%s branch '%s' "
+               strbuf_addf(sb, "Your branch is ahead of '%s' "
                            "by %d commit%s.\n",
-                           remote_msg, base,
-                           num_ours, (num_ours == 1) ? "" : "s");
+                           base, num_ours, (num_ours == 1) ? "" : "s");
        else if (!num_ours)
-               strbuf_addf(sb, "Your branch is behind the tracked%s branch '%s' "
-                           "by %d commit%s,\n"
+               strbuf_addf(sb, "Your branch is behind '%s' "
+                           "by %d commit%s, "
                            "and can be fast-forwarded.\n",
-                           remote_msg, base,
-                           num_theirs, (num_theirs == 1) ? "" : "s");
+                           base, num_theirs, (num_theirs == 1) ? "" : "s");
        else
-               strbuf_addf(sb, "Your branch and the tracked%s branch '%s' "
-                           "have diverged,\nand respectively "
-                           "have %d and %d different commit(s) each.\n",
-                           remote_msg, base,
-                           num_ours, num_theirs);
+               strbuf_addf(sb, "Your branch and '%s' have diverged,\n"
+                           "and have %d and %d different commit(s) each, "
+                           "respectively.\n",
+                           base, num_ours, num_theirs);
        return 1;
 }