On Wed, 18 Oct 2023 17:22:31 GMT, Andrey Turbanov <[email protected]> wrote:
>> Weijun Wang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> inline some throw statements, remove duplicated whitespaces
>
> src/java.security.jgss/windows/classes/sun/security/krb5/internal/tools/Kinit.java
> line 131:
>
>> 129: } catch (Exception e) {
>> 130: String msg;
>> 131: if (e instanceof KrbException) {
>
> Let's simplify with pattern matching variable
+1. Rest looks good to me.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15951#discussion_r1370484614
