On 13/04/16 16:59, Alan Bateman wrote:
On 13/04/2016 16:43, Chris Hegarty wrote:
This review is for the second significant part of the changes for JEP 260 [1]. When created, the jdk.unsupported module [2] initially contained the sun.misc package. This issue is will move all the non-Critical APIs out of sun.reflect, leaving only the critical ones, at which point sun.reflect can be moved to the jdk.unsupported module.
http://cr.openjdk.java.net/~chegar/8137058/ https://bugs.openjdk.java.net/browse/JDK-8137058
This looks good. A few comments:
I assume the new sun.reflect.Reflection should have a private constructor to prevent it being instantiated.
D'oh! of course. Updated in-place. http://cr.openjdk.java.net/~chegar/8137058/jdk/src/jdk.unsupported/share/cla...
You've probably thought about this already but I assume the @Deprecated in Reflection.getCallerClass should reference the supported API.
If it is ok, I'd like to address that separately. I think there is probably more we can do there, and it probably deserves its own discussion.
The changes to InetAddress don't seem unrelated but good.
They were sitting in my repo. Just some minor reformatting. -Chris.