On Thu, 4 Jun 2026 13:13:33 GMT, Maurizio Cimadamore <[email protected]> 
wrote:

> After some more analysis, we have decided to drop the initialization warning 
> from the initial JEP 401 integration.
> 
> First, the definition of initialization warnings is a little ambiguous, as it 
> can mean one of two things:
> * pretend super() was at the bottom of a constructor, and see what happens
> * pretend all fields were strict, and see what happens
> 
> These two things are not identical, and lead to a different sets of warnings 
> being generated.
> 
> Secondly, there's a lot of noise with these warnings:
> * JDK code contains ~67 suppressions
> * if we tweak the code to also report the warning when `--enable-preview` is 
> disabled, an additional 80+ warnings show up during an early phase of the 
> build (but there's possibly many many more)
> 
> For the time being, we'll revert the code that issues the new warnings, 
> related tests and suppressions.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

> please compare with the PR that introduced it: #1650, there are other files 
> that should be modified, for example: 
> src/jdk.compiler/share/classes/module-info.java 
> src/jdk.compiler/share/man/javac.md

Good catch! Addressed in 
https://github.com/openjdk/valhalla/pull/2512/commits/1c548ee5fe47c5a3cdc6ed82f6809dba68e953e0

-------------

PR Comment: https://git.openjdk.org/valhalla/pull/2512#issuecomment-4623301549

Reply via email to