Merge branch 'sb/branch-avoid-repeated-strbuf-release'

gitster · gitster · commit cfa0fd0ffc7a · 2017-10-07T16:27:54.000+09:00
* sb/branch-avoid-repeated-strbuf-release:
  branch: reset instead of release a strbuf
diff --git a/builtin/branch.c b/builtin/branch.c
@@ -217,7 +217,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
 		if (!head_rev)
 			die(_("Couldn't look up commit object for HEAD"));
 	}
-	for (i = 0; i < argc; i++, strbuf_release(&bname)) {
+	for (i = 0; i < argc; i++, strbuf_reset(&bname)) {
 		char *target = NULL;
 		int flags = 0;
 
@@ -282,8 +282,9 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
 	}
 
 	free(name);
+	strbuf_release(&bname);
 
-	return(ret);
+	return ret;
 }
 
 static int calc_maxwidth(struct ref_array *refs, int remote_bonus)

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,`
`217`	`217`	`if (!head_rev)`
`218`	`218`	`die(_("Couldn't look up commit object for HEAD"));`
`219`	`219`	`}`
`220`		`- for (i = 0; i < argc; i++, strbuf_release(&bname)) {`
	`220`	`+ for (i = 0; i < argc; i++, strbuf_reset(&bname)) {`
`221`	`221`	`char *target = NULL;`
`222`	`222`	`int flags = 0;`
`223`	`223`
`@@ -282,8 +282,9 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,`
`282`	`282`	`}`
`283`	`283`
`284`	`284`	`free(name);`
	`285`	`+ strbuf_release(&bname);`
`285`	`286`
`286`		`- return(ret);`
	`287`	`+ return ret;`
`287`	`288`	`}`
`288`	`289`
`289`	`290`	`static int calc_maxwidth(struct ref_array *refs, int remote_bonus)`