RFR: 8284415: Collapse identical catch branches in security libs
Mike StJohns
mstjohns at comcast.net
Wed Apr 6 17:43:03 UTC 2022
Before you approve this, why would you rethrow just the message rather than the actual original cause? Seems like you’re losing debug info including the type of the original exception for no good reason.
Mike
Sent from my iPad
> On Apr 6, 2022, at 11:26, Xue-Lei Andrew Fan <xuelei at openjdk.java.net> wrote:
>
> 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`
>
> Looks good to me.
>
> -------------
>
> Marked as reviewed by xuelei (Reviewer).
>
> PR: https://git.openjdk.java.net/jdk/pull/8068
More information about the security-dev
mailing list