On Thu, 20 Aug 2026 17:11:20 GMT, Andy Goryachev <[email protected]> wrote:

>> Digging here, `ImageIO.write(bi, format, out);` returns `false` - no 
>> appropriate writer found
>
> // yes it can:
> byte[] b = null;
> 
> 
> 
> StubToolkit cannot write jpg.  The test works with the real toolkit.

I mean in our `writeImage` case, null is never returned, no? Could we perhaps 
split the jpg tests, because I like that we test `assertTrue(b.length > 0);` 
for `png` and `gif` with an opaque pixel. And for the jpg test, we then could 
test, that the `length == 0` perhaps?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2267#discussion_r3823763985

Reply via email to