On Thu, 27 Apr 2023 15:23:33 GMT, Andy Goryachev <[email protected]> wrote:
>> John Hendrikx has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix NaN returns when using Math.ulp
>
> modules/javafx.graphics/src/main/java/javafx/scene/layout/Region.java line
> 310:
>
>> 308: private static double scaledFloor(double value, double scale) {
>> 309: double d = value * scale;
>> 310:
>
> could we get rid of this newline - here and on lines 330, 413 please?
It's the style I use, I tend to separate assignment, declaration and other code.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1118#discussion_r1184800203
