RFR: 8313424: JavaFX controls in the title bar [v56]

Michael Strauß mstrauss at openjdk.org
Tue Mar 18 05:09:22 UTC 2025


On Mon, 17 Mar 2025 21:00:42 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   enable preview features in tests
>
> modules/javafx.base/src/main/java/com/sun/javafx/PreviewFeature.java line 79:
> 
>> 77:         if (enabledForTesting) {
>> 78:             // do nothing
>> 79:         } else if (!enabled) {
> 
> `if (!enabledForTesting && !enabled) {` ?

I would have to surround both branches (if and else), as otherwise the warning will be emitted during tests. That would, however, increase nesting…

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r2000212445


More information about the openjfx-dev mailing list