Re: [PATCH] Add option -b/--branch to clone for select a new HEAD

From: Björn Steinbrink <b.steinbrink@gmx.de>
Date: Aug 26, 2009, 16:56 UTC
Message-ID: <20090826165618.GA7477@atjola.homenet>
In-Reply-To: <20090826161059.GC32741@coredump.intra.peff.net>

On 2009.08.26 12:10:59 -0400, Jeff King wrote:

Show 11 quoted lines

> So I think the code should probably look like this (totally untested):
> 
>   remote_head = find_ref_by_name(refs, "HEAD");
>   if (option_branch) {
>           strbuf_addf(&branch_head, "%s%s", src_ref_prefix, option_branch);
>           head_points_at = find_ref_by_name(refs, branch_head.buf);
>           if (!head_points_at)
>                   warning("remote branch not found, etc");
>   }
>   if (!head_points_at)
>     head_points_at = guess_remote_head(remote_head, mapped_refs, 0);

Hm, why "refs" for find_ref_by_name(), but "mapped_ref" for guess_remote_head()?

Björn