RFR: 8378197: Remove AppContext from sun/swing/plaf/DesktopProperty.java [v2]
Phil Race
prr at openjdk.org
Sat Feb 21 07:19:01 UTC 2026
On Thu, 19 Feb 2026 23:08:15 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
>> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8378197
>
> src/java.desktop/share/classes/sun/swing/plaf/DesktopProperty.java line 98:
>
>> 96: */
>> 97: private static synchronized void setUpdatePending(boolean update) {
>> 98: updatePending = update;
>
> It seems the synchronized accessors are no longer needed? And the field itself marked as volatile can be used instead, since both of these methods are called from just the updateUI() method?
updated
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29801#discussion_r2835932183
More information about the client-libs-dev
mailing list