On Thu, 18 May 2023 22:00:35 GMT, Kevin Driver <kdri...@openjdk.org> wrote:
>> src/java.base/share/classes/sun/security/ssl/CertificateRequest.java line >> 385: >> >>> 383: X509ExtendedKeyManager km = >>> chc.sslContext.getX509KeyManager(); >>> 384: String clientAlias = null; >>> 385: >> >> Many of these lines are > 80 chars, which makes side-by-side challenging. >> Please keep line changes to a max of 80 chars. >> >> Didn't check the other file, but think I saw similar things there. > > This might be something on your end @bradfordwetmore. I opened this in my IDE > and went to the end of the line... It was right under the "String > clientAlias" capital "S". <img width="174" alt="image" src="https://github.com/openjdk/jdk/assets/1783591/c549e612-5ec7-47ce-add5-828cf79c31d9"> ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13466#discussion_r1198348358