diff options
| author |  Jonathan Tan <jonathantanmy@google.com> | 2018-06-14 15:54:24 -0700 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2018-06-15 08:44:23 -0700 |
| commit | 21bcf6e429e01961d38bc639b2dc4a6f64b8fdcf (patch) | |
| tree | fa356779736089a0cb8b09a508f76702168e7102 /t/t5516-fetch-push.sh | |
| parent | fetch-pack: clear marks before re-marking (diff) | |
| download | tgif-21bcf6e429e01961d38bc639b2dc4a6f64b8fdcf.tar.xz | |
fetch-pack: directly end negotiation if ACK ready
When "ACK %s ready" is received, find_common() clears rev_list in an
attempt to stop further "have" lines from being sent [1]. It is much
more readable to explicitly break from the loop instead.
So explicitly break from the loop, and make the clearing of the rev_list
happen unconditionally.
[1] The rationale is further described in the originating commit
f2cba9299b ("fetch-pack: Finish negotation if remote replies "ACK %s
ready"", 2011-03-14).
Signed-off-by: Jonathan Tan <jonathantanmy@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5516-fetch-push.sh')
0 files changed, 0 insertions, 0 deletions