RFR: 8272805: Avoid looking up standard charsets [v2]
Sergey Bylokhov
serb at openjdk.java.net
Mon Aug 23 19:34:38 UTC 2021
On Sun, 22 Aug 2021 15:09:26 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Sergey Bylokhov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 14 additional commits since the last revision:
>>
>> - Update the usage of Files.readAllLines()
>> - Update datatransfer
>> - Merge branch 'master' into standard-encodings-in-non-public-modules
>> - Merge branch 'master' into standard-encodings-in-non-public-modules
>> - Fix related imports
>> - Merge branch 'master' into standard-encodings-in-non-public-modules
>> - Cleanup UnsupportedEncodingException
>> - Update PacketStream.java
>> - Rollback TextTests, should be compatible with jdk1.4
>> - Rollback TextRenderTests, should be compatible with jdk1.4
>> - ... and 4 more: https://git.openjdk.java.net/jdk/compare/465eb90c...e7127644
>
> src/jdk.attach/linux/classes/sun/tools/attach/VirtualMachineImpl.java line 342:
>
>> 340:
>> 341: try {
>> 342: for (String line : Files.readAllLines(statusPath, UTF_8)) {
>
> The 1-arg readAllLines is specified to use UTF-8 so you can drop the second parameter here if you want.
Thank you for the suggestion, I have fixed this and a few other places as well.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5210
More information about the security-dev
mailing list