RFR: 8364149: Conditional stylesheet imports [v3]
Michael Strauß
mstrauss at openjdk.org
Thu Feb 26 00:59:00 UTC 2026
On Wed, 25 Feb 2026 20:41:24 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Michael Strauß has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>>
>> - Merge branch 'master' into feature/conditional-import
>> - Merge branch 'master' into feature/conditional-import
>> - Conditional stylesheet imports
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/css/media/MediaQueryParser.java line 90:
>
>> 88: public MediaQueryList parseMediaQueryList(List<Token> tokens) {
>> 89: var stream = new TokenStream(tokens);
>> 90: var expressions = new MediaQueryList();
>
> minor complaint: we should probably limit the usage of 'var' keywords for tests. these are easier to write, I admit, but make the code harder to read.
I never use `var` when the type isn't visible, but when it's just a `new` expression, I find them to be quite easy to read as it makes the variable name stand out.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2031#discussion_r2856257861
More information about the openjfx-dev
mailing list