RFR: 8338544: Dedicated Array class descriptor implementation [v3]

Chen Liang liach at openjdk.org
Tue Oct 8 21:10:00 UTC 2024


On Tue, 8 Oct 2024 19:42:14 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   mlchung suggestions: elementType, improve utility methods
>
> src/java.base/share/classes/jdk/internal/constant/ConstantUtils.java line 313:
> 
>> 311:             throw new IllegalArgumentException("rank " + rank + " is not a positive value");
>> 312:         }
>> 313:     }
> 
> Suggest to inline the rank argument check in the caller method which makes the check explicit to the reader.

So like `if (rank <= 0) throw ConstantUtils.rankNotPositive(rank);` at individual use sites?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20665#discussion_r1792507736


More information about the core-libs-dev mailing list