RFR: 8345348: CSS media feature queries [v23]
Michael Strauß
mstrauss at openjdk.org
Wed May 7 09:28:30 UTC 2025
On Tue, 6 May 2025 15:08:53 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>>
>> improve synchronization in PreferenceProperties
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/application/preferences/PreferenceProperties.java line 312:
>
>> 310:
>> 311: // This method must only be called when synchronized on 'mutex'.
>> 312: public void updateEffectiveValue() {
>
> maybe make this method private then: the instance of this class is available via `Platform.getPreferences()` if I read this correctly
No, the instance of `PreferenceProperties` only stored as a private field in `PlatformPreferences`. I'm usually using public methods in nested classes to indicate that the method is supposed to be called from outside of the nested class. That's not strictly necessary, but I think it serves to differentiate them from "purely" private methods.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1655#discussion_r2077215278
More information about the openjfx-dev
mailing list