Glen Daniels wrote:
+1!
However, this is a slightly more complicated problem. The
engageModule(AxisModule, AxisConfiguration) API comes from
AxisDescription, so we'd need to change THAT one to really do this. This
is exactly what I think we should do, since all AxisDescription objects
should be able to find their AxisConfiguration without any problem -
therefore they shouldn't need to have it passed in (and in fact even
leaving the API the option of passing it means that it's possible for
people to screw up and pass a *different* AxisConfiguration than the one
the current object belongs to... good APIs should protect against things
like this).
So I'd propose we remove the first version, and refactor the second
version up to AxisDescription.
+1 from me.
Sanjiva.
--
Sanjiva Weerawarana, Ph.D.
Founder & Director; Lanka Software Foundation; http://www.opensource.lk/
Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
Director; Open Source Initiative; http://www.opensource.org/
Member; Apache Software Foundation; http://www.apache.org/
Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
