On Fri, 1 Apr 2022 07:32:21 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:
> Let's take advantage of Java 7 language feature - "Catching Multiple > Exception Types". > It simplifies code. Reduces duplication. > Found by IntelliJ IDEA inspection `Identical 'catch' branches in 'try' > statement` Other that the 1 copyright issue, LGTM. I didn't notice any expect behavioral changes between the new and old code. src/jdk.security.auth/share/classes/com/sun/security/auth/module/Krb5LoginModule.java line 788: > 786: > 787: } > 788: } catch (KrbException | IOException e) { Please update copyright date. ------------- Marked as reviewed by wetmore (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/8068