On Wed, 8 Mar 2023 11:37:03 GMT, Daniel Fuchs <[email protected]> wrote:

>> The HttpClient internal DebugLogger could be simplified if its configuration 
>> was held in a record. Some of the methods in Utils that return a debug 
>> logger could also be simplified/removed. The system property that configures 
>> the debug logging could be extended to choose between System.err, 
>> System.out, System.Logger, or any of these configurations.
>> Passing -Djdk.internal.httpclient.debug=true still works as before.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Use HashSet.newHashSet(capacity)

Thank you Daniel for the changes. The latest state in commit `0adb05bb` looks 
good to me.

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

Marked as reviewed by jpai (Reviewer).

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

Reply via email to