Thread (1 message) 1 message, 1 author, 2018-06-14

Re: [PATCH 2/2] builtin/blame: highlight recently changed lines

From: Junio C Hamano <hidden>
Date: 2018-06-14 17:37:22

René Scharfe [off-list ref] writes:
This adds a minor memory leak; fix below.

-- >8 --
Subject: [PATCH] blame: release string_list after use in parse_color_fields()

Signed-off-by: Rene Scharfe <redacted>
---
Thanks.  Will apply.
quoted hunk
 builtin/blame.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/builtin/blame.c b/builtin/blame.c
index 4202584f97..3295718841 100644
--- a/builtin/blame.c
+++ b/builtin/blame.c
@@ -411,6 +411,7 @@ static void parse_color_fields(const char *s)
 		die (_("must end with a color"));
 
 	colorfield[colorfield_nr].hop = TIME_MAX;
+	string_list_clear(&l, 0);
 }
 
 static void setup_default_color_by_age(void)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help