<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <br>

    <br>

    <div class="moz-cite-prefix">On 4/30/18 8:25 PM, Claes Redestad

      wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:85f31369-ba35-5d60-0d8c-cca0a3fd9598@oracle.com">Hi,

      <br>

      <br>

      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.

      <br>

      <br>

      Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~redestad/8202419/open.00/">http://cr.openjdk.java.net/~redestad/8202419/open.00/</a>

      <br>

    </blockquote>

    <br>

    Looks okay to me.<br>

    <br>

    I agree with Alan and Sean w.r.t. to the comment in

    ACCESS_PERMISSION that would be better to keep it in

    AccessibleObject::checkPermission and remove line 131 in

    ReflectionFactory.<br>

    <br>

    Mandy<br>

  </body>

</html>