On Wed, 13 Dec 2023 08:04:59 GMT, Markus KARG <d...@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