RFR: 8356126: Revisit CaptureCallState [v4]

Chen Liang liach at openjdk.org
Mon May 5 15:55:51 UTC 2025


On Mon, 5 May 2025 05:31:33 GMT, ExE Boss <duke at openjdk.org> wrote:

>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   No env to test
>
> src/java.base/share/classes/jdk/internal/foreign/abi/CapturableState.java line 96:
> 
>> 94:     public static String displayString(int mask) {
>> 95:         var displayList = new ArrayList<>();
>> 96:         for (var e : LOOKUP.values()) {
> 
> Suggestion:
> 
>         for (var e : SUPPORTED) {

Good point, however this is display only. To fix this I should add a comment to displayList.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25025#discussion_r2073708594


More information about the core-libs-dev mailing list