Hi Semyon,

Modified the code as per your suggestions. Here is the new webrev: 
http://cr.openjdk.java.net/~kaddepalli/8197785/webrev02

Thanks,
Krishna

-----Original Message-----
From: Semyon Sadetsky 
Sent: Tuesday, March 6, 2018 11:46 PM
To: Phil Race <philip.r...@oracle.com>; Krishna Addepalli 
<krishna.addepa...@oracle.com>
Cc: swing-dev@openjdk.java.net
Subject: Re: <Swing Dev> 
[11][JDK-8197785]javax.accessibility.AccessibilityBundle will reload the 
ResourceBundle for every call to toDisplayString

On 03/06/2018 10:01 AM, Phil Race wrote:

>
>
> On 03/06/2018 09:45 AM, Semyon Sadetsky wrote:
>> Can you point to place where this Hashmap is updated other then where 
>> it is initialized?
>
> You mean Hashtable ?
Right, sorry.
>
>  161                 table.put(locale, resourceTable);
>
> Will be executed once per locale.
It updates "table" not "resourceTable". The "table" field could be treated 
differently since writing in it really very rare but it is ok to leave it as it 
is. All what I meant concerned only the "resourceTable".

--Semyon
>
> -phil.
>

Reply via email to