Re: [PATCH 02/13] remove_branches(): remove temporary

From: Stefan Beller <sbeller@google.com>
Date: Jun 8, 2015, 16:45 UTC
Message-ID: <CAGZ79kaW-jT5x2SNLjL6xtY=Qe_U86G6+nh8y2sP9Sv=CHc5Kg@mail.gmail.com>
In-Reply-To: <77713838b598b95c5973a1fe73569635f5f31ee9.1433763494.git.mhagger@alum.mit.edu>

On Mon, Jun 8, 2015 at 4:45 AM, Michael Haggerty <mhagger@alum.mit.edu> wrote:

> Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>

Found-to-be-Obviously-Correct-by: Stefan Beller <sbeller@google.com> ;)

Show 21 quoted lines

> ---
>  builtin/remote.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/builtin/remote.c b/builtin/remote.c
> index f4a6ec9..53b8e13 100644
> --- a/builtin/remote.c
> +++ b/builtin/remote.c
> @@ -756,8 +756,7 @@ static int remove_branches(struct string_list *branches)
>         strbuf_release(&err);
>
>         for (i = 0; i < branches->nr; i++) {
> -               struct string_list_item *item = branches->items + i;
> -               const char *refname = item->string;
> +               const char *refname = branches->items[i].string;
>
>                 if (delete_ref(refname, NULL, 0))
>                         result |= error(_("Could not remove branch %s"), refname);
> --
> 2.1.4
>