As for the Context vs. Map issue, I think the approach I'll take in the Morph framework is to have the DelegatingContext class implement Map. That way, in most situations the Map interface will be very easily accessible. Also, to create a new Context implementation, only the methods defined in the BeanReflector interface will have to be implemented, which is a lot easier than implementing the entire Map contract.
Just to give you an idea how easy the BeanReflector interface is to implement, the entire implementation is only 45 lines for a reflector that exposes HTTP request parameters. That's in contrast to 121 lines in org.apache.commons.chain.web.servlet.ServletParamMap. (Note, these counts start at the line of code that contains the opening brace for the class... for example, at the line 'final class ServletParamMap implements Map {')
Matt
----- Original Message ----- From: "Shapira, Yoav" <[EMAIL PROTECTED]>
To: "Jakarta Commons Developers List" <[EMAIL PROTECTED]>
Sent: Monday, November 22, 2004 8:47 AM
Subject: RE: [VOTE] Release Chain 1.0
Hi,
--------------------------------------------------------------- [ X ] +1 I support this release and am willing to help [ ] +0 I support this release but am unable to help [ ] -0 I do not support this release [ ] -1 I do not support this release, and here are my reasons ---------------------------------------------------------------
The design discussions have merit, but they should be deferred to version 1.1 or later, and should not hold up the 1.0 release.
Yoav
This e-mail, including any attachments, is a confidential business communication, and may contain information that is confidential, proprietary and/or privileged. This e-mail is intended only for the individual(s) to whom it is addressed, and may not be saved, copied, printed, disclosed or used by anyone else. If you are not the(an) intended recipient, please immediately delete this e-mail from your computer system and notify the sender. Thank you.
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
