On Thu, 20 Aug 2026 13:43:47 GMT, Marius Hanl <[email protected]> wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> try with
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/util/ImageUtils.java
> line 163:
>
>> 161: }
>> 162: switch (fxFormat.getType()) {
>> 163: default:
>
> Minor, also if you like: The `default` branch is the first. Looks very weird,
> should be the last instead
I thought about reformatting the original file, and decided against it, to
speed up the review.
On the other hand, I like cleaning up code...
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2267#discussion_r3822476515
