RFR: 8296442: EncryptedPrivateKeyInfo can be created with an uninitialized AlgorithmParameters [v2]

[Weijun Wang]

On Thu, 10 Nov 2022 16:41:13 GMT, Kevin Driver <kdriver at openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   address xuelei's comments
>
> src/java.base/share/classes/javax/crypto/EncryptedPrivateKeyInfo.java line 187:
> 
>> 185:             throw new NullPointerException("algParams must be non-null");
>> 186:         }
>> 187:         AlgorithmId tmp;
> 
> style-comment: meaningful variable names should be preferred (ie- choose something other than `tmp`)

I don't usually use tmp as variable names, but when I do, it's indeed temporary and will be discarded within 8 lines (comment and empty lines not counted). :-D

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

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