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