RFR: 8196869: Optimize Locale creation
Paul Sandoz
paul.sandoz at oracle.com
Thu Feb 8 02:06:15 UTC 2018
This is looking much better, definitely easier to understand.
Paul.
> On Feb 7, 2018, at 4:38 PM, Claes Redestad <claes.redestad at oracle.com> wrote:
>
> 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
More information about the core-libs-dev
mailing list