I don't think containsAll and retainsAll need to call
collectionNullClean either because SecureSet.contains(null) returns
false now.
--Sean
On 4/28/20 9:58 AM, Weijun Wang wrote:
A new webrev is at
http://cr.openjdk.java.net/~weijun/8243592/webrev.01/
I take this chance to do some formatting and add a new test. My first testng
test!
Thanks,
Max
On Apr 28, 2020, at 8:16 PM, Sean Mullan <sean.mul...@oracle.com> wrote:
On 4/27/20 10:39 PM, Weijun Wang wrote:
Reading the Set spec, it looks like an NPE is still needed for add(), but
remove() can be modified.
Good point, I agree.
--Sean
--Max
On Apr 27, 2020, at 10:27 PM, Sean Mullan <sean.mul...@oracle.com> wrote:
The fix looks fine to me. For consistency, you could make the same change for
null elements in the other SecureSet methods: add, remove.
--Sean
On 4/25/20 3:39 AM, Weijun Wang wrote:
Please take a review at
http://cr.openjdk.java.net/~weijun/8243592/webrev.00/
This is helpful if we do any set arithmetic between 2 Subject objects.
No new regression test, I intend to add a noreg-trivial label.
*Tigran*: Please confirm you are OK with the "Contributed-by" line.
Thanks,
Max