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

Reply via email to