RFR: 8303682: Simplify HttpClient DebugLogger [v3]

Daniel Fuchs dfuchs at openjdk.org
Wed Mar 8 11:37:10 UTC 2023


On Wed, 8 Mar 2023 11:01:07 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use HashSet.newHashSet(capacity)
>
> test/jdk/java/net/httpclient/DebugLoggerTest.java line 242:
> 
>> 240:         out =  new RecordingPrintStream(stdOut, true, UTF_8);
>> 241:         System.setErr(err);
>> 242:         System.setOut(out);
> 
> Should we reset these back to their original streams in a `finally` block?

The test runs in /othervm and the RecordingPrintStream acts like a `tee` so it should not matter.

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

PR: https://git.openjdk.org/jdk/pull/12900


More information about the net-dev mailing list