RFR: 8272120: Avoid looking for standard encodings in "java." modules
Daniel Fuchs
dfuchs at openjdk.java.net
Wed Aug 11 09:15:24 UTC 2021
On Tue, 10 Aug 2021 05:08:54 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
> This is the continuation of JDK-8233884 and JDK-8271456. This change affects fewer cases so I fix all "java." modules at once.
>
> In many places standard charsets are looked up via their names, for example:
> absolutePath.getBytes("UTF-8");
>
> This could be done more efficiently(up to x20 time faster) with use of java.nio.charset.StandardCharsets:
> absolutePath.getBytes(StandardCharsets.UTF_8);
>
> The later variant also makes the code cleaner, as it is known not to throw UnsupportedEncodingException in contrary to the former variant.
>
> tested by the tier1/tier2/tier3 tests on Linux/Windows/macOS.
>From my pure developer's perspective the proposed changes look good. If the performance concerns are removed I'd say it looks good to me.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5063
More information about the security-dev
mailing list