RFR: 8313424: JavaFX controls in the title bar [v45]
Kevin Rushforth
kcr at openjdk.org
Tue Feb 4 01:03:22 UTC 2025
On Mon, 3 Feb 2025 23:43:47 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>>
>> macOS bugfixes, default button behavior
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/scene/layout/HeaderButtonBehavior.java line 88:
>
>> 86: stage.setMaximized(!stage.isMaximized());
>> 87: }
>> 88: });
>
> I think it might be better to add a default case here, _in case this enum evolves_.
Why? This isn't some external enum that might evolve out from under this code, but is part of the same feature. If the enum evolves then the resulting compiler error is a _good_ thing as it alerts whoever added the new enum that they didn't finish implementing it.
A default makes sense when the enum might evolve separately from the use of the enum.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r1940334817
More information about the openjfx-dev
mailing list