mhag...@alum.mit.edu writes:
> Re-roll, incorporating Jeff's suggestions. Some commit messages have
> also been improved, but the only interdiff is that match_pos is
> renamed to head_pos in filter_refs().
>
> This patch series applies to the merge between master and
> jc/maint-push-refs-all, though the dependency on the latter is only
> textual.
I've read this twice and saw nothing questionable in the series
(other than that the titles of a few weren't clear enough when
placed in a short-log like below). Very nicely done.
Thanks. Will queue.
> Michael Haggerty (17):
> t5500: add tests of error output for missing refs
> Rename static function fetch_pack() to http_fetch_pack()
> Fix formatting.
> Name local variables more consistently
> Do not check the same head_pos twice
> Let fetch_pack() inform caller about number of unique heads
> Pass nr_heads to do_pack_ref() by reference
> Pass nr_heads to everything_local() by reference
> Pass nr_heads to filter_refs() by reference
> Remove ineffective optimization
> filter_refs(): do not leave gaps in return_refs
> filter_refs(): compress unmatched refs in heads array
> cmd_fetch_pack: return early if finish_connect() returns an error
> Report missing refs even if no existing refs were received
> cmd_fetch_pack(): simplify computation of return value
> fetch_pack(): free matching heads
> filter_refs(): simplify logic
>
> builtin/fetch-pack.c | 128
> --
> fetch-pack.h | 19 +---
> http-walker.c | 4 +-
> t/t5500-fetch-pack.sh | 32 -
> transport.c | 8 ++--
> 5 files changed, 101 insertions(+), 90 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html