RFR: JDK-8307563: make most fields final in `JavacTrees`
Jonathan Gibbons
jjg at openjdk.org
Wed May 10 22:06:44 UTC 2023
On Fri, 5 May 2023 22:10:19 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Please review a small cleanup fix to address a comment left a long time back.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTrees.java line 221:
>
>> 219: fileManager = context.get(JavaFileManager.class);
>> 220: var task = context.get(JavacTask.class);
>> 221: javacTaskImpl = (task instanceof JavacTaskImpl taskImpl) ? taskImpl : null;
>
> Suggestion:
>
> javacTaskImpl = context.get(JavacTask.class) instanceof JavacTaskImpl taskImpl
> ? taskImpl : null;
I don't see that as a significant improvement.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13847#discussion_r1190433247
More information about the compiler-dev
mailing list