From: Ramkumar Ramachandra Date: Tue, 02 Aug 2011 04:54:43 GMT Subject: [GSoC update] Iterating over a stable series Message-ID: <1312260884-5087-1-git-send-email-artagnon@gmail.com> Hi, The post midterm work is suffering because I'm constantly rewriting the sequencer-stable series: I often resort to throwing away big patches that move functions from builtin/revert.c to sequencer.c. Hopefully, the latest iteration [1] will not require rewriting. I'd like some early feedback for one of the "design patches" in my new series: I've chosen to use a commit + action to represent a todo_list. I'd initially tried a commit + opts keeping future expantion in mind (allowing instruction-specific command-line options), but the result is quite inelegant. Although commit message/ tests are missing, I'd like to describe the intent in detail: This patch is a prerequisite for decoupling todo parsing from opts parsing. I want to decouple them so that I can achieve tighter coupling between "git commit" and the sequencer [2]. After that, I want to craft a nice API and move/ expose various functions in builtin/revert.c starting with the parsing functions. Thanks for reading. [1]: http://thread.gmane.org/gmane.comp.version-control.git/178372 [2]: http://mid.gmane.org/CALkWK0=9kwgtZB-BA12tOQrQXS8XRbhTg6K=Ak00o2nurX16Fg@mail.gmail.com Ramkumar Ramachandra (1): revert: Allow arbitrary sequencer instructions builtin/revert.c | 101 +++++++++++++++++++++++++++++------------------------- sequencer.h | 10 +++++ 2 files changed, 64 insertions(+), 47 deletions(-) -- 1.7.6.351.gb35ac.dirty