RFR: 8290775: Some doc errors in DerOutputStream.java [v2]
Weijun Wang
weijun at openjdk.org
Fri Jul 22 12:47:05 UTC 2022
On Fri, 22 Jul 2022 08:27:38 GMT, jquanC <duke at openjdk.org> wrote:
>> There are some doc errors in sun.security.util.DerOutputStream, like the followings,
>>
>>
>> /**
>> * Private helper routine for writing DER encoded string values.
>> * @param s the string to write
>> * @param stringTag one of the DER string tags that indicate which
>> * encoding should be used to write the string out.
>> * @param enc the name of the encoder that should be used corresponding
>> * to the above tag.
>> */
>> private void writeString(String s, byte stringTag, Charset charset) throws IOException
>>
>> The parameter is charset, but not enc.
>>
>>
>> /**
>> * Marshals a DER integer on the output stream.
>> *
>> * @param i the integer in bytes, equivalent to BigInteger::toByteArray.
>> */
>> public void putInteger(byte[] buf) throws IOException {
>>
>> The parameter is buf, but not i.
>
> jquanC has updated the pull request incrementally with one additional commit since the last revision:
>
> 8290775: Some doc errors in DerOutputStream.java Update: better improvement
`s/is should used/should be used/`.
-------------
PR: https://git.openjdk.org/jdk/pull/9585
More information about the security-dev
mailing list