fetch: use free_refs()

peff · gitster · commit 259eddde6a86 · 2019-04-15T14:00:51.000+09:00
There's no need for us to write this loop manually when a helper
function can already do it.

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/fetch-pack.c b/fetch-pack.c
@@ -630,10 +630,7 @@ static void filter_refs(struct fetch_pack_args *args,
 	}
 
 	oidset_clear(&tip_oids);
-	for (ref = unmatched; ref; ref = next) {
-		next = ref->next;
-		free(ref);
-	}
+	free_refs(unmatched);
 
 	*refs = newlist;
 }

Original file line number	Diff line number	Diff line change
`@@ -630,10 +630,7 @@ static void filter_refs(struct fetch_pack_args *args,`
`630`	`630`	`}`
`631`	`631`
`632`	`632`	`oidset_clear(&tip_oids);`
`633`		`- for (ref = unmatched; ref; ref = next) {`
`634`		`- next = ref->next;`
`635`		`- free(ref);`
`636`		`- }`
	`633`	`+ free_refs(unmatched);`
`637`	`634`
`638`	`635`	`*refs = newlist;`
`639`	`636`	`}`