Skip to content

Commit 0c03dbb

Browse files
docs: explain more clearly why it was bad
1 parent 23855b6 commit 0c03dbb

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
@@ -3977,11 +3977,15 @@ declare_clippy_lint! {
39773977

39783978
declare_clippy_lint! {
39793979
/// ### What it does
3980-
/// Checks for unnecessary calls to `min()` or `max()`
3980+
/// Checks for unnecessary calls to `min()` or `max()` in the following cases
3981+
/// - Either both side is constant
3982+
/// - One side is clearly larger than the other, like i32::MIN and an i32 variable
39813983
///
39823984
/// ### Why is this bad?
39833985
///
3984-
/// In these cases it is not necessary to call `min()`
3986+
/// In the aformentioned cases it is not necessary to call `min()` or `max()`
3987+
/// to compare values, it may even cause confusion.
3988+
///
39853989
/// ### Example
39863990
/// ```no_run
39873991
/// let _ = 0.min(7_u32);

0 commit comments

Comments
 (0)