Subject: Re: [PATCH v6 3/5] builtin/stash: factor out revision parsing into a function

From: Junio C Hamano <[email-masked]>
Date: Thu, 22 May 2025 13:34:16 -0700

"brian m. carlson" <[email-masked]> writes:

> +	strbuf_init(&info->revision, 0);
> +	if (parse_stash_revision(&info->revision, commit, 0)) {
> +		return -1;
>  	}

It does not look like this series add more code inside this block in
a later step, so let's lose the unnecessary {braces} around a single
statement "return -1" here.

By the way, what is "pwodd" I saw in the e-mail header?