On Thu, 16 Nov 2023 11:25:07 GMT, Chen Liang <[email protected]> wrote:
>> Adam Sotona has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> changed StackMapGenerator::generatorError to return an exception instead
>> of directly throw
>
> src/java.base/share/classes/jdk/internal/classfile/impl/StackMapGenerator.java
> line 1138:
>
>> 1136: }
>> 1137: if (stackSize != target.stackSize) {
>> 1138: generatorError("Stack size mismatch");
>
> Just a side comment, `generatorError` should return a Throwable than throw
> directly, so when used as `throw generatorError()`, it is more clear that the
> code branch will terminate to javac.
Good point, changed, thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16685#discussion_r1399513530
