On Fri, Apr 1, 2016 at 12:56 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Christian Couder <christian.couder@xxxxxxxxx> writes: > >> On Wed, Mar 16, 2016 at 3:35 PM, Christian Couder >> <christian.couder@xxxxxxxxx> wrote: >>> When parse_chunk() fails it can return -1, for example >>> when find_header() doesn't find a patch header. >>> >>> In this case it's better in apply_patch() to free the >>> "struct patch" that we just allocated instead of >>> leaking it. >> >> Maybe this patch has fallen through the cracks too. > > Anything worthy of discussion that you sent during the feature > freeze, please resend them to the list for discussion. It looks like only the two patches I replied to have not been applied to next. I had also sent a three patch long series (http://thread.gmane.org/gmane.comp.version-control.git/289559), but we agreed that only two of them could be merged for now and that's what you did already (fda3e2c and db354b7). By the way my guess is that replying to them is ok, but you ask to "resend them", so I am wondering if you really prefer them to be resent rather than just replied to. Thanks! -- 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