On Tue, 6 Dec 2022 12:43:44 GMT, Daniel Fuchs <[email protected]> wrote:
>> test/jdk/sun/net/www/http/KeepAliveStream/KeepAliveStreamFinalizer.java line
>> 175:
>>
>>> 173: public InputStream getInputStream() throws IOException {
>>> 174: if (finalized) {
>>> 175: System.out.println(failureReason = "getInputStream
>>> called after finalize");
>>
>> For ease of debugging, perhaps use `System.err.println` instead of
>> `System.out`? That way the `Thread.dumpStack()` which is done on the next
>> line will appear in the same jtreg section as this message.
>>
>> Same comment in few other places where we have this construct.
>
> I agree with Jaikiran that since we're not using testng here using
> `System.err` consistently is probably better.
Good point. Replaced `System.out` with `System.err` everywhere.
-------------
PR: https://git.openjdk.org/jdk/pull/11474
