Hi Claes,

This looks good. And can even be backported if desired. If/When Christoph's proposal to add Executable.getParameterType(index) gets accepted, all getParameterTypes() usages will be fixed at once, including this one...

Regards, Peter

On 10/20/2017 04:10 PM, Claes Redestad wrote:
Hi,

please review this patch, contributed by Christoph Dreis, to speed up AnnotationInvocationHandler.invoke, primarily by avoiding cloning the Method parameterTypes array unnecessarily.

Webrev: http://cr.openjdk.java.net/~redestad/8189266/jdk.00/

Bug: https://bugs.openjdk.java.net/browse/JDK-8189266

Thanks!

/Claes


Reply via email to