On Wed, 28 Aug 2024 20:36:06 GMT, Kevin Driver <kdri...@openjdk.org> wrote:

>> test/jdk/security/unsignedjce/java.base/javax/crypto/ProviderVerifier.java 
>> line 1:
>> 
>>> 1: /*
>> 
>> This is exact the same code as the one in OpenJDK.
>> For the Delayed.java, it just needs the bare minimum, e.g. do not error out 
>> when verify() is called. For testing purpose, maybe we can just use a stub 
>> version instead of duplicating the OpenJDK one? If later, the OpenJDK one 
>> changed, people may wonder what this is for and have to read through the 
>> whole impl, If it's just the stub version, it's very clear.
>
> Resolved in 
> https://github.com/openjdk/jdk/pull/20301/commits/a65dc9da044297c313e02d698255bad915dc0a10.

Good.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1735397788

Reply via email to