Junio C Hamano <gitster@xxxxxxxxx> writes: > Stefan Beller <sbeller@xxxxxxxxxx> writes: > >> This patch was heavily inspired by a part of the ref-transactions-rename >> series[1], but people tend to dislike large series and this part is >> relatively easy to take out and unrelated, so I'll send it as a single >> patch. >> >> 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. >> >> [1] https://www.mail-archive.com/git@xxxxxxxxxxxxxxx/msg60604.html >> >> Idea-by: Ronnie Sahlberg <sahlberg@xxxxxxxxxx> >> Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> >> --- >> builtin/remote.c | 22 +++++++--------------- >> refs.c | 41 ++++++++++++++++++++--------------------- >> refs.h | 3 +-- >> 3 files changed, 28 insertions(+), 38 deletions(-) > > In one codepath we were already using a string_list delete_refs_list > anyway, so it makes sense to reuse that by movingan existing call to > string_list_insert() a bit higher, instead of maintaining another > array of pointers delete_refs[] to strings. > > OK, it simplifies the code by reducing the line count, which is a > plus ;-) > > Sounds good. I queued this but as I suspected yesterday had to drop all the other rs/ref-transaction-* topics that are not in 'next' yet. I am guessing that your plan is to make them come back one piece at a time in many easier-to-digest bite sized series. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html