<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: "Calibri Light", "Helvetica Light", sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
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);" class="elementToProof">
<br>
</div>
<div style="font-family: "Calibri Light", "Helvetica Light", sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
Thanks, Chen</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> compiler-dev <compiler-dev-retn@openjdk.org> on behalf of Archie Cobbs <archie.cobbs@gmail.com><br>
<b>Sent:</b> Monday, August 11, 2025 11:57 AM<br>
<b>To:</b> compiler-dev <compiler-dev@openjdk.org><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 class="x_gmail_signature_prefix">-- </span><br>
<div dir="ltr" class="x_gmail_signature" data-smartmail="gmail_signature">Archie L. Cobbs<br>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>