Re: [PATCH 16/18] revert: Remove sequencer state when no commits are pending
- From
Ramkumar Ramachandra <artagnon@gmail.com>
- Date
- Jul 28, 2011, 16:16 UTC
- Message-ID
- <CALkWK0mUGiCRQoW6UM+mF5tsn4DxRuaiXn0TS2CcEsR0OeusKg@mail.gmail.com>
- In-Reply-To
- <20110727154830.GA29924@elie>
Hi again,
Jonathan Nieder writes:
> If "tight coupling" means "builtin/commit.c calls a function declared > in sequencer.h", then yes, I think it should be fine.
My bad. I'll first have to figure out a way to expose the todo parsing functions (and dependent functions) without touching much else. This'll be a huge non-trivial patch, and my post midterm work has to include this anyway. So I think it's better to leave it as-it-is for now, and work on this coupling after this series is merged.
-- Ram