Ramkumar Ramachandra wrote: > Okay, I don't follow. Wouldn't this function need to parse the todo > sheet and figure out how many instructions are left? For that, the > todo parsing functions can't be buried in builtin/revert.c, no? Hence > I said "expose" them -- or move them to sequencer.c. Oh, that makes sense. Sorry for the nonsense. If I were in your shoes, I'd move everything below pick_commits to sequencer.c. -- 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