On Fri, 25 Jun 2021 19:39:22 GMT, Valerie Peng <valer...@openjdk.org> wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Move TokenPoller to Runnable
>
> src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/SunPKCS11.java line
> 952:
>
>> 950: AccessController.doPrivileged((PrivilegedAction<Void>) () -> {
>> 951: Thread t = InnocuousThread.newSystemThread(
>> 952: "Poller " + getName(),
>
> nit: "Poller " -> "Poller-" (like before)?
It seems that the AccessController.doPrivileged((PrivilegedAction<Void>) () ->
{} is un-necessary? I tried your test without it and test still passes.
-------------
PR: https://git.openjdk.java.net/jdk17/pull/117
