Subject: Re: [PATCH 16/18] revert: Remove sequencer state when no commits are pending

From: Jonathan Nieder <[email-masked]>
Date: Sat, 30 Jul 2011 15:10:50 +0200

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.