On Tue, 3 Mar 2026 01:31:07 GMT, David Holmes <[email protected]> wrote:
> Please review these simple changes to some of the JNI functions: > > - JNI `AllocObject` should not throw `InstantiationException` for value types > (and I've added a very simple test for that) > - JNI `NewObject*` had leftover code to explicitly throw > `InstantiationException` but that is already handled inside > `allocate_instance` > > Also fixed some alignment/whitespace issues that had crept in. > > Thanks Agree with @walulyai cleanup suggestion, otherwise looks good. ------------- Marked as reviewed by dsimms (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2190#pullrequestreview-3881531519
