On Thu, 1 Sep 2022 02:55:21 GMT, Weijun Wang <[email protected]> wrote:
>> Mark Powers has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> comments from Sean and Max
>
> src/java.base/share/classes/sun/security/provider/certpath/BasicChecker.java
> line 105:
>
>> 103: prevPubKey = trustedPubKey;
>> 104: if (PKIX.isDSAPublicKeyWithoutParams(prevPubKey)) {
>> 105: // If TrustAnchor is a DSA public key, and it has no
>> params, it
>
> Not a native English speaker, but I found the sentence more fluent without
> the comma.
Removing the comma.
> src/java.base/share/classes/sun/security/provider/certpath/BuildStep.java
> line 192:
>
>> 190: String resultString;
>> 191: switch (res) {
>> 192: case POSSIBLE:
>
> `case` should be indented here. See
> http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html#toc-indentation.
>
> That said, I think this is a good chance to use the switch expressions
> syntax, i.e `return switch (res)...`.
Brad pointed me to this:
https://www.oracle.com/technetwork/java/codeconventions-150003.pdf
I think your reference is better since we're working on OpenJDK.
-------------
PR: https://git.openjdk.org/jdk/pull/9972
