On Mon, Feb 18, 2013 at 01:25:35PM -0800, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > > But it's easy to do (1), and it starts the clock ticking for > > the 1000-byte readers to become obsolete. > > Yup, I agree with that goal. Having just looked at the pkt-line callers a lot, I think most of them could go for something like: char *packet_read(int fd, unsigned *len_p) { static char buffer[LARGE_PACKET_MAX]; int len; len = packet_read_to_buf(fd, buffer, sizeof(buffer)); if (len < 0) return NULL; *len_p = len; return buffer; } That would actually simplify the callers a bit, and would harmonize the buffer sizes at the same time. I'll look into doing a series tonight. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html