RFR: 8015831: Add lint check for calling overridable methods from a constructor [v10]

Archie L. Cobbs duke at openjdk.org
Tue Jan 17 17:58:42 UTC 2023


On Mon, 16 Jan 2023 13:30:57 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> Archie L. Cobbs has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Fix bug where field initializer warnings could be incorrectly suppressed.
>>  - Consolidate all the unit tests that generate warnings into one.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/ThisEscapeAnalyzer.java line 298:
> 
>> 296:         }.scan(env.tree);
>> 297: 
>> 298:         // TODO: eliminate sealed classes where all permitted subclasses are in this compilation unit
> 
> +1 on better check for permitted (shouldn't be too difficult, or am I mistaken?)

Yes it's easy & I was just procrastinating. Will fix.

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

PR: https://git.openjdk.org/jdk/pull/11874



More information about the build-dev mailing list