From: René Scharfe Date: Sat, 31 Oct 2015 17:37:12 GMT Subject: [PATCH 4/5] wt-status: don't skip a magical number of characters blindly Message-ID: <5634FC48.7070609@web.de> In-Reply-To: <5634FB59.1000506@web.de> Use the variable branch_name, which already has "refs/heads/" removed, instead of blindly advancing in the ->branch string by 11 bytes. This is safer and less magical. Signed-off-by: Rene Scharfe --- wt-status.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/wt-status.c b/wt-status.c index e206cc9..42ea15e 100644 --- a/wt-status.c +++ b/wt-status.c @@ -1656,7 +1656,7 @@ static void wt_shortstatus_print_tracking(struct wt_status *s) if (starts_with(branch_name, "refs/heads/")) branch_name += 11; - branch = branch_get(s->branch + 11); + branch = branch_get(branch_name); color_fprintf(s->fp, branch_color_local, "%s", branch_name); -- 2.6.2