Looks fine to me. I think the TO DO comment on line 131 of ReflectionFactory can be removed - it looks like a leftover comment which doesn't seem relevant anymore.

--Sean

On 4/30/18 8:25 AM, Claes Redestad wrote:
Hi,

moving a couple of local permission constants to sun.security.util.SecurityConstants ensures we delay and avoid loading permission classes very early during bootstrap. Delaying load is profitable since it's happening early enough (before System.initPhase1) to contribute to delaying the initialization of VM subsystems.

Webrev: http://cr.openjdk.java.net/~redestad/8202419/open.00/

Thanks!

/Claes

Reply via email to