On Tue, 16 Mar 2021 14:35:12 GMT, Michael McMahon <micha...@openjdk.org> wrote:

>> Julia Boes has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - wrap List<String> with unmodifiable map and update test
>>  - remove map wrapping
>
> Changes requested by michaelm (Reviewer).

The changes LGTM. What about modifying ExchangeImpl to create the 
UnmodifiableHeaders in its constructor, instead of creating a new instance 
every time getRequestHeaders() is called?

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

PR: https://git.openjdk.java.net/jdk/pull/3032

Reply via email to