On Fri, 4 Nov 2022 08:44:34 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

>> This change adds constructors (String,Throwable) and (Throwable) to 
>> InvalidParameterException and uses them at a few places in the jdk coding.
>
> Matthias Baesken has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Small adjustments to javadoc, some exceptions and add test coding

LGTM, too.

test/jdk/java/security/Exceptions/ChainingConstructors.java line 189:

> 187: 
> 188:         InvalidParameterException ipe =
> 189:                 new InvalidParameterException(cause);

Indentation looks unusual, but it's consistent with some other parts. No need 
to change!

-------------

Marked as reviewed by mdoerr (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10966

Reply via email to