Skip to content

Commit 882f85b

Browse files
docs: explain more clearly why it was bad
1 parent 961adc9 commit 882f85b

File tree

1 file changed

+6
-2
lines changed
  • clippy_lints/src/methods

1 file changed

+6
-2
lines changed

clippy_lints/src/methods/mod.rs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3964,11 +3964,15 @@ declare_clippy_lint! {
39643964

39653965
declare_clippy_lint! {
39663966
/// ### What it does
3967-
/// Checks for unnecessary calls to `min()` or `max()`
3967+
/// Checks for unnecessary calls to `min()` or `max()` in the following cases
3968+
/// - Either both side is constant
3969+
/// - One side is clearly larger than the other, like i32::MIN and an i32 variable
39683970
///
39693971
/// ### Why is this bad?
39703972
///
3971-
/// In these cases it is not necessary to call `min()`
3973+
/// In the aformentioned cases it is not necessary to call `min()` or `max()`
3974+
/// to compare values, it may even cause confusion.
3975+
///
39723976
/// ### Example
39733977
/// ```no_run
39743978
/// let _ = 0.min(7_u32);

0 commit comments

Comments
 (0)