On Fri, 4 Feb 2022 23:02:21 GMT, Xue-Lei Andrew Fan wrote:
> Please review this trivial code clean up, for a little bit better performance.
There's a small compatibility risk with this change, e.g.,
`List.copyOf(...).contains(null)` will throw NPE while
`Collections.unmodifiableList(...).conta
On Sat, 5 Feb 2022 13:22:25 GMT, Claes Redestad wrote:
> There's a small compatibility risk with this change, e.g.,
> `List.copyOf(...).contains(null)` will throw NPE while
> `Collections.unmodifiableList(...).contains(null)` won't.
>
> If we accept that compatibility risk (which should probab
In [JDK-8281289](https://bugs.openjdk.java.net/browse/JDK-8281289), the
SSLParameters class was updated with the patch:
- return Collections.unmodifiableList(new
ArrayList<>(sniNames.values()));
+ return List.copyOf(sniNames.values());
However, there's a small compatibility risk with this chang
