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