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.


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