From: Ramkumar Ramachandra Date: Thu, 28 Jul 2011 16:16:50 GMT Subject: Re: [PATCH 16/18] revert: Remove sequencer state when no commits are pending Message-ID: 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