RFR: 8349373: Support JavaFX preview features
Andy Goryachev
angorya at openjdk.org
Tue Feb 4 18:22:34 UTC 2025
On Wed, 7 Feb 2024 20:05:01 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.
PREVIEW-FEATURES.md line 1:
> 1: # Preview features
is this a good location of this document?
should it be in /doc-files somewhere?
PREVIEW-FEATURES.md line 32:
> 30: application has not opted into the use of preview features. All preview features have equal status
> 31: in any given JavaFX release and can not be enabled individually.
> 32:
do you want to talk about similarities and differences between this and jdk preview features https://openjdk.org/jeps/12 ?
PREVIEW-FEATURES.md line 45:
> 43: for the preview feature. This name will be used in warning and error messages when the preview feature
> 44: is used by application developers.
> 45: 4. Add runtime checks in appropriate places by invoking `com.sun.javafx.PreviewFeature.<FEATURE>.checkEnabled()`.
what do you think about adding this checklist to the PreviewFeature class?
possibly following by an example (to allow for copy-paste)?
modules/javafx.base/src/main/java/com/sun/javafx/PreviewFeature.java line 37:
> 35: * has opted into preview features.
> 36: */
> 37: public enum PreviewFeature {
Why enum?
Maybe use a regular class with a public constructor should do it.
What do you think?
modules/javafx.base/src/main/java/com/sun/javafx/PreviewFeature.java line 78:
> 76: }
> 77:
> 78: private final String featureName;
would it be possible to move field to the beginning of the class please?
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1359#discussion_r1941678440
PR Review Comment: https://git.openjdk.org/jfx/pull/1359#discussion_r1941685598
PR Review Comment: https://git.openjdk.org/jfx/pull/1359#discussion_r1941683672
PR Review Comment: https://git.openjdk.org/jfx/pull/1359#discussion_r1941689779
PR Review Comment: https://git.openjdk.org/jfx/pull/1359#discussion_r1941690914
More information about the openjfx-dev
mailing list