Skip to content

Clarify that isqrt returns the principal (non-negative) square root#155558

Open
nivekithan wants to merge 1 commit intorust-lang:mainfrom
nivekithan:issue-154000-fix
Open

Clarify that isqrt returns the principal (non-negative) square root#155558
nivekithan wants to merge 1 commit intorust-lang:mainfrom
nivekithan:issue-154000-fix

Conversation

@nivekithan
Copy link
Copy Markdown

Clarify which root is returned from isqrt and checked_isqrt

Fixes: #154000
Previous PRs: #154462

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Apr 20, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 20, 2026

r? @scottmcm

rustbot has assigned @scottmcm.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @scottmcm, libs
  • @scottmcm, libs expanded to 7 candidates
  • Random selection from Mark-Simulacrum, jhpratt, scottmcm

@nivekithan
Copy link
Copy Markdown
Author

r? jhpratt

Since jhratt was reviewing previous PR

@rustbot rustbot assigned jhpratt and unassigned scottmcm Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(checked_)isqrt documentation for signed integers could clarify which root is returned

4 participants