RFR: 8306707: Support pluggable image loading via javax.imageio [v15]
Kevin Rushforth
kcr at openjdk.org
Wed Nov 6 21:29:49 UTC 2024
On Wed, 6 Nov 2024 18:46:38 GMT, Michael Strauß <mstrauss at 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
More information about the openjfx-dev
mailing list