RFR: 8284415: Collapse identical catch branches in security libs
Bradford Wetmore
wetmore at openjdk.java.net
Wed Apr 6 23:22:41 UTC 2022
On Fri, 1 Apr 2022 07:32:21 GMT, Andrey Turbanov <aturbanov at 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
More information about the security-dev
mailing list