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. >
