On Tue, 14 May 2024 15:59:33 GMT, Paul Sandoz <psan...@openjdk.org> wrote:

>> Adam Sotona has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   fixed exception message
>
> src/java.base/share/classes/jdk/internal/classfile/impl/CodeImpl.java line 
> 241:
> 
>> 239:                     int startPc = classReader.readU2(p);
>> 240:                     if (startPc > codeLength) {
>> 241:                         throw new 
>> IllegalArgumentException(String.format("Line number out of range; 
>> start_pc=%d, codeLength=%d",
> 
> It's the byte code index that is out of range, not the line number associated 
> with it.

Right, fixed the message, thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19230#discussion_r1601131607

Reply via email to