RFR: 8194743: Compiler implementation for Statements before super() [v16]
Archie Cobbs
acobbs at openjdk.org
Mon Sep 25 19:25:19 UTC 2023
On Mon, 25 Sep 2023 17:49:52 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Archie Cobbs has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Have RefBeforeCtorCalledError extend StaticError and customize debug strings.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties line 245:
>
>> 243:
>> 244: # 0: name
>> 245: compiler.err.calls.not.allowed.here=\
>
> The compiler message key seems overly general. maybe `constructor.call.not.allowed.here` ?
Agreed - will fix.
> src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties line 250:
>
>> 248: # 0: name
>> 249: compiler.err.return.before.superclass.initialized=\
>> 250: ''return'' not allowed prior to {0}()
>
> We have no other match in this file for `prior to`, but we do have matches for `before`. I suggest using the latter for a more consistent style.
Agreed - will fix.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13656#discussion_r1336299193
PR Review Comment: https://git.openjdk.org/jdk/pull/13656#discussion_r1336299902
More information about the compiler-dev
mailing list