[ http://issues.apache.org/jira/browse/WODEN-40?page=comments#action_12446068 ] Jeremy Hughes commented on WODEN-40: ------------------------------------
#1 and #2 are resolved. #3 and #4 are not. We could either keep this open to show that #3 and #4 are still to do, or close this and open a new issue for #3 & #4 or just close this and not open a new issue (because no-one has actually asked for the impl classes to implement clone() ... it's a lot of work for something we're not sure users want. So I favour closing this and not opening a new issue. Anyone object? Please say so otherwise I'll close this in 72hrs. > Use composition in the element model instead of aggregation > ----------------------------------------------------------- > > Key: WODEN-40 > URL: http://issues.apache.org/jira/browse/WODEN-40 > Project: Woden > Issue Type: Improvement > Components: Parser > Reporter: Jeremy Hughes > Assigned To: Jeremy Hughes > > Following on from this list discussion: > http://mail-archives.apache.org/mod_mbox/ws-woden-dev/200606.mbox/[EMAIL > PROTECTED] > 1. remove addXxxElement methods. Change semantics of createXxxElement to > "create and add" > 2. distribute createXxxElement methods so they appear on the YyyElement > interface where Yyy is the parent element of Xxx > 3. XxxElementImpl to implement Cloneable and clone() to carry out a deep clone > 4. add createXxxElement(XxxElement) methods to deep clone the parameter and > add the clone to the target object -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
