I'm not sure if Jonathan Nieder meant the enum-from-process_acks() change should be in this patch set, but I've included it here anyway. Also I've removed an extraneous debug "cp" from patch 2/3 (previously patch 1/2). Jonathan Tan (3): fetch-pack: return enum from process_acks() fetch-pack: in protocol v2, in_vain only after ACK fetch-pack: in protocol v2, reset in_vain upon ACK fetch-pack.c | 49 +++++++++++++++++++++++++++++++++---------- t/t5500-fetch-pack.sh | 48 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 86 insertions(+), 11 deletions(-) -- 2.26.2.303.gf8c07b1a785-goog