On Fri, 13 Jan 2023 15:14:05 GMT, Archie L. Cobbs <[email protected]> wrote:
>> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/ThisEscapeAnalyzer.java
>> line 516:
>>
>>> 514: Name name = TreeInfo.name(invoke.meth);
>>> 515: if (name == names._super) {
>>> 516: scanInitializers();
>>
>> it seems like the code scan initializers every time it finds a super()
>> invocation, I guess that this scanning could be done once per class
>
> Yes... I did it that way is so that it doesn't require any adaptation if/when
> JDK-8194743 ever gets implemented. And it keeps the code a little simpler in
> exchange for a little redundancy.
>
> I'm happy to fix this if you think it is necessary though.
I'm OK either way we can revisit this later either as part of this PR or in a
future one. I let it to your consideration
-------------
PR: https://git.openjdk.org/jdk/pull/11874
