RFR: 8265372: Simplify PKCS9Attribute [v2]
Valerie Peng
valeriep at openjdk.org
Wed Jan 10 02:21:24 UTC 2024
On Thu, 4 Jan 2024 20:44:01 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> Ben Perez has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Minor fixes to make the code more readable, inlined init(), removed PKCS9Attributes.getAttributes()
>
> src/java.base/share/classes/sun/security/pkcs/PKCS9Attribute.java line 218:
>
>> 216: private static void add(ObjectIdentifier oid, boolean singleValued,
>> 217: Class<?> valueClass, byte... valueTags) {
>> 218: AttributeInfo info = new AttributeInfo(valueTags,valueClass,singleValued);
>
> Nit: add a space after ','.
nit: match the ordering of arguments here w/ the one used in record?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17132#discussion_r1446811223
More information about the security-dev
mailing list