On Mon, 11 Dec 2023 14:54:37 GMT, Markus KARG <d...@openjdk.org> wrote:
>> Sergey Tsypanov 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 11 additional >> commits since the last revision: >> >> - Merge branch 'master' into 8320971 >> - 8320971: Rename method >> - 8320971: Extract utility class >> - 8320971: Rearrange code >> - 8320971: Rewrite comment >> - 8320971: Use same approach as BAOS >> - Merge branch 'master' into 8320971 >> - 8320971: Add test >> - 8320971: Trust any OutputStream from java.* >> - 8320971: Use BufferedInputStream.class.getPackageName() >> - ... and 1 more: https://git.openjdk.org/jdk/compare/2ac25f70...2df52c72 > > src/java.base/share/classes/java/io/IOStreams.java line 1: > >> 1: /* > > Please fix Whitespace errors. Thanks. :-) I've just copy-pasted the header from another class. How should I fix it? > src/java.base/share/classes/java/io/IOStreams.java line 39: > >> 37: * <pre> >> 38: * - the reference to {@code byte[]} is not kept within the class >> 39: * - the argument of {@link OutputStream#write(byte[])}} and {@link >> OutputStream#write(byte[], int, int)}} is not modified within the methods > > `- the byte[] is not read outside of the given bounds` Added ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16879#discussion_r1423051206 PR Review Comment: https://git.openjdk.org/jdk/pull/16879#discussion_r1423052717