On Thu, 19 Jun 2025 22:37:11 GMT, Michael Strauß <[email protected]> wrote:
>> Johan Vos has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix missing ;
>
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/headless/HeadlessApplication.java
> line 23:
>
>> 21: private final HeadlessWindowManager windowManager = new
>> HeadlessWindowManager();
>> 22: private Screen[] screens = null;
>> 23: private HeadlessCursor cursor;
>
> This field is not used in this class, other than assigning a value.
removed and changed createCursor method
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/headless/HeadlessWindow.java
> line 268:
>
>> 266: }
>> 267:
>> 268: boolean setFullscreen(boolean full) {
>
> None of the two callers use the return value of this method.
changed return type to void
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1836#discussion_r2189366361
PR Review Comment: https://git.openjdk.org/jfx/pull/1836#discussion_r2189364588