+1

--Semyon


On 3/7/18 5:28 AM, Krishna Addepalli wrote:
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