Re: [PATCH 16/18] revert: Remove sequencer state when no commits are pending
- From
Jonathan Nieder <jrnieder@gmail.com>
- Date
- Jul 30, 2011, 13:10 UTC
- 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.