On Tue, 6 Dec 2022 12:43:44 GMT, Daniel Fuchs <dfu...@openjdk.org> 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