From: Jonathan Nieder <jrnieder@gmail.com>
Date: Sat, 30 Jul 2011 13:10:50 GMT
Subject: Re: [PATCH 16/18] revert: Remove sequencer state when no commits are pending
Message-ID: <20110730131050.GA4848@elie>
In-Reply-To: <CALkWK0=qXBteRjj5vXDEWGVausQ3ssOvy4hyHHz84ORFzDaHaQ@mail.gmail.com>

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.