On Wed, 7 Oct 2026 15:20:05 GMT, Marius Hanl <[email protected]> wrote:

>> modules/javafx.graphics/src/main/java/com/sun/javafx/tk/quantum/QuantumToolkit.java
>>  line 163:
>> 
>>> 161: 
>>> 162:     static final boolean liveResize = ((Supplier<Boolean>) () -> {
>>> 163:         String result = (PlatformUtil.isMac() || 
>>> PlatformUtil.isWindows()) && !GlassPlatform.isSWT() ? "true" : "false";
>> 
>> I think we should get rid of all these `Supplier`s since we removed the 
>> `AccessController`.
>
> yes, same thought. Shall I do in this PR? Otherwise I will create a ticket 
> and clean it up afterwards.

The usual argument for unrelated changes is to make them in a separate PR for 
the sake or reviewers and backporters, but if we are touching this code we 
might as well clean it up.

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/2299#discussion_r4208738918

Reply via email to