RFR: 8354871: Replace stack map frame type magics with constants [v3]

Sat Sep 6 13:48:54 UTC 2025

On Sat, 6 Sep 2025 13:22:56 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   inclusive
>
> src/java.base/share/classes/jdk/internal/classfile/impl/StackMapGenerator.java line 188:
> 
>> 186: 
>> 187:     public static final int
>> 188:             SAME_FRAME_END = 63,
> 
> Can you add a comment on these constants that they are inclusive on both ends? A lot of ranges in Java are open on their higher end, like range notations in Arrays.copyOf or String.substring.

Sure: 0c6c04b7b8874d3b3bd53c4f9456fe01b151241d

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27029#discussion_r2327436560