On Wed, 6 Nov 2024 18:46:38 GMT, Michael Strauß <mstra...@openjdk.org> wrote:
>> modules/javafx.graphics/src/test/addExports line 22: >> >>> 20: --add-exports javafx.graphics/com.sun.javafx.iio=ALL-UNNAMED >>> 21: --add-exports javafx.graphics/com.sun.javafx.iio.png=ALL-UNNAMED >>> 22: --add-exports javafx.graphics/com.sun.javafx.iio.javax=ALL-UNNAMED >> >> I am little bit hesitant about this package name. Using "javax" doesn't give >> clear idea what we are trying to do here. >> >> May be com.sun.javafx.iio.imageio is a better option. > > I'm not so sure about this, since there's already "iio" in in the name. How about one of the following as an alternative? * com.sun.javafx.iio.java2d * com.sun.javafx.iio.j2d * com.sun.javafx.iio.desktop That's roughly in preference order, but I like any of them better than `...iio.javax`. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1593#discussion_r1831730909