rename implicit_saturating_{add,sub} and inverted_saturating_sub

[ada4a]

View all comments

As per #clippy > `implicit_saturating_{add,sub}`: rename to `manual_*`

changelog: [implicit_saturating_add]: rename to manual_saturating_add
changelog: [implicit_saturating_sub]: rename to manual_saturating_sub
changelog: [inverted_saturating_sub]: rename to almost_saturating_sub

[rustbot]

r? @blyxyas

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

[github-actions]

Lintcheck changes for d7d7322

Lint	Added	Removed	Changed
clippy::implicit_saturating_sub	0	3	0
clippy::manual_saturating_sub	3	0	0

This comment will be updated if you push new changes

[ada4a]

reviewer is on vacation, let's reroll

r? clippy

[samueltardieu]

Do you still want to do this @ada4a? If you do, can you please open a FCP?

[rustbot]

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

[ada4a]

Thank you for taking the time to go through the old PRs:)

FCP opened at #clippy > FCP: rename {implicit,inverted}_saturating_{add,sub}

[rustbot]

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

[ada4a]

Let's do this in one big PR, as per #clippy > FCP: rename {implicit,inverted}saturating{add,sub} @ 💬