Re: [PATCH 14/16] fetch-pack: refactor parsing in get_ack

On Wed, Jun 18, 2014 at 3:56 PM, Jeff King p...@peff.net wrote: There are several uses of the magic number line+45 when parsing ACK lines from the server, and it's rather unclear why 45 is the correct number. We can make this more clear by keeping a running pointer as we parse, using

[PATCH 14/16] fetch-pack: refactor parsing in get_ack

There are several uses of the magic number line+45 when parsing ACK lines from the server, and it's rather unclear why 45 is the correct number. We can make this more clear by keeping a running pointer as we parse, using skip_prefix to jump past the first ACK , then adding 40 to jump past