Re: [PATCH v4] refs.c: use a stringlist for repack_without_refs

2014-11-20 Thread Junio C Hamano
Jonathan Nieder writes: > [...] >> +++ b/builtin/remote.c > [...] >> @@ -1361,8 +1352,9 @@ static int prune_remote(const char *remote, int >> dry_run) >> abbrev_ref(refname, "refs/remotes/")); >> } >> >> -warn_dangling_symrefs(stdout, dangling_msg, &delete_

Re: [PATCH v4] refs.c: use a stringlist for repack_without_refs

2014-11-19 Thread Jonathan Nieder
Stefan Beller wrote: > From: Ronnie Sahlberg > > This patch doesn't intend any functional changes. It is just > a refactoring, which replaces a char** array by a stringlist > in the function repack_without_refs. > This is easier to read and maintain as it delivers the same > functionality with le

[PATCH v4] refs.c: use a stringlist for repack_without_refs

2014-11-19 Thread Stefan Beller
From: Ronnie Sahlberg This patch doesn't intend any functional changes. It is just a refactoring, which replaces a char** array by a stringlist in the function repack_without_refs. This is easier to read and maintain as it delivers the same functionality with less lines of code and less pointers.