Hi Emmanuel,

Most of the logic in the chain can be moved out into simple stupid methods
removing much of
the complexity.  For now we might just flatten and use a method to do what
each link was doing.
I have no idea why the chain pattern was used in the first place.  Do you
know?  We might use
CoR in the core for intercepting calls but here I cannot see any reason for
it.  It's a damn mess.

After you're done refactoring I might clean up a tiny bit more.  I think we
need to take a pass
to just review the code perhaps looking for more cleanups we can make.  Then
we can get on
with Phase II.  If you like we can do a peer review of the entire state of
the server.  Perhaps
around the weekend or right after. WDYT?

I'll be hard to reach tomorrow but tonight I will review some of those new
interfaces for Phase II.

Alex

Reply via email to