<html><body><div><div dir="auto">Looks good to me. <div><br></div><div>Thanks, <br>   Tigran</div></div><br><br>-------- Original message --------<br>From: Weijun Wang <weijun.wang@oracle.com><br>Date: Sat, Apr 25, 2020, 09:40<br>To: security-dev@openjdk.java.net<br>Cc: "Mkrtchyan, Tigran" <tigran.mkrtchyan@desy.de><br>Subject: RFR 8243592: Subject$SecureSet::contains(null) is suboptimal<br></div><br><div>Please take a review at

<br>

<br>   <a href="http://cr.openjdk.java.net/~weijun/8243592/webrev.00/">http://cr.openjdk.java.net/~weijun/8243592/webrev.00/</a>

<br>

<br>This is helpful if we do any set arithmetic between 2 Subject objects.

<br>

<br>No new regression test, I intend to add a noreg-trivial label.

<br>

<br>*Tigran*: Please confirm you are OK with the "Contributed-by" line.

<br>

<br>Thanks,

<br>Max

<br>

<br></div></body></html>