Junio C Hamano <[email protected]> writes: > Stefan Beller <[email protected]> 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/[email protected]/msg60604.html >> >> Idea-by: Ronnie Sahlberg <[email protected]> >> Signed-off-by: Stefan Beller <[email protected]> >> --- >> 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
