On Fri, 4 Apr 2025 09:14:22 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> Implement JEP 502.
>> 
>> The PR passes tier1-tier3 tests.
>
> Per Minborg has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Address comments on docs

src/java.base/share/classes/jdk/internal/lang/stable/StableValueImpl.java line 
65:

> 63:     // | null           |  Unset        |
> 64:     // | nullSentinel() |  Set(null)    |
> 65:     // | other          |  Set(other)   |

There is no longer any method called `nullSentinel()`:
Suggestion:

    // | Value         |  Meaning      |
    // | ------------- |  ------------ |
    // | null          |  Unset        |
    // | NULL_SENTINEL |  Set(null)    |
    // | other         |  Set(other)   |

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2029434983

Reply via email to