On Fri, May 20, 2005 at 08:35:56AM -0700, Titus Brown wrote:
> Here's the corrected patch, introducing warn() statements appropriately
> and removing internal dependence on the dict-like interface.

Thanks for spending the time to make a patch.  However, I think
SessionManager should provide some kind of API for accessing the
managed sessions.  Forcing the application poke around in .sessions
breaks encapsulation and is a step backward, IMHO.  Upon reflection,
while the dict interface is not ideal maybe it's not so bad either.
Maybe it's not worth changing.

  Neil
_______________________________________________
Quixote-users mailing list
[email protected]
http://mail.mems-exchange.org/mailman/listinfo/quixote-users

Reply via email to