On Nov 30, 2005, at 4:40 AM, mario ruggier wrote:
>
> However, wouldn't it be more correct if it used self.__class__
> instead of DurusDirectory to use itself recursively?

Okay, done.

>
> The other suggestion is for the display method, that nests the
> format() function. Because this nested function is rather generic
> object formatter, and display() is really about page layout,
> wouldn't it be better to just make format() an external generic
> helper function? Actually, I would also suggest to collect these
> "dev/debug/useful" functions in one place... I have for example a
> fill.debug module where I have put format_sessions(), format_request
> (), and format_p_obj().

The advantage of having it as a nested function is that it is
easier to change later.  I'm not ready to set these particular
formatting functions in stone.