[code-reflection] RFR: throw instead returning when erasing checked to unchecked exception

Hannes Greule hgreule at openjdk.org
Wed Jan 17 10:24:45 UTC 2024


On Wed, 17 Jan 2024 10:08:14 GMT, Hannes Greule <hgreule at openjdk.org> wrote:

> As discussed in https://mail.openjdk.org/pipermail/babylon-dev/2024-January/000006.html, I change the `erase` method to `eraseAndThrow` and added a test as suggested. I ensured that the test fails without my change and does not fail with it.

test/jdk/java/lang/reflect/code/interpreter/TestThrowing.java line 45:

> 43:         Method method = TestThrowing.class.getDeclaredMethod(methodName);
> 44:         try {
> 45:             Interpreter.invoke(method.getCodeModel().orElseThrow(), List.of());

Note I had to add `List.of()` here as the interpreter otherwise throws an exception because of mismatching parameter sizes. I assume that's a bug too? If you want, I can look into that as well.

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

PR Review Comment: https://git.openjdk.org/babylon/pull/2#discussion_r1455149906


More information about the babylon-dev mailing list