[ https://issues.apache.org/jira/browse/CLI-21?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509022 ]
Henri Yandell commented on CLI-21: ---------------------------------- Another thought - as I kick myself to keep momentum on the CLI release. Need to find out if a change from: public Object clone() to protected Object clone() throws CloneNotSupportedException will be voted down. I presume it will. > [cli] clone method in Option should use super.clone() > ----------------------------------------------------- > > Key: CLI-21 > URL: https://issues.apache.org/jira/browse/CLI-21 > Project: Commons CLI > Issue Type: Bug > Components: CLI-1.x > Affects Versions: 1.0 > Environment: Operating System: other > Platform: Other > Reporter: Nathan McDonald > Fix For: 1.1 > > Attachments: bug21.patch > > > In > org.apache.commons.cli.Option > public method clone is implemented by creating a new instance through one of > the class Constructors, and then assigning values as required through the > setter methods. > This means that any subclasses of the Option class will not return a true > clone, but a new Option instance instead. > A proper implementation of clone should use super.clone() to create a new > instance, rather than calling the class constructor. This allows shallows > clones to propogate correctly down to subclasses. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]