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.