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

Reply via email to