Re: [PATCH V2 1/3] Define hooks infrasructure

2013-04-14 Thread Thomas Adam
Hi, On Sun, Apr 14, 2013 at 06:15:25PM +0100, Nicholas Marriott wrote: > Let nitpicking commence! > > Apart from nitpicks this looks good, will look at the other diffs a bit > later. Thanks. Please see the V3 round of this, including man page entry. -- Thomas Adam

Re: [PATCH V2 1/3] Define hooks infrasructure

2013-04-14 Thread Nicholas Marriott
Let nitpicking commence! Apart from nitpicks this looks good, will look at the other diffs a bit later. On Sun, Apr 14, 2013 at 05:01:36PM +0100, Thomas Adam wrote: > This adds an implementation hooks using a RB tree to store the name of the > hook and the command-list it uses. These will be acc

[PATCH V2 1/3] Define hooks infrasructure

2013-04-14 Thread Thomas Adam
This adds an implementation hooks using a RB tree to store the name of the hook and the command-list it uses. These will be accessed via commands to add/remove hooks. --- Makefile.am | 3 ++ cmd-set-hook.c | 100 ++ cmd-show-hooks.c | 71 +