Nevin Pratt wrote:

> The new 'ChieftainModule' class in the newer Comanche is interesting. 
> Congratulations and thanks to whoever did this.
>
> One comment though.  At the end of the #process: method, where the 
> comment reads "otherwise, the default response", it just returns nil. 
> In other words, "do nothing" is the default.  I suggest we allow a 
> default that is a bit more meaningful than this.  I suggest we be able 
> to install any Comanche module and use it as the default, and only 
> return nil as the default when there hasn't been a specific Comanche 
> module installed as the default handler.
>
> So, we add a 'default' instance variable, give it a setter method, and 
> send the request to the default (if it exists) instead of answering nil. 

I was thinking about #defaultPrefix. In case of request for root (/) it 
would send response with redirect (header Location:) to that defaultPrefix.

> But otherwise, I think the ChieftainModule is nice, clean, 
> straight-forward, and (of course) simple.
>
> Nevin

Oh yea... but not evolving somehow. There's todo list in class comment 
and last change claims to be on 6/21/2001 03:08.

-- 
Kamil