DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=29908>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29908

clone method in Option should use super.clone()

           Summary: clone method in Option should use super.clone()
           Product: Commons
           Version: unspecified
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: CLI
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


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.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to