On Fri, 5 May 2023 09:30:54 GMT, Jaikiran Pai <[email protected]> wrote:
>> Jim Laskey has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - Anonymous main classes renamed to unnamed classes
>> - Add test
>
> src/java.base/share/classes/jdk/internal/misc/MainMethodFinder.java line 142:
>
>> 140:
>> 141: if (mainMethod.getDeclaringClass() != mainClass) {
>> 142: System.err.println("WARNING: static main in super class
>> will be deprecated.");
>
> Similarly, this warning would have to be logged only if the method is
> `static`. Furthermore, do you think we should include the declaring class in
> the log message to provide some context on what's causing this warning?
> Something like:
>> WARNING: static main(String[]) in super class foo.bar.Parent will be
>> deprecated.
Yes this is an oversight on my part.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13689#discussion_r1186223626
