<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="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">Archie L. Cobbs<br></div></div>
</div>
</div>
</div>