[jfx17] RFR: 8270246: Deprecate for removal implementation methods in Scene
Ambarish Rapte
arapte at openjdk.java.net
Thu Jul 15 15:32:13 UTC 2021
On Mon, 12 Jul 2021 15:52:17 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
> The following methods in Scene were former `impl_*` method that were intended to be encapsulated as part of [JDK-8157295](https://bugs.openjdk.java.net/browse/JDK-8157295) in JDK 9, but were mistakenly left as public:
>
>
> public void disposePeer()
> public void enableInputMethodEvents(boolean enable)
> public void processKeyEvent(KeyEvent e)
>
>
> In the fix for [JDK-8157295](https://bugs.openjdk.java.net/browse/JDK-8157295), the `impl_` prefix was removed, and the necessary accessors were added, but the `public` modifier was not removed. See the diffs for these three methods in Scene.java: [`disposePeer`](https://github.com/openjdk/jfx/commit/d864a620dfc7631585872547b42dcf93e9d570e6#diff-72cb5f9e4cc8a3804417166fdef390361e82d5890603f4e5158c9fc70c22e9daL833), [`enableInputMethodEvents`](https://github.com/openjdk/jfx/commit/d864a620dfc7631585872547b42dcf93e9d570e6#diff-72cb5f9e4cc8a3804417166fdef390361e82d5890603f4e5158c9fc70c22e9daL2221), [`processKeyEvent`](https://github.com/openjdk/jfx/commit/d864a620dfc7631585872547b42dcf93e9d570e6#diff-72cb5f9e4cc8a3804417166fdef390361e82d5890603f4e5158c9fc70c22e9daL2129)
>
> We will deprecate these methods for removal with the intention to remove them from the public API in a future version of JavaFX. This will need a CSR.
>
> NOTE: this is targeted to `jfx17`.
Looks good to me.
-------------
Marked as reviewed by arapte (Reviewer).
PR: https://git.openjdk.java.net/jfx/pull/573
More information about the openjfx-dev
mailing list