Skip to content
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.

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

Fix and rename overflow_check_conditional #12944

Merged
merged 3 commits into from
Jul 10, 2024
Merged

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Jun 17, 2024

fixes #2457

Other changes:

  • Limit the lint to unsigned types.
  • Actually check if the operands are the same rather than using only the first part of the path.
  • Allow the repeated expression to be anything as long as there are no side effects.

changelog: Rename overflow_check_conditional to panicking_overflow_check and move to correctness

@rustbot
Copy link
Collaborator

rustbot commented Jun 17, 2024

r? @y21

rustbot has assigned @y21.
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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 17, 2024
@bors
Copy link
Collaborator

bors commented Jul 5, 2024

☔ The latest upstream changes (presumably #12974) made this pull request unmergeable. Please resolve the merge conflicts.

Copy link
Member

@y21 y21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, r=me with merge conflicts fixed

@y21
Copy link
Member

y21 commented Jul 10, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 10, 2024

📌 Commit d2ff2b9 has been approved by y21

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jul 10, 2024

⌛ Testing commit d2ff2b9 with merge b012421...

@bors
Copy link
Collaborator

bors commented Jul 10, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: y21
Pushing b012421 to master...

@bors bors merged commit b012421 into rust-lang:master Jul 10, 2024
11 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

false positive: a - b > b is not really an underflow check
4 participants