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

Reply via email to