> On Jun 15, 2016, at 1:27 AM, Wang Weijun <weijun.w...@oracle.com> wrote: > > All suggestions accepted. Webrev updated at > > http://cr.openjdk.java.net/~weijun/8130302/webrev.05 > >> 241 throw (InvalidParameterException) >> >> This cast should not be needed? >> > > } catch (UcryptoException ue) { > throw (InvalidParameterException) > new InvalidParameterException("Error using " + configArg). > initCause(ue.getCause()); > } > > initCause() returns Throwable but the method's signature throws > InvalidParameterException. >
Perhaps have a local variable for InvalidParameterException exception. Mandy