8 Feb
2018
8 Feb
'18
12:38 a.m.
Hi, On 2018-02-07 15:23, Claes Redestad wrote:
- use map.replace to safely update the entry when putIfAbsent returns an object but it points to a cleared value, so that (benign) races to create new Locale objects will canonicalize
turns out the implementation attempted here with map.replace was problematic, causing certain tests to fail. Seems there are subtle issues here around establishing a stable equality relationship which may or may not be easy to resolve, so I reverted back these particular changes to LocaleObjectCache from this RFE. I also did some cleanup in BaseLocale based on offline feedback from Paul: http://cr.openjdk.java.net/~redestad/8196869/jdk.03/ Thanks! /Claes