On Fri, 21 Aug 2026 05:10:03 GMT, Prasanta Sadhukhan <[email protected]>
wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> unused imports
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/util/ImageUtils.java
> line 323:
>
>> 321: ImageIO.setUseCache(false);
>> 322: try {
>> 323: BufferedImage bi = fromFXImage(im, null);
>
> fromFXImage can return null too, guess null check is required so that it
> doesnt throw IllegalArumentException or maybe update javadoc to ensure user
> knows about it
good point: added error checking and updated the javadoc comments
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2267#discussion_r3831531186
