Re: [PATCH v8 3/6] remote set-head: better output for --auto
Bence Ferdinandy <bence@ferdinandy.com> writes:
[snip]
Show 24 quoted lines
> +static void report_set_head_auto(const char *remote, const char *head_name,
> + struct strbuf *buf_prev) {
> + struct strbuf buf_prefix = STRBUF_INIT;
> + const char *prev_head = NULL;
> +
> + strbuf_addf(&buf_prefix, "refs/remotes/%s/", remote);
> + skip_prefix(buf_prev->buf, buf_prefix.buf, &prev_head);
> +
> + if (prev_head && !strcmp(prev_head, head_name))
> + printf("'%s/HEAD' is unchanged and points to '%s'\n",
> + remote, head_name);
> + else if (prev_head)
> + printf("'%s/HEAD' has changed from '%s' and now points to '%s'\n",
> + remote, prev_head, head_name);
> + else if (!buf_prev->len)
> + printf("'%s/HEAD' is now created and points to '%s'\n",
> + remote, head_name);
> + else
> + printf("'%s/HEAD' used to point to '%s' "
> + "(which is not a remote branch), but now points to '%s'\n",
> + remote, buf_prev->buf, head_name);
> + strbuf_release(&buf_prefix);
> +}
> +Shouldn't the messages above be marked for translation too? using `_()`?
Show 6 quoted lines
> static int set_head(int argc, const char **argv, const char *prefix)
> {
> int i, opt_a = 0, opt_d = 0, result = 0;
> - struct strbuf buf = STRBUF_INIT, buf2 = STRBUF_INIT;
> + struct strbuf buf = STRBUF_INIT, buf2 = STRBUF_INIT,
> + buf_prev = STRBUF_INIT;Nit: these names could definitely be better.
[snip]