Jeff King <peff@xxxxxxxx> writes: > On Wed, Sep 21, 2016 at 07:45:50PM +0200, Kevin Daudt wrote: > >> On Wed, Sep 21, 2016 at 10:36:57AM -0700, Junio C Hamano wrote: >> > Kevin Daudt <me@xxxxxxxxx> writes: >> > >> > > On Mon, Sep 19, 2016 at 04:30:34PM -0700, Junio C Hamano wrote: >> > >> >> > >> * kd/mailinfo-quoted-string (2016-09-19) 2 commits >> > >> - mailinfo: unescape quoted-pair in header fields >> > >> - t5100-mailinfo: replace common path prefix with variable >> > > >> > > Is this good enough, or do you want me to look into the feedback from >> > > jeff? >> > >> > If you are talking about the simplified loop that deliberately sets >> > a rule that is looser than RFC, yes, I'd like to see you at least >> > consider the pros and cons of his approach, which looked nicer to my >> > brief reading of it. >> > >> > It is perfectly OK by me (it may not be so if you ask Peff) if you >> > decide that your version is better after doing so, though. >> >> Alright, I'll look into it. > > Thanks. I am OK if we do not use my simplified version, but I think > there were some issues I noted with your last version. Yup, even some automated tool noticed a new leak ;-)