RFR(m): 8177290 add copy factory methods for unmodifiable List, Set, Map
Stuart Marks
stuart.marks at oracle.com
Thu Sep 21 18:55:54 UTC 2017
On 9/21/17 5:42 AM, Alan Bateman wrote:
> On 21/09/2017 01:02, Stuart Marks wrote:
>> http://cr.openjdk.java.net/~smarks/reviews/8177290/webrev.0/
> I read through the updated/new definitions and they read well.
Great.
> For the copyOf methods then I can't immediately tell from the javadoc if the
> given collection can contain null elements. Taking List.copyOf as an example
> where coll may be null or it may contain null elements. The javadoc does link to
> "Unmodifiable lists" where it specifies the characteristics of the lists
> returned by the static factory methods - these include disallowing null
> elements. So I think this needs to be clarified.
Agreed, I'll work on some clarifications here, and also disallow null for the
argument itself.
> Minimal implementation is okay to get started but what is the reason not to
> include some basic tests?
Sorry, I should have been more clear about this. The changeset is clearly not
ready to go in as it stands. I wanted to get an initial review of the
specifications going, then file a CSR request, etc. while continuing to work on
tests and better implementations. I'll post a subsequent review when they're ready.
Thanks.
s'marks
More information about the core-libs-dev
mailing list