Stefan Beller <sbeller@xxxxxxxxxx> writes: > Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> > --- The change in 2/16 to extract something into a helper was sort-of guessable without any comment (i.e. perhaps it will gain new callers in later patches), but a change like this needs to hint why this is a good thing to do, I would think. I think you'll be rerolling this anyway due to droppage of 1/16, so hopefully we'll see the next version a bit better explained than this round ;-) Thanks. > builtin/receive-pack.c | 13 +++++---- > connect.c | 79 ++++++++++++++++++++++---------------------------- > connect.h | 2 +- > upload-pack.c | 11 +++++-- > 4 files changed, 51 insertions(+), 54 deletions(-) -- 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