On 2017-01-16 22:39, Mandy Chung wrote:
On Jan 12, 2017, at 6:48 AM, Claes Redestad <claes.redes...@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