RFR: 8334248: Invalid error for early construction local class constructor method reference [v2]

Maurizio Cimadamore mcimadamore at openjdk.org
Fri Jun 28 17:45:48 UTC 2024


On Thu, 27 Jun 2024 12:42:44 GMT, Vicente Romero <vromero at openjdk.org> wrote:

>> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix test summary
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java line 3772:
> 
>> 3770:         Env<AttrContext> env1 = env;
>> 3771:         boolean staticOnly = false;
>> 3772:         Symbol bestSoFar = varNotFound;
> 
> nit: it seems like this variable either holds this value, varNotFound, or an error but never the symbol we are looking for, so probably the name should reflect this?

I've renamed this to `firstError`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19904#discussion_r1659110498


More information about the compiler-dev mailing list