On Wed, 16 Nov 2022 14:12:05 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   comment and exception message
>
> test/jdk/sun/security/x509/AlgorithmId/Uninitialized.java line 35:
> 
>> 33: import java.security.AlgorithmParameters;
>> 34: 
>> 35: public class Uninitialized {
> 
> Is this test necessary? It seems to be duplicating the additional test you 
> added to GetAlgName.java

It is necessary, `GetAlgName.java` is about `EncryptedPrivateKeyInfo` accepting 
uninitialized AP, but this test is about `AlgorithmId` NOT accepting one.

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

PR: https://git.openjdk.org/jdk/pull/11067

Reply via email to