On Fri, 11 Sep 2026 11:23:30 GMT, Marius Hanl <[email protected]> wrote:
>> Michael Strauß has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> review comments
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/css/ParsedValueImpl.java
> line 736:
>
>> 734: case 0 -> INT;
>> 735: case 1 -> DOUBLE;
>> 736: default -> throw new IOException("Unknown number type");
>
> Should this also be rather an `InternalError` ? IO makes more sense here but
> still asking.
I think it should be an IOException.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2303#discussion_r3989711872
