Re: RFR 8191216: SimpleTimeZone.clone() has a data race on cache fields

2017-12-06 Thread Naoto Sato
Hi Peter, Venkat, Thank you for the fix. It looks good to me. Improved performance is a nice bonus! Would you be able to provide with a regression test? Naoto On 12/6/17 6:10 AM, Peter Levart wrote: Hi, On 12/06/2017 02:30 PM, Alan Bateman wrote: I think this class is normally maintained on

Re: RFR 8191216: SimpleTimeZone.clone() has a data race on cache fields

2017-12-06 Thread Peter Levart
Hi, On 12/06/2017 02:30 PM, Alan Bateman wrote: I think this class is normally maintained on i18n-dev but I think introducing the Cache object looks good and making this much easier to understand. -Alan Thanks Alan, I'm forwarding to i18n-dev to see if maintainers of that part of JDK have