RFR: 8296072: CertAttrSet::encode and DerEncoder::derEncode should write into DerOutputStream [v3]
Sean Mullan
mullan at openjdk.org
Mon Oct 31 18:37:21 UTC 2022
On Mon, 31 Oct 2022 18:26:25 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> same for DerEncoder::derEncode
>>
>> only in patch2:
>> unchanged:
>
> src/java.base/share/classes/sun/security/pkcs10/PKCS10Attributes.java line 96:
>
>> 94: * @exception IOException on encoding errors.
>> 95: */
>> 96: public void encode(DerOutputStream out) throws IOException {
>
> Add `@Override`.
I think you are missing `@Override` on some of the other `encode` and `derEncode` methods.
-------------
PR: https://git.openjdk.org/jdk/pull/10906
More information about the security-dev
mailing list