[PATCHv2 05/10] pkt-line: rename s/packet_read_line/packet_read/

Originally packets were used just for the line-oriented ref advertisement and negotiation. These days, we also stuff packfiles and sidebands into them, and they do not necessarily represent a line. Drop the _line suffix, as it is not informative and makes the function names quite long (especially

Re: [PATCHv2 05/10] pkt-line: rename s/packet_read_line/packet_read/

Jeff King wrote: Originally packets were used just for the line-oriented ref advertisement and negotiation. These days, we also stuff packfiles and sidebands into them, and they do not necessarily represent a line. Drop the _line suffix, as it is not informative and makes the function names

Re: [PATCHv2 05/10] pkt-line: rename s/packet_read_line/packet_read/

On Mon, Feb 18, 2013 at 02:19:15AM -0800, Jonathan Nieder wrote: Jeff King wrote: Originally packets were used just for the line-oriented ref advertisement and negotiation. These days, we also stuff packfiles and sidebands into them, and they do not necessarily represent a line. Drop

Re: [PATCHv2 05/10] pkt-line: rename s/packet_read_line/packet_read/

Jeff King wrote: On Mon, Feb 18, 2013 at 02:19:15AM -0800, Jonathan Nieder wrote: In combination with patch 3, this changes the meaning of packet_read() without changing its signature, which could make other patches cherry-picked on top change behavior in unpredictable ways. :( So I'd be