<div dir="ltr"><div dir="ltr">On Thu, Jun 20, 2024 at 12:04 PM Maurizio Cimadamore <<a href="mailto:maurizio.cimadamore@oracle.com">maurizio.cimadamore@oracle.com</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>

  
  <div><blockquote type="cite"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
            <div>
              <div>
                <p style="margin:0px 0px 1.2em">I think that this fix,
                  coupled with the translation fixes I shared last week
                  should implement JEP 482 “as intended”</p>
              </div>
            </div>
          </blockquote>
          <div>Awesome, thanks for tackling this! Fyi, I've merged this
            patch into my <a href="https://urldefense.com/v3/__https://github.com/openjdk/jdk/compare/master...archiecobbs:jdk:javac-pre-capture-fixes?expand=1__;!!ACWV5N9M2RV99hQ!LNS_Z7ARnFzONoErnZklxoEZgNNqB6Z94YH73CC6lr9bNGs2ity2KjdM20yZjTgLM7aJTjO6MNMN0sLbdJaN4ojJJcfNUQ$" target="_blank">javac-pre-capture-fixes</a> branch
            (already includes your lambda refactoring).</div>
        </div>
      </div>
    </blockquote>
    <p>Let me know if that fixes all the issue we're seeing (it would be
      great if we could collect all test cases in some github repo, so
      that we can add to them easily).</p></div></blockquote></div><div class="gmail_quote">It fixes all the tests I've been running - these tests are already being collected in the aforementioned github branch.</div><div class="gmail_quote"><br></div><div class="gmail_quote">And I just added a couple more, so that now all of the "javac-pre-capture" issues are represented with tests, and they are all passing.</div><div class="gmail_quote"><br></div><div class="gmail_quote">So we're looking good! Let me know how you'd like to proceed and if I can help in any way.</div><div class="gmail_quote"><br></div><div class="gmail_quote">A few housekeeping notes about the current "javac-pre-capture" branch:</div><div class="gmail_quote"><ul><li>I fixed a typo in your "canRefThis" comment in Lower.java ("is" → "if")<br></li><li>The fix for JDK-8334252 (already committed) had the wrong bug # in the unit test (now fixed - does this need a separate bug?)</li><li>We may want a new error message for the JDK-8322882 error, which e.g. now reports "non-static method foo(Object) cannot be referenced from a static context" for the test case even though foo() is a static method.<br></li></ul></div><div class="gmail_quote">Thanks, <br></div><div class="gmail_quote">-Archie<br></div><div class="gmail_quote"><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Archie L. Cobbs<br></div></div>