On Fri, 21 Aug 2026 15:36:05 GMT, Andy Goryachev <[email protected]> wrote:

>> Any internal code that needs to read/write javafx images currently goes 
>> through the `ImageIO` (`java.desktop` module) and the `SwingFXUtils` (in 
>> `javafx.swing` module).
>> 
>> To avoid adding dependencies, we should move the bulk of implementation to 
>> the `javafx.graphics` module which in turn can be used by the `SwingFXUtils` 
>> and other internal code without adding additional explicit dependencies.
>> 
>> This PR moves the `SwingFXUtils` implementation to 
>> javafx.graphics/com.sun.javafx.util.ImageUtils and also adds a utility method
>> 
>> public static byte[] writeImage(Image im, String format) throws IOException
>> 
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Andy Goryachev has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains six additional 
> commits since the last revision:
> 
>  - error checking
>  - Merge branch 'master' into 8390345.image.io
>  - unused imports
>  - tests
>  - try with
>  - image utils

Marked as reviewed by mhanl (Reviewer).

-------------

PR Review: https://git.openjdk.org/jfx/pull/2267#pullrequestreview-5003056405

Reply via email to