Compiler PR review requests
Archie Cobbs
archie.cobbs at gmail.com
Mon Aug 11 18:57:33 UTC 2025
Dear compiler-dev reviewers,
I have a few small PR's and one big PR that need reviewers and would
appreciate your help in getting these resolved.
The big one...
JDK-8348611: Eliminate DeferredLintHandler and emit warnings after
attribution <https://github.com/openjdk/jdk/pull/24584>
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 log.warning() to log a warning and the right thing happens
automatically.
Already reviewed by Maurizio but needs a second reviewer.
The small ones...
JDK-8349847: Support configuring individual lint categories as errors
<https://github.com/openjdk/jdk/pull/23622>
Adds the capability for more precise control of which warnings cause an
error. For example, -Werror:all,-serial.
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.
Any strong opinions one way or the other?
JDK-8353487: JShell LocalExecutionControl should allow decorating the
execution task <https://github.com/openjdk/jdk/pull/24398>
This is a simple API addition, but there is unresolved discussion about the
best way to augment the API.
Need someone in a position of authority (Jan) to weigh in.
JDK-8359383 <https://bugs.openjdk.org/browse/JDK-8359383>: Incorrect
starting positions for implicitly typed variables
<https://github.com/openjdk/jdk/pull/25786>
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.
Thanks,
-Archie
--
Archie L. Cobbs
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/compiler-dev/attachments/20250811/d0eab036/attachment.htm>
More information about the compiler-dev
mailing list