On Sat, 16 Dec 2023 17:51:30 GMT, Markus KARG <[email protected]> wrote:
>> Sergey Tsypanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8320971: Revert irrelevant changes
>
> test/jdk/java/io/BufferedInputStream/TransferToTrusted.java line 85:
>
>> 83: }
>> 84:
>> 85: private static void byteArrayOutputStream(BufferedInputStream bis,
>> byte[] buf) throws IOException {
>
> As we agreed that this PR shall only cover BIS, we can remove the additional
> classes' tests again.
Shouldn't we keep at least the method for the classes checked in
`BIS.isTrusted()` (`ByteArrayOutputStream`, `FileOutputStream`,
`PipedOutputStream`)?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16879#discussion_r1431195222