RFR: 4617681: constructor of BufferedImage throws unexpected IllegalArgumentException

[Sergey Bylokhov]

On Wed, 29 Oct 2025 13:27:49 GMT, Jayathirth D V <jdv at openjdk.org> wrote:

>I see that none of the code path under this API uses BandedSampleModel.

This discussion is about the possibility of changing that in the future. The current change prevents us from implementing those for large images(even by simply reusing BandedSampleModel instead of new class), since it includes strict limits and the TCK will enforce them. So, if we decide to change it later, we’ll also need to update the specification, which means the change can’t be backported.

>But that seems nearly impossible. There's too many things in the spec. and implementation that would need revisision.

What things? I don’t think we have any limits, except that we still need to implement it. Right now, we don’t define that behavior clearly. So, later, the user will get the correct large image when they ask for it, instead of getting an error. But the current patch is one of the thing which will prevent that, no?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r2488342302