<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 26, 2023 at 12:00 PM Brian Goetz <<a href="mailto:brian.goetz@oracle.com">brian.goetz@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

  
  <div>
    <font size="4"><font face="monospace">Is there any mechanical
        impediment to pushing to an experimental branch (e.g., committer
        status, repo config), </font></font></div></blockquote><div><br></div><div>I don't have access / permission to create experimental branches on the leyden repo.  So yes to mechanical impediment.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><font size="4"><font face="monospace">or are you just asking about how to
        coordinate work in progress?  If the latter, creating
        experimental branches is fair game, and clear naming and
        communication is really the only requirement.<br></font></font></div></blockquote><div><br></div><div>And even if I had access, I wanted to confirm experimental branches were OK.  Not looking to step out of bounds here =)</div><div><br></div><div>--Dan</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><font size="4"><font face="monospace">
        <br>
        <br>
      </font></font><br>
    <div>On 6/26/2023 11:18 AM, Dan Heidinga
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr">I've updated my jlink patch to pregenerate lambda
        classes [0] so it works with the current Leyden repo and the
        updated classfile api (which is awesome - check it out if you
        haven't yet!).  Previously discussed in the "Bytecode
        transformation investigation" thread [1].
        <div><br>
        </div>
        <div>I'd like to push it to an experimental branch here on the
          leyden repo so that others can freely pick up and experiment
          with the code.  Would that be possible?</div>
        <div><br>
        </div>
        <div>My local branch is called "pregenerate-lambdas" but any
          name would work for me.</div>
        <div><br>
        </div>
        <div>The patch pre-generates during jlink time classes for
          non-Serializable lambda expressions and uses NestMates to
          handle access to the private implementation method.  It's a
          good example of the kinds of changes needed to shift
          runtime-generated classes to jlink time.</div>
        <div><br>
        </div>
        <div>Still lots to update but worth getting into Leyden repo so
          the code is shareable.</div>
        <div><br>
        </div>
        <div>Mark / Brian - is it possible to get an experimental branch
          on the leyden repo for this patch?</div>
        <div><br>
        </div>
        <div>--Dan<br>
          <div><br>
          </div>
          <div>
            <div>[0] <a href="https://github.com/openjdk/leyden/compare/master...DanHeidinga:leyden:pregenerate-lambdas?expand=1" target="_blank">https://github.com/openjdk/leyden/compare/master...DanHeidinga:leyden:pregenerate-lambdas?expand=1</a><br>
            </div>
          </div>
        </div>
        <div><br>
        </div>
        <div>[1] <a href="https://mail.openjdk.org/pipermail/leyden-dev/2022-August/000055.html" target="_blank">https://mail.openjdk.org/pipermail/leyden-dev/2022-August/000055.html</a></div>
      </div>
    </blockquote>
    <br>
  </div>

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