On Tue, 22 Nov 2022 21:30:09 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> John Hendrikx has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Fix indentation
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/tk/quantum/WindowStage.java
>  line 418:
> 
>> 416:                     scaleFactor = 1.0 / scaleDivider;
>> 417:                     adjw = (int)Math.round(iw / scaleDivider);
>> 418:                     adjh = (int)Math.round(ih / scaleDivider);
> 
> Same comment here about the old code being clearer.

`scaleDivider` is defined just 2 lines above as a `double`. I don't see how the 
cast helps here.

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

PR: https://git.openjdk.org/jfx/pull/960

Reply via email to