<div dir="ltr">Done: <a href="http://cr.openjdk.java.net/~cushon/8171132/webrev.03/">http://cr.openjdk.java.net/~cushon/8171132/webrev.03/</a></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 15, 2016 at 5:29 AM, Maurizio Cimadamore <span dir="ltr"><<a href="mailto:maurizio.cimadamore@oracle.com" target="_blank">maurizio.cimadamore@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <p>Looks good - the only minor issue is that the new resource keys
      should probably be excluded in the examples-not-yet.txt file
      (otherwise diags test will fail). We typically try and provide an
      example of all diagnostics generated by javac. IN case of errors
      that are hard to reproduce (and classfile errors related to bad
      classfiles definitively fall in this category), we typically just
      omit the diagnostic key from the test (albeit it will be nice one
      day to have an example for _all_ diagnostics).</p><span class="HOEnZb"><font color="#888888">
    <p>Maurizio<br>
    </p></font></span><div><div class="h5">
    <br>
    <div class="m_-68537106510632773moz-cite-prefix">On 15/12/16 01:16, Liam Miller-Cushon
      wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">
        <div class="gmail_extra">
          <div class="gmail_quote">On Wed, Dec 14, 2016 at 12:31 PM,
            Alex Buckley <span dir="ltr"><<a href="mailto:alex.buckley@oracle.com" target="_blank">alex.buckley@oracle.com</a>></span>
            wrote:<br>
            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
              <div id="m_-68537106510632773m_-2858592650416056683gmail-:8tg" class="m_-68537106510632773m_-2858592650416056683gmail-a3s m_-68537106510632773m_-2858592650416056683gmail-aXjCH m_-68537106510632773m_-2858592650416056683gmail-m158ff08101df9bbb">I'm not
                sure which spec you're asking about ... the JLS, for
                example, doesn't mention reading class files so it can't
                be that. I mentioned "quality-of-implementation detail"
                to highlight that javac can react to the ConstantValue
                attribute for Lib.B (or Test.TWO I guess) how ever it
                wishes. javac could convert the value <span class="m_-68537106510632773m_-2858592650416056683gmail-aBn"><span class="m_-68537106510632773m_-2858592650416056683gmail-aQJ">on Monday</span></span>,
                zero it <span class="m_-68537106510632773m_-2858592650416056683gmail-aBn"><span class="m_-68537106510632773m_-2858592650416056683gmail-aQJ">on Tuesday</span></span>,
                and throw error every other day.</div>
            </blockquote>
            <div><br>
            </div>
            <div>Thanks! I was asking about the JVMS, which I understand
              doesn't require a class file to be rejected for this
              reason. I now realize it doesn't disallow that.</div>
            <div><br>
            </div>
            <div>I updated the change to report an error on out-of-range
              values:</div>
            <div><a href="http://cr.openjdk.java.net/%7Ecushon/8171132/webrev.02/" target="_blank">http://cr.openjdk.java.net/~cu<wbr>shon/8171132/webrev.02/</a></div>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
  </div></div></div>

</blockquote></div><br></div>