accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey (JIRA)" <>
Subject [jira] [Created] (ACCUMULO-2619) Review use of Cloneable
Date Thu, 03 Apr 2014 08:47:15 GMT
Sean Busbey created ACCUMULO-2619:

             Summary: Review use of Cloneable
                 Key: ACCUMULO-2619
             Project: Accumulo
          Issue Type: Improvement
            Reporter: Sean Busbey
            Priority: Minor

We have a few places that claim cloneable, including in the public API. Go through and bring
them in line with Effective  Java.

Note that this means where possible we should favor a copy constructor to using the clone
method, since it is fundamentally broken.

Where we must have it we need to make sure we obey all the rules.

# return the type of the class implementing Cloneable
# do not throw CloneNotSupportedException
# make sure to return the result of super.clone() (unless the class is final)
# any fields that are not safe to share between an object and its clone must be deep copied
after using super.clone().

This message was sent by Atlassian JIRA

View raw message