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