RFR: 8349373: Support JavaFX preview features [v3]
Kevin Rushforth
kcr at openjdk.org
Wed Mar 12 13:34:10 UTC 2025
On Fri, 7 Feb 2025 09:16:54 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:
>> This PR contains a definition of preview features for JavaFX, as well as a helper class to verify that an application has opted into preview features.
>
> Michael Strauß has updated the pull request incrementally with four additional commits since the last revision:
>
> - javadoc
> - remove supporting documentation
> - warning can't be suppressed
> - initialize PreviewFeature early
Looks good. I tested it and it does what I expected. I did some testing with the JDK and I was wrong about one thing:
> I don't think the warning should be suppressible; the equivalent JDK warning isn't
The key difference between the JDK and JavaFX is that the JDK only produces warnings (which are not suppressible) at compile time. Because of the compiler and JVM support, they didn't feel the need for a runtime warning.
Given that ours is a runtime warning, it might be worth restoring the system property to allow the warnings to be suppressed at runtime (default is false). What do you think?
Regarding compile time warnings for deprecated API: Should we specify that the `@Deprecated` annotation on preview features be marked as `forRemoval=true`? I can see arguments for both choices.
-------------
Marked as reviewed by kcr (Lead).
PR Review: https://git.openjdk.org/jfx/pull/1359#pullrequestreview-2678498489
More information about the openjfx-dev
mailing list