Re: [PATCH 2/2] use strbuf_add_unique_abbrev() for adding short hashes, part 2

[Junio C Hamano]

René Scharfe <l....@web.de> writes:

> diff --git a/diff.c b/diff.c
> index a178ed3..be11e4e 100644
> --- a/diff.c
> +++ b/diff.c
> @@ -3109,7 +3109,7 @@ static void fill_metainfo(struct strbuf *msg,
>               }
>               strbuf_addf(msg, "%s%sindex %s..", line_prefix, set,
>                           find_unique_abbrev(one->oid.hash, abbrev));
> -             strbuf_addstr(msg, find_unique_abbrev(two->oid.hash, abbrev));
> +             strbuf_add_unique_abbrev(msg, two->oid.hash, abbrev);

OK.

> diff --git a/submodule.c b/submodule.c
> index dcc5ce3..8cf40ea 100644
> --- a/submodule.c
> +++ b/submodule.c
> @@ -396,7 +396,7 @@ static void show_submodule_header(FILE *f, const char 
> *path,
>                       find_unique_abbrev(one->hash, DEFAULT_ABBREV));
>       if (!fast_backward && !fast_forward)
>               strbuf_addch(&sb, '.');
> -     strbuf_addstr(&sb, find_unique_abbrev(two->hash, DEFAULT_ABBREV));
> +     strbuf_add_unique_abbrev(&sb->hash, two, DEFAULT_ABBREV);

I wonder how could this change come out of this definition:

    @@
    expression E1, E2, E3;
    @@
    - strbuf_addstr(E1, find_unique_abbrev(E2, E3));
    + strbuf_add_unique_abbrev(E1, E2, E3);