Jay Soffian <jaysoffian@xxxxxxxxx> writes: > This issue existed before, but the other callers of match_heads() only > call it once before the process terminates, and don't bother to free > what it returns. So those callers didn't notice. This topic introduces > the first caller that calls match_heads() repeatedly, so it frees the > returned ref list after it has copied out what it needs. > > The change is independent and can be moved anywhere in the series, or > even introduced as a patch independent of the series itself. Thanks for explanation. If I am reading the above correctly, a sequence that does not have 10/13 before the patch that makes match_heads() called repeatedly would break bisection, so it cannot be "can be moved anywhere". And it sounds like keeping this as a separate patch would make more sense. Your proposed commit log message for the patch would contain the first paragraph I quoted above, I'd think. -- 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
