On Mon, 20 Mar 2023 14:50:01 GMT, Jan Lahoda <[email protected]> wrote:
> `this-escape` lint is disabled for jdk.compiler, so it should be disabled for
> the ant-based langtools-only build as well.
Just a note... the fix for JDK-8304443 adds `@SuppressWarnings("this-escape")`
to the compiler code, and that should also fix this bug as a side effect. This
PR, which disables the `this-escape` lint warning, works too, of course.
So, just in case you change your mind and choose to NOT disable the
`this-escape` lint warning, then for consistency's sake we should also
re-enable it for the regular build in `modules/jdk.compiler/Gendata.gmk` and
`modules/jdk.compiler/Java.gmk`.
My apologies for using two different approaches to suppressing the warning at
different times.
-------------
PR: https://git.openjdk.org/jdk/pull/13102
