<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 01/15/2013 02:38 PM, Vitaly
      Davidovich wrote:<br>
    </div>
    <blockquote
cite="mid:CAHjP37E9O_4BtUqNvnuR_ecq_dhJ4_4smcWMydiT3Zys2UUOwA@mail.gmail.com"
      type="cite">
      <p dir="ltr">Looks good Stefan. <br>
      </p>
    </blockquote>
    <br>
    Thanks, Vitaly.<br>
    <br>
    <blockquote
cite="mid:CAHjP37E9O_4BtUqNvnuR_ecq_dhJ4_4smcWMydiT3Zys2UUOwA@mail.gmail.com"
      type="cite">
      <p dir="ltr"> The annotation creation code in classFileParser is
        really asking for a macro or helper function that does the
        checks and construction.</p>
    </blockquote>
    <br>
    I agree. Does anyone in the runtime team have any opinions on this?<br>
    <br>
    thanks,<br>
    StefanK<br>
    <br>
    <blockquote
cite="mid:CAHjP37E9O_4BtUqNvnuR_ecq_dhJ4_4smcWMydiT3Zys2UUOwA@mail.gmail.com"
      type="cite">
      <p dir="ltr">Thanks</p>
      <p dir="ltr">Sent from my phone</p>
      <div class="gmail_quote">On Jan 15, 2013 7:27 AM, "Stefan
        Karlsson" &lt;<a moz-do-not-send="true"
          href="mailto:stefan.karlsson@oracle.com">stefan.karlsson@oracle.com</a>&gt;
        wrote:<br type="attribution">
        <blockquote class="gmail_quote" style="margin:0 0 0
          .8ex;border-left:1px #ccc solid;padding-left:1ex">
          <a moz-do-not-send="true"
            href="http://cr.openjdk.java.net/%7Estefank/8005994/webrev.00/"
            target="_blank">http://cr.openjdk.java.net/~stefank/8005994/webrev.00/</a><br>
          <br>
          HotSpot unnecessarily allocates annotations arrays when they
          are not needed. This memory regression was introduced when the
          annotation data structures were changed in the Permgen Removal
          project.<br>
          <br>
          This fix:<br>
          1) reintroduces the old null checks<br>
          2) adds a null check before creating the newly introduced
          method type annotations.<br>
          3) adds a missing null check where the type annotations are
          used, since they now can be null.<br>
          <br>
          Testing:<br>
          jprt, jdk_lang and type annotations tests<br>
          <br>
          thanks,<br>
          StefanK<br>
        </blockquote>
      </div>
    </blockquote>
    <br>
  </body>
</html>