RFR: 8287860: Revise usage of volatile in j.u.Locale [v3]
Naoto Sato
naoto at openjdk.java.net
Mon Jun 6 20:37:02 UTC 2022
On Mon, 6 Jun 2022 20:19:22 GMT, Сергей Цыпанов <duke at openjdk.java.net> wrote:
>> - cached hash code of `Locale` and `Locale$LanguageRange` shouldn't be volatile, even in case of race in the worst case it is recalculated at most once per thread
>> - `defaultLocale` field is read multiple times in `initDefault()`
>> - `isoLanguages` is accessed multiple times in `getISOLanguages()`
>> - `languageTag` is read twice in `toLanguageTag()`
>
> Сергей Цыпанов has updated the pull request incrementally with one additional commit since the last revision:
>
> 8287860: Rename local vars
Looks good. Thanks for fixing this.
-------------
Marked as reviewed by naoto (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/9041
More information about the core-libs-dev
mailing list