RFR: 8344248: Remove Security Manager dependencies from java.security.jgss and jdk.security.jgss modules [v2]

[Sean Mullan]

On Thu, 21 Nov 2024 06:49:10 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix minor spacing issues.
>
> src/java.security.jgss/share/classes/sun/security/jgss/wrapper/SunNativeProvider.java line 115:
> 
>> 113:                  }
>> 114:                  Oid[] mechs = GSSLibStub.indicateMechs();
>> 115:                  HashMap<String,String> map = new HashMap<>();
> 
> Suggestion:
> 
>                  HashMap<String, String> map = new HashMap<>();

Fixed.

> src/java.security.jgss/share/classes/sun/security/krb5/KdcComm.java line 349:
> 
>> 347:                 DEBUG.println(">>> KDCCommunication: kdc=" + kdc
>> 348:                         + " " + proto + ":"
>> 349:                         +  port +  ", timeout="
> 
> Suggestion:
> 
>                         + port + ", timeout="

Fixed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22256#discussion_r1852176744
PR Review Comment: https://git.openjdk.org/jdk/pull/22256#discussion_r1852176203