On Fri, 1 Apr 2022 07:32:21 GMT, Andrey Turbanov <[email protected]> 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