Hi Kirill, (I'm not a reviewer)
Is there a specification of what shall be thrown, or the justification of the changes done? One can think that the currently thrown NoSuchMethodException is correct since　 methods, which differ in signature only, are still different methods. Best regards, Alexander ----- Original Message ----- From: [email protected] To: [email protected] Sent: Friday, May 6, 2016 8:11:08 PM GMT +03:00 Iraq Subject: RFR(XXS): 8156226: DiagnosticCommandImpl::invoke throws NoSuchMethodException even if the method actually exists but parameters are wrong Dear all, Could you please review this small fix for 8156226? A case when a method exists but parameters' signature is wrong now causes new ReflectionException(new IllegalArgumentException()) thrown instead of new ReflectionException(new NoSuchMethodException()). WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8156226/webrev.00/ CR: https://bugs.openjdk.java.net/browse/JDK-8156226 Thank you. Regards, Kirill
