RFR: 8320971: Use BufferedInputStream.buf directly when param of implTransferTo() is trusted [v12]

Sergey Tsypanov stsypanov at openjdk.org
Wed Dec 13 09:08:50 UTC 2023


On Wed, 13 Dec 2023 08:04:59 GMT, Markus KARG <duke at openjdk.org> wrote:

>> Sergey Tsypanov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8320971: Whitespaces
>
> src/java.base/share/classes/com/sun/io/IOStreams.java line 53:
> 
>> 51:     public static boolean isTrusted(OutputStream os) {
>> 52:         var clazz = os.getClass();
>> 53:         return clazz == ByteArrayOutputStream.class
> 
> I wonder what the committers think: Do we need a unit test for each trusted class to proof that they actually behave accordingly to the rules of trust listed in the Javadocs?

Added tests for all involved OutputStreams

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16879#discussion_r1425050812


More information about the core-libs-dev mailing list