RFR: 8037325: Class.getConstructor() performance regression

[Claes Redestad]

On 2017-01-16 22:39, Mandy Chung wrote:
>
>> On Jan 12, 2017, at 6:48 AM, Claes Redestad <claes.redestad at oracle.com> wrote:
>>
>> Hi,
>>
>> please review this fix to various performance regressions observed
>> as the security model has evolved over the years.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8037325
>> Webrev: http://cr.openjdk.java.net/~redestad/8037325/webrev.01
>>
>
> Looks good.

Thanks!

>
> Nit: methodName method returns the string representation of the method signature and so more than the method name.  Maybe it should call “methodToString”?  The argumentTypesToString method is only used to print the method signature.  You could merge these two methods if you like.
>
> ReflectUtil.java
>  262     public static boolean isNonPublicProxyClass(Class<?> cls) {
>  263         String pkg;
>  264         return Proxy.isProxyClass(cls) &&
>  265                 ((pkg = cls.getPackageName()) == null || !pkg.startsWith(PROXY_PACKAGE));
>  266     }
>
> Nit: just a personal preference: move Proxy.isProxyClass(cls) check in a separate if-statement and the declaration pkg can be moved with the assignment.

http://cr.openjdk.java.net/~redestad/8037325/webrev.02/

Like that?

/Claes
>
> Mandy
>