RFR: 4617681: constructor of BufferedImage throws unexpected IllegalArgumentException
Phil Race
prr at openjdk.org
Tue Oct 7 22:45:54 UTC 2025
On Tue, 7 Oct 2025 19:49:55 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
> > The work and compatibility concerns to address these are out of all proportion to simply documenting what has been true since the very beginning - 27 years plus - without a compelling reason.
>
> What compatibility impact?
That is in reponse to your suggestion to change the internal raster type. Not to a spec working change.
It has been throwing exceptions all this time, but currently it is allowed to work if we change that in the future, unlike the current proposal, which describes all implementation details as part of the specification, making it much harder to change later.
(1) No it allows leeway
(2) Even if it didn't we can relax it if there's ever a compelling enough reason - which I doubt we'll find.
> Instead we can mention "maximum supported length" in the spec and in the "impl section" mention that currently it is the size of the array/maxint,
I don't see how that is better.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r2412096298
More information about the client-libs-dev
mailing list