Re: [PATCH] fetch: do not redundantly calculate tag refmap

2016-11-10 Thread Jeff King
On Thu, Nov 10, 2016 at 04:13:28PM -0800, Jonathan Tan wrote: > builtin/fetch.c redundantly calculates refmaps for tags twice. Remove > the first calculation. > > This is only a code simplification and slight performance improvement - > the result is unchanged, as the redundant refmaps are subseq

[PATCH] fetch: do not redundantly calculate tag refmap

2016-11-10 Thread Jonathan Tan
builtin/fetch.c redundantly calculates refmaps for tags twice. Remove the first calculation. This is only a code simplification and slight performance improvement - the result is unchanged, as the redundant refmaps are subsequently removed by the invocation to "ref_remove_duplicates" anyway. This