RFR: 8313424: JavaFX controls in the title bar (Preview) [v78]
Michael Strauß
mstrauss at openjdk.org
Wed Jun 4 14:48:31 UTC 2025
On Tue, 3 Jun 2025 23:48:33 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>>
>> fix memory leak in ViewScene
>
> modules/javafx.base/src/main/java/com/sun/javafx/PreviewFeature.java line 63:
>
>> 61: * This method is only used for testing purposes.
>> 62: */
>> 63: public static void enableForTesting() {
>
> Minor: As an alternative, have you considered adding a utility method in a test utility class that sets both the `javafx.enablePreview` and `javafx.suppressPreviewWarning` system properties? Unless there a reason that wouldn't work, it seems cleaner to me.
I've modified the build script instead to always specify the system properties for tests.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r2126794640
More information about the openjfx-dev
mailing list