<div dir="ltr"><div>Hi Chen,</div><div><br></div><div>Thanks for responding. I understand, and there's absolutely no hurry on these. I'm just trying to keep them from falling into the "lost forever" bucket.</div><div><br></div><div>I'm targeting smaller, lower priority fixes, minor improvements, and cleanups that other people are too busy to deal with ("code janitor"), so it's understood that these PR's are going to be inherently less exciting and lower priority in general. That also means sending out occasional (and hopefully non-annoying) pings for review, which I don't mind doing, but let me know if the ping rate needs to be adjusted :)</div><div><br></div><div>-Archie</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, Aug 12, 2025 at 10:13 AM Chen Liang <<a href="mailto:chen.l.liang@oracle.com">chen.l.liang@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 class="msg5629643087155689989">




<div dir="ltr">
<div style="font-family:"Calibri Light","Helvetica Light",sans-serif;font-size:12pt;color:rgb(0,0,0)">
Hi Archie, FYI people are coming back from JVMLS so it may take some time. In addition, most of our new development are focused on experimental projects like Babylon, Valhalla, and Amber, and javac is mainly just routine bugfixes, so these did not catch people's
 attention. So I think 8359383 is the most likely to be prioritized being a fix, while 8353487 is less because it involves API additions that has not seen a usage request.</div>
<div style="font-family:"Calibri Light","Helvetica Light",sans-serif;font-size:12pt;color:rgb(0,0,0)">
<br>
</div>
<div style="font-family:"Calibri Light","Helvetica Light",sans-serif;font-size:12pt;color:rgb(0,0,0)">
Thanks, Chen</div>
<div id="m_5629643087155689989appendonsend"></div>
<hr style="display:inline-block;width:98%">
<div id="m_5629643087155689989divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> compiler-dev <<a href="mailto:compiler-dev-retn@openjdk.org" target="_blank">compiler-dev-retn@openjdk.org</a>> on behalf of Archie Cobbs <<a href="mailto:archie.cobbs@gmail.com" target="_blank">archie.cobbs@gmail.com</a>><br>
<b>Sent:</b> Monday, August 11, 2025 11:57 AM<br>
<b>To:</b> compiler-dev <<a href="mailto:compiler-dev@openjdk.org" target="_blank">compiler-dev@openjdk.org</a>><br>
<b>Subject:</b> Compiler PR review requests</font>
<div> </div>
</div>
<div>
<div dir="ltr">
<div dir="ltr">
<div dir="ltr">
<div dir="ltr">
<div>Dear compiler-dev reviewers,</div>
<div><br>
</div>
<div>I have a few small PR's and one big PR that need reviewers and would appreciate your help in getting these resolved.</div>
<div><br>
</div>
<div>The big one...</div>
<div><br>
</div>
<div><a href="https://github.com/openjdk/jdk/pull/24584" target="_blank">JDK-8348611: Eliminate DeferredLintHandler and emit warnings after attribution</a></div>
<div><br>
</div>
<div style="margin-left:40px">This is a refactoring/cleanup that simplifies how warnings are logged, so that regardless of compiler phase, lint configuration, etc., you can always just invoke
<code>log.warning()</code> to log a warning and the right thing happens automatically.</div>
<div style="margin-left:40px"><br>
</div>
<div style="margin-left:40px">Already reviewed by Maurizio but needs a second reviewer.</div>
<div><br>
</div>
<div>The small ones...</div>
<div>
<div><br>
</div>
<div><a href="https://github.com/openjdk/jdk/pull/23622" target="_blank">JDK-8349847: Support configuring individual lint categories as errors</a></div>
<div><br>
</div>
<div style="margin-left:40px">Adds the capability for more precise control of which warnings cause an error. For example,<span style="font-family:monospace"> -Werror:all,-serial<font face="arial,sans-serif">.</font></span></div>
<div style="margin-left:40px"><br>
</div>
<div style="margin-left:40px">This one has been sitting out there with a CSR since February. The code is fairly straightforward; I think the main question is whether this is something we want to add or not.</div>
<div style="margin-left:40px"><br>
</div>
<div style="margin-left:40px">Any strong opinions one way or the other?</div>
<br>
<div><a href="https://github.com/openjdk/jdk/pull/24398" target="_blank">JDK-8353487: JShell LocalExecutionControl should allow decorating the execution task</a></div>
<div><br>
</div>
<div style="margin-left:40px">This is a simple API addition, but there is unresolved discussion about the best way to augment the API.</div>
<div style="margin-left:40px"><br>
</div>
<div style="margin-left:40px">Need someone in a position of authority (Jan) to weigh in.</div>
<br>
</div>
<div><a href="https://bugs.openjdk.org/browse/JDK-8359383" target="_blank">JDK-8359383</a><a href="https://github.com/openjdk/jdk/pull/25786" target="_blank">: Incorrect starting positions for implicitly typed variables</a></div>
<div><br>
</div>
<div style="margin-left:40px">This fixes a few minor and obscure glitches in computed lexical starting positions. Not the most important problem in the world right now, but it would be nice to fix these corner cases.</div>
<br>
<div>Thanks,</div>
<div>-Archie</div>
<div><br>
</div>
<span>-- </span><br>
<div dir="ltr">Archie L. Cobbs<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>

</div></blockquote></div><div><br clear="all"></div><br><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Archie L. Cobbs<br></div>