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