groovy

Defensive copying doesn't work for @Immutable classes

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7.10
  • Fix Version/s: 1.8-rc-4, 1.7.11
  • Component/s: None
  • Labels:
    None
  • Testcase included:
    yes
  • Number of attachments :
    0

Description

Seems like @Immutable classes do not do defensive copying of the collection provided into constructor:

@Immutable
final class Person {
    String name
    List<String> address
}

def myAddress = ["Line1","Line2"]
def person = new Person(name:"Name", address:myAddress)
assert person.address == ["Line1","Line2"]
myAddress << "Line3"
assert myAddress == ["Line1","Line2","Line3"]
assert person.address == ["Line1","Line2"] //<- fails

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul King added a comment -

Workaround is to use ArrayList not List as the type for address.

Show
Paul King added a comment - Workaround is to use ArrayList not List as the type for address .
Hide
Permalink
Paul King added a comment -

Thanks for spotting that. Should be fixed now.

Show
Paul King added a comment - Thanks for spotting that. Should be fixed now.

People

  • Assignee:
    Paul King
    Reporter:
    Denis Solonenko
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: