On Wed, 19 May 2021 18:44:06 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Similar as the one above, it's because of
>>
>> static {
>> // Don't lazy-read because every app uses invalidate()
>> isJavaAwtSmartInvalidate = AccessController.doPrivileged(
>> new GetBooleanAction("java.awt.smartInvalidate"));
>> }
>
> We are thinking of more tweaks after this overall change to make the
> annotation more precise. For example, in this case we can assign the
> `doPrivileged` result to a local variable right at its declaration, and then
> assign it to `isJavaAwtSmartInvalidate`. Some people might think this is
> ugly. Such manual code changes need to done little by little to ease code
> reviewing.
I know it's not easy to read the commit and that's why I make the 3rd commit
totally automatic. Hopefully you have more confidence on the program than my
hand. All annotations are added to the nearest declarations.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4073
