RFR: 8350542: Optional.orElseThrow(Supplier) does not specify behavior when supplier returns null
Christoph Langer
clanger at openjdk.org
Tue Apr 29 04:19:55 UTC 2025
On Mon, 28 Apr 2025 20:04:22 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Javadoc for java.util.Optional.orElseThrow(Supplier) misses mentioning of another possible cause of a NullPointerException - when the exception supplying function returns a null result.
>> ---------
>> ### Progress
>> - [x] Change must be properly reviewed (1 review required, with at least 1 [Reviewer](https://openjdk.org/bylaws#reviewer))
>> - [x] Change requires CSR request [JDK-8354953](https://bugs.openjdk.org/browse/JDK-8354953) to be approved
>> - [x] Change must not contain extraneous whitespace
>> - [x] Commit message must refer to an issue
>>
>> ### Error
>> ⚠️ The pull request body must not be empty.
>>
>>
>>
>> ### Reviewing
>> <details><summary>Using <code>git</code></summary>
>>
>> Checkout this PR locally: \
>> `$ git fetch https://git.openjdk.org/jdk.git pull/23905/head:pull/23905` \
>> `$ git checkout pull/23905`
>>
>> Update a local copy of the PR: \
>> `$ git checkout pull/23905` \
>> `$ git pull https://git.openjdk.org/jdk.git pull/23905/head`
>>
>> </details>
>> <details><summary>Using Skara CLI tools</summary>
>>
>> Checkout this PR locally: \
>> `$ git pr checkout 23905`
>>
>> View PR using the GUI difftool: \
>> `$ git pr show -t 23905`
>>
>> </details>
>> <details><summary>Using diff file</summary>
>>
>> Download this PR as a diff file: \
>> <a href="https://git.openjdk.org/jdk/pull/23905.diff">https://git.openjdk.org/jdk/pull/23905.diff</a>
>>
>> </details>
>
> Also please add some of your own text to the pull request body, like the first sentence of the issue report:
>
>> Javadoc for java.util.Optional.orElseThrow(Supplier) misses mentioning of another possible cause of a NullPointerException - when the exception supplying function returns a null result.
>
> Once you have pushed the changes, Chris or I may review this for integration.
> @liach Can't I integrate?
When you are not a committer, some committer needs to sponsor it for you. I'm doing it. 😄
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23905#issuecomment-2837404350
More information about the core-libs-dev
mailing list