Skip to content

Fix Javadoc in DoubleRange.of(Double, Double) to reflect actual exception type thrown #1581

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
garydgregory merged 1 commit into from
Jan 24, 2026
Merged

Fix Javadoc in DoubleRange.of(Double, Double) to reflect actual exception type thrown #1581

garydgregory merged 1 commit into from
Jan 24, 2026
+1 −1

Conversation

@inponomarev
Copy link
Contributor

@inponomarev inponomarev commented Jan 24, 2026

Update Javadoc @throws from IllegalArgumentException to NullPointerException to match actual behavior when either element is null (thrown via Objects.requireNonNull).

@garydgregory garydgregory merged commit b076e47 into apache:master Jan 24, 2026
20 checks passed
@garydgregory garydgregory changed the title Fix JavaDoc to reflect actual exception type thrown (IllegalArgumentException to NullPointerException) Fix Javadoc in DoubleRange.of(Double, Double) to reflect actual exception type thrown (IllegalArgumentException to NullPointerException) Jan 24, 2026
@garydgregory garydgregory changed the title Fix Javadoc in DoubleRange.of(Double, Double) to reflect actual exception type thrown (IllegalArgumentException to NullPointerException) Fix Javadoc in DoubleRange.of(Double, Double) to reflect actual exception type thrown Jan 24, 2026
garydgregory added a commit that referenced this pull request Jan 24, 2026
@garydgregory
Fix Javadoc in DoubleRange.of(Double, Double) to reflect actual
ae6b3eb 
exception type thrown #1581.
@garydgregory
Copy link
Member

garydgregory commented Jan 24, 2026

@inponomarev
Thank you for your PR. I merged to git master and fixed the same issue in the other classes in this hierarchy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@inponomarev @garydgregory