RFR: JDK-8153768 Miscellaneous changes imported from jsr166 CVS 2016-05
Paul Sandoz
paul.sandoz at oracle.com
Tue May 24 11:57:58 UTC 2016
> On 24 May 2016, at 01:03, Martin Buchholz <martinrb at google.com> wrote:
>
> On Mon, May 23, 2016 at 3:29 PM, Aleksey Shipilev
> <aleksey.shipilev at oracle.com> wrote:
>> On 05/23/2016 11:32 PM, Martin Buchholz wrote:
>>> JSR166 CVS has some unfinished work in progress, but enough minor
>>> changes have accumulated that we should integrate into openjdk9:
>>>
>>> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/miscellaneous/
>>
+1
>> *) Removing Cloneable from submaps probably requires CCC?
>
> It never appears in the API.
Yes, cannot write Java source to call clone on the returned ConcurrentNavigableMap, so i think we are ok here.
Paul.
> The use of Cloneable here was a mistake
> - the clone method was never implemented! So only users who checked
> for "implements Cloneable" without actually trying to call clone could
> be affected. And who would try to clone submaps anyways?
>
>> *) Want to be extra modern in sample code? Use method references!
>> - * solvers.forEach((solver) -> cs.submit(solver));
>> + * solvers.forEach(cs::submit);
>
> OK - switched to using method references!
More information about the core-libs-dev
mailing list