What do you think of this ImmutableList class defi...

# random

[gsala]

What do you think of this ImmutableList class definition?

class ImmutableList<T>(list: List<T>) : List<T> by list.toList()

[LeoColman]

I don't think it's a given that toList will return you a new list

[LeoColman]

So maybe it's not immutable?

[Loney Chou]

toList will give you a new ArrayList

[Chris Fillmore]

You might be interested in this
https://github.com/Kotlin/kotlinx.collections.immutable

[gsala]

@LeoColman good point, it depends on the implementation of toList. What about declaring it as:

class ImmutableList<T>(list: List<T>) : List<T> by ArrayList(list)

[gsala]

we miss the empty list and singletonlist optimizations from inside toList but that’s probably alright