On Fri, Aug 26, 2016 at 10:10:50AM -0700, Junio C Hamano wrote: > Lars Schneider <larsxschneider@xxxxxxxxx> writes: > > > I agree with your criticism of the code duplication. > > > > However, I thought it would be OK, as Peff already > > tried to refactor it... > > http://public-inbox.org/git/20160810150139.lpxyrqkr53s5f4sx@xxxxxxxxxxxxxxxxxxxxx/ > > > > ... and I got the impression you agreed with Peff: > > http://public-inbox.org/git/xmqqvaz84g9y.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx/ > > The former does not exactly show how ugly it was, but I do not have > to see it. It is talking about eliminating the need for memcpy() > and duplicated header generation code, which the suggestion you are > responding to didn't even attempt. If Peff said he tried an even > more aggressive refactoring and it ended up too ugly to live, I > believe him and agree with his assessment. Right, what I found difficulty with was factoring out format_packet(). I think the "gently" part is easy. -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