Re: [PATCH 1/3] checkout: describe_detached_head: remove 3dots after committish

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 7, 2017, 00:52 UTC
Message-ID: <xmqq375qj4fe.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20171105162730.31405-1-bedhanger@gmx.de>

Ann T Ropea <bedhanger@gmx.de> writes:

> Lest we confuse the meticulous observer, we ought to retire the 3dots in
> the circumstances described above.

Yes, as I said in my response to 3/3, I think it is a good goal to avoid n-dots used as a "here we truncated something longer" sign, which was a very old convention that was invented without knowing that we'd later come up with a syntax that would conflict with it.

For this particular output, I wonder if it is even better to follow our own advice, though. Documentation/SubmittingPatches says:

    If you want to reference a previous commit in the history of a stable
    branch, use the format "abbreviated sha1 (subject, date)",
    with the subject enclosed in a pair of double-quotes, like this:

        Commit f86a374 ("pack-bitmap.c: fix a memleak", 2015-03-30)
        noticed that ...

I dunno.

Show 18 quoted lines

> Signed-off-by: Ann T Ropea <bedhanger@gmx.de>
> ---
>  builtin/checkout.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/builtin/checkout.c b/builtin/checkout.c
> index fc4f8fd2ea29..59cc52e55855 100644
> --- a/builtin/checkout.c
> +++ b/builtin/checkout.c
> @@ -404,7 +404,7 @@ static void describe_detached_head(const char *msg, struct commit *commit)
>  	struct strbuf sb = STRBUF_INIT;
>  	if (!parse_commit(commit))
>  		pp_commit_easy(CMIT_FMT_ONELINE, commit, &sb);
> -	fprintf(stderr, "%s %s... %s\n", msg,
> +	fprintf(stderr, "%s %s %s\n", msg,
>  		find_unique_abbrev(commit->object.oid.hash, DEFAULT_ABBREV), sb.buf);
>  	strbuf_release(&sb);
>  }