Sorry, I missed this while being busy hacking and not reading email ;) On Mon, 22 Oct 2007, Jeff King wrote: > > Patch is below (please just squash with the one from Linus). Your patch is better than what used to be there, but > - /* Already picked as a destination? */ > + /* Already picked as a source? */ > if (!p->src_dst) > continue; the above is wrong, the whole thing should be dropped (we *want* to be able to re-use sources). Anyway, the set of fixes I sent out earlier included fixing that stupid loop as one of the things. Linus - 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