RFR: 8342914: Remove calls to doPrivileged in swt [v2]

Kevin Rushforth kcr at openjdk.org
Wed Oct 30 18:57:16 UTC 2024


On Wed, 30 Oct 2024 18:08:45 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> modules/javafx.swt/src/main/java/javafx/embed/swt/FXCanvas.java line 359:
>> 
>>> 357:             });
>>> 358:             return null;
>>> 359:         }, null, FXCANVAS_PERMISSION);
>> 
>> I think `FXCANVAS_PERMISSION` is unused after this and can be removed (along with the related import).
>
> there is a duplicate FXCANVAS_PERMISSION in PlatformImpl which should also be removed.

I think that one could have waited, since the code in question would never be executed, but it does seem cleaner to remove it now.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1611#discussion_r1823199171


More information about the openjfx-dev mailing list