On Fri, 29 Sep 2023 18:56:26 GMT, 温绍锦 <d...@openjdk.org> wrote:
>> Raffaello Giulietti has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Uppercase JLA. > > src/java.base/share/classes/jdk/internal/math/DoubleToDecimal.java line 110: > >> 108: private static final int NAN = 5; >> 109: >> 110: private static final JavaLangAccess JLA = >> SharedSecrets.getJavaLangAccess(); > > JLA is defined repeatedly in many places. Is this a good practice? Can we > use it directly, like this: > > return SharedSecrets > .getJavaLangAccess() > .newStringNoRepl(bytes, StandardCharsets.ISO_8859_1); @wenshao I'll take a note for the next commit, thanks. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/15861#discussion_r1341741225