RFR: JDK-8178278 Move Standard Algorithm Names document to specs directory

Brad R. Wetmore bradford.wetmore at oracle.com
Tue May 9 20:11:46 UTC 2017


>> Can you add similar wording to that effect in DrbgParameters.java?
>
> I'm sorry, that instruction is too unclear for me. :-( The main purpose
> of this patch is to prepare for the markdown based specification, not
> fixing the entire content. I'll leave modification of the actual
> documentation to the component owners.

In rereading this, I withdraw this comment.  It didn't make sense.

>> jdk/src/java.base/share/classes/java/security/cert/CertPathValidator.java
>>
>> 155-156, 193-194:  extra spaces at beginning of line.
> Fixed.
>
>>
>> jdk/src/java.base/share/classes/javax/crypto/Cipher.java
>>
>> 153:  Why did you choose to use "Cipher Algorithm Names" instead of
>> Cipher Algorithms?  None of the other sections use "Names".
>
> I did not choose that, that's what the section is called in the document
> (and has been since at least JDK 8). The difference here is that the
> anchor links are named after the actual headings in the document, not
> arbitrarily named anchors (for better and for worse).

When I got down into the .md file, I figured out your methodology. 
Thanks for explaining.

>> Can you add a link in the JSSE KeyManagerFactory class description to
>> the standard names sections for KeyManagerFactory's?  Even though we
>> don't require any currently, we should probably point to the names
>> anyway.
>
> I'll give you the same reply here as for DrbgParameters.java: I don't
> know enough about the domain to be able to do this job properly. Please
> file a follow-up bug and fix it yourself, or assign to someone who knows.

I've filed:  JDK-8180005

Thanks.

> Updated webrev: (contains whitespace fixes and rename of document, but
> only the jdk repo)

Whitespace looks fine.  I did a spot check of the renames and seemed 
pretty straightforward, so I didn't go through all of them.

Brad


> http://cr.openjdk.java.net/~ihse/JDK-8178278-standard-names-spec-as-markdown/webrev.02
>
>
> /Magnus
>
>
> /Magnus
>
>
>>
>> Thanks,
>>
>> Brad
>>
>>
>>
>>
>>
>> On 5/5/2017 6:17 AM, Magnus Ihse Bursie wrote:
>>> The Security Standard Names document will be moved to a new location, so
>>> all links needs to be updated. Also, a minor fix to the build system was
>>> needed.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8178278
>>> WebRev:
>>> http://cr.openjdk.java.net/~ihse/JDK-8178278-standard-names-spec-as-markdown/webrev.01
>>>
>>>
>>> /Magnus
>>>
>



More information about the security-dev mailing list