Junio C Hamano <gitster@xxxxxxxxx> writes:
> Glen Choo <chooglen@xxxxxxxxxx> writes:
>
>> branch_ref = resolve_ref_unsafe("HEAD", 0, NULL, &flags);
>> if (flags & REF_ISSYMREF)
>> - branch_name = strrchr(branch_ref, '/') + 1;
>> + skip_prefix(branch_ref, "refs/heads/", &branch_name);
>
> The branch_name variable is initialized to a constant string "(no branch)",
> so if HEAD is poihnting elsewhere (which you could do manually),
> skip_prefix() would fail and leave branch_name intact, which would
> give us the desirable outcome, too.
>
> Looking good.
Hm, did we ever pick this up? I dug through the old "What's Cooking"
mails and didn't find any mention of this.
Admittedly, this dropped off my radar until performance review season
reminded me of this. Though now that I say this, it sounds like I want
this for the sake of performance review :p
(Which is not the case btw, I just want to scratch my own itch :))
