RFR: 8367530: The exhaustiveness errors could be improved [v13]

Maurizio Cimadamore mcimadamore at openjdk.org
Mon Jan 26 12:16:29 UTC 2026


On Mon, 26 Jan 2026 12:11:26 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Jan Lahoda has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Attempting to use the number of 'base checks' as the limit, instead of wall-clock time.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/ExhaustivenessComputer.java line 691:
> 
>> 689:     }
>> 690: 
>> 691:     protected void checkTimeout() {
> 
> This is probably no longer a good name for this method.
> 
> More seriously, I have a question as to why this method is only checked (AFAICS) from `isBpCovered`. I would have expected it to be more directly tied to `computeMissingPatternDescriptions` ?

Maybe the intent is that you don't want to just check the "depth" of the check, but also the "width" -- e.g. if there's few nested record patterns, but each has a huge number of components.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27256#discussion_r2727375413


More information about the build-dev mailing list