Re: [PATCH v2 4/8] fetch-pack: use ref adv. to prune "have" sent

2018-06-14 Thread Junio C Hamano
Jonathan Tan writes: > +test_expect_success 'use ref advertisement to prune "have" lines sent' ' > + rm -rf server client && > + git init server && > + test_commit -C server both_have_1 && > + git -C server tag -d both_have_1 && > + test_commit -C server both_have_2 && > + >

Re: [PATCH v2 4/8] fetch-pack: use ref adv. to prune "have" sent

2018-06-14 Thread Brandon Williams
On 06/06, Jonathan Tan wrote: > In negotiation using protocol v2, fetch-pack sometimes does not make > full use of the information obtained in the ref advertisement: > specifically, that if the server advertises a commit that the client > also has, the client never needs to inform the server that

[PATCH v2 4/8] fetch-pack: use ref adv. to prune "have" sent

2018-06-06 Thread Jonathan Tan
In negotiation using protocol v2, fetch-pack sometimes does not make full use of the information obtained in the ref advertisement: specifically, that if the server advertises a commit that the client also has, the client never needs to inform the server that it has the commit's parents, since it