Commit 7a9ae0c

committed

Auto merge of #106731 - flip1995:clippy_backport, r=Mark-Simulacrum

[beta] Clippy: Move manual_clamp to nursery There was a lot of discussion about this lint in rust-lang/rust-clippy#9484 (comment) We decided to move the lint to `nursery`. But since this lint broke code of many popular projects, we don't want to wait another release cycle until this move gets into stable. So we'd like to backport this commit to `beta`. cc `@rust-lang/clippy` for approval from the Clippy side.

2 parents 275123c + e74700c commit 7a9ae0cCopy full SHA for 7a9ae0c

File tree

1 file changed

-1

lines changed

src/tools/clippy/clippy_lints/src
- manual_clamp.rs

1 file changed

-1

lines changed

`‎src/tools/clippy/clippy_lints/src/manual_clamp.rs‎`

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,9 @@ declare_clippy_lint! {`
`35`	`35`	`/// Some may consider panicking in these situations to be desirable, but it also may`
`36`	`36`	`/// introduce panicking where there wasn't any before.`
`37`	`37`	`///`
	`38`	`+ /// See also [the discussion in the`
	`39`	`+ /// PR](https://github.com/rust-lang/rust-clippy/pull/9484#issuecomment-1278922613).`
	`40`	`+ ///`
`38`	`41`	`/// ### Examples`
`39`	`42`	/// ```rust
`40`	`43`	`/// # let (input, min, max) = (0, -2, 1);`
`@@ -78,7 +81,7 @@ declare_clippy_lint! {`
`78`	`81`	/// ```
`79`	`82`	`#[clippy::version = "1.66.0"]`
`80`	`83`	`pub MANUAL_CLAMP,`
`81`		`- complexity,`
	`84`	`+ nursery,`
`82`	`85`	`"using a clamp pattern instead of the clamp function"`
`83`	`86`	`}`
`84`	`87`	`impl_lint_pass!(ManualClamp => [MANUAL_CLAMP]);`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 7a9ae0c

File tree

1 file changed

1 file changed

`‎src/tools/clippy/clippy_lints/src/manual_clamp.rs‎`

0 commit comments