Kevin Willford <kcwillford@xxxxxxxxx> writes: > From: Kevin Willford <kewillf@xxxxxxxxxxxxx> > > The cherry_pick_list was looping through the original side checking the > seen indicator and setting the cherry_flag on the commit. If we save > off the commit in the patch_id we can set the cherry_flag on the correct > commit when running through the other side when a patch_id match is found. > > Signed-off-by: Kevin Willford <kcwillford@xxxxxxxxx> > --- > patch-ids.c | 1 + > patch-ids.h | 2 +- > revision.c | 18 +++--------------- > 3 files changed, 5 insertions(+), 16 deletions(-) We lose the final loop to fix up the shorter side, because the second loop now marks both sides. And as a side effect, we do not use commit->util which is a scarce shared resource (aka historical API wart). Makes sense. 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