On Wed, 13 Dec 2023 08:04:59 GMT, Markus KARG <[email protected]> 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