On Wed, 12 Nov 2025 02:54:08 GMT, Sergey Bylokhov <[email protected]> wrote:
> This is an update to the patch integrated in > https://github.com/openjdk/jdk/pull/28127. It changes the exception from > IllegalArgumentException back to NullPointerException, which was thrown > previously. > > Rationale: > - Since an NPE was already thrown before, this does not introduce a behavior > change > - Throwing NPE on null is the common pattern used across the codebase, > including in the affected package > > Additional notes: > The patch uses the idiomatic way to check parameters for null via > Objects.requireNonNull(). I am not sure whether this code path is performance > critical. The use of >> 1 instead of / 2 suggests that performance might > matter. If this code is performance sensitive, we can remove the > Objects.requireNonNull() call, because even without it the resulting NPE > would still clearly show which variable was null when its field was accessed. > > It is also possible to remove the use of multiplyExact(). We can simply > multiply width/height as long values and compare the result with the data > length. In case of overflow, the existing “Data array too small” exception > would still be thrown. This pull request has now been integrated. Changeset: 37cd8d6c Author: Sergey Bylokhov <[email protected]> URL: https://git.openjdk.org/jdk/commit/37cd8d6ca0bc4638d81e9a3c1e0bc785861ffbef Stats: 41 lines in 2 files changed: 18 ins; 6 del; 17 mod 8371501: Change IAE to NPE in java.awt.image.Kernel when data is null Reviewed-by: prr, azvegint, aivanov ------------- PR: https://git.openjdk.org/jdk/pull/28249
