RFR: 8323706: Remove SimpleSelector and CompoundSelector classes [v9]
John Hendrikx
jhendrikx at openjdk.org
Mon Sep 9 16:32:14 UTC 2024
On Tue, 3 Sep 2024 16:47:12 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Reject CSS where compound selectors don't consist of simple selectors
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/css/BinarySerializer.java line 109:
>
>> 107:
>> 108: if (type != TYPE_SIMPLE) {
>> 109: throw new IllegalStateException("Expected compound selector to consist of simple selectors only, but found type: " + type);
>
> minor: the output will be a signed decimal. would it be better if this can be formatted as hex?
>
> `String.format("Expected compound selector to consist of TYPE_SIMPLE only, but found type: 0x%02X", type)`
I don't think that would matter much. This error will only ever be interpreted by programmers, I'm sure they'll be able to understand what's going on.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1333#discussion_r1750569742
More information about the openjfx-dev
mailing list