Added additional tests

Author: pmk21

tests/ui/implicit_saturating_sub.rs

@@ -102,6 +102,16 @@ fn main() {
         i_8 -= 1;
     }
 
+    // Lint
+    if i_8 != i8::MIN {
+        i_8 -= 1;
+    }
+
+    // Lint
+    if i_8 != i8::min_value() {
+        i_8 -= 1;
+    }
+
     let endi_16: i16 = 45;
     let starti_16: i16 = 44;
 
@@ -117,6 +127,16 @@ fn main() {
         i_16 -= 1;
     }
 
+    // Lint
+    if i_16 != i16::MIN {
+        i_16 -= 1;
+    }
+
+    // Lint
+    if i_16 != i16::min_value() {
+        i_16 -= 1;
+    }
+
     let endi_32: i32 = 45;
     let starti_32: i32 = 44;
 
@@ -132,6 +152,16 @@ fn main() {
         i_32 -= 1;
     }
 
+    // Lint
+    if i_32 != i32::MIN {
+        i_32 -= 1;
+    }
+
+    // Lint
+    if i_32 != i32::min_value() {
+        i_32 -= 1;
+    }
+
     let endi_64: i64 = 45;
     let starti_64: i64 = 44;
 
@@ -147,8 +177,23 @@ fn main() {
         i_64 -= 1;
     }
 
+    // Lint
+    if i_64 != i64::MIN {
+        i_64 -= 1;
+    }
+
+    // Lint
+    if i_64 != i64::min_value() {
+        i_64 -= 1;
+    }
+
     // No Lint
     if i_64 > 0 {
         i_64 -= 1;
     }
+
+    // No Lint
+    if i_64 != 0 {
+        i_64 -= 1;
+    }
 }

tests/ui/implicit_saturating_sub.stderr

@@ -64,53 +64,117 @@ LL | |         i_8 -= 1;
 LL | |     }
    | |_____^ help: try: `i_8 = i_8.saturating_sub(1);`
 
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:106:5
+   |
+LL | /     if i_8 != i8::MIN {
+LL | |         i_8 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_8 = i_8.saturating_sub(1);`
+
 error: Implicitly performing saturating subtraction
   --> $DIR/implicit_saturating_sub.rs:111:5
    |
+LL | /     if i_8 != i8::min_value() {
+LL | |         i_8 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_8 = i_8.saturating_sub(1);`
+
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:121:5
+   |
 LL | /     if i_16 > i16::MIN {
 LL | |         i_16 -= 1;
 LL | |     }
    | |_____^ help: try: `i_16 = i_16.saturating_sub(1);`
 
 error: Implicitly performing saturating subtraction
-  --> $DIR/implicit_saturating_sub.rs:116:5
+  --> $DIR/implicit_saturating_sub.rs:126:5
    |
 LL | /     if i_16 > i16::min_value() {
 LL | |         i_16 -= 1;
 LL | |     }
    | |_____^ help: try: `i_16 = i_16.saturating_sub(1);`
 
 error: Implicitly performing saturating subtraction
-  --> $DIR/implicit_saturating_sub.rs:126:5
+  --> $DIR/implicit_saturating_sub.rs:131:5
+   |
+LL | /     if i_16 != i16::MIN {
+LL | |         i_16 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_16 = i_16.saturating_sub(1);`
+
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:136:5
+   |
+LL | /     if i_16 != i16::min_value() {
+LL | |         i_16 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_16 = i_16.saturating_sub(1);`
+
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:146:5
    |
 LL | /     if i_32 > i32::MIN {
 LL | |         i_32 -= 1;
 LL | |     }
    | |_____^ help: try: `i_32 = i_32.saturating_sub(1);`
 
 error: Implicitly performing saturating subtraction
-  --> $DIR/implicit_saturating_sub.rs:131:5
+  --> $DIR/implicit_saturating_sub.rs:151:5
    |
 LL | /     if i_32 > i32::min_value() {
 LL | |         i_32 -= 1;
 LL | |     }
    | |_____^ help: try: `i_32 = i_32.saturating_sub(1);`
 
 error: Implicitly performing saturating subtraction
-  --> $DIR/implicit_saturating_sub.rs:141:5
+  --> $DIR/implicit_saturating_sub.rs:156:5
+   |
+LL | /     if i_32 != i32::MIN {
+LL | |         i_32 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_32 = i_32.saturating_sub(1);`
+
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:161:5
+   |
+LL | /     if i_32 != i32::min_value() {
+LL | |         i_32 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_32 = i_32.saturating_sub(1);`
+
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:171:5
    |
 LL | /     if i_64 > i64::MIN {
 LL | |         i_64 -= 1;
 LL | |     }
    | |_____^ help: try: `i_64 = i_64.saturating_sub(1);`
 
 error: Implicitly performing saturating subtraction
-  --> $DIR/implicit_saturating_sub.rs:146:5
+  --> $DIR/implicit_saturating_sub.rs:176:5
    |
 LL | /     if i_64 > i64::min_value() {
 LL | |         i_64 -= 1;
 LL | |     }
    | |_____^ help: try: `i_64 = i_64.saturating_sub(1);`
 
-error: aborting due to 14 previous errors
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:181:5
+   |
+LL | /     if i_64 != i64::MIN {
+LL | |         i_64 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_64 = i_64.saturating_sub(1);`
+
+error: Implicitly performing saturating subtraction
+  --> $DIR/implicit_saturating_sub.rs:186:5
+   |
+LL | /     if i_64 != i64::min_value() {
+LL | |         i_64 -= 1;
+LL | |     }
+   | |_____^ help: try: `i_64 = i_64.saturating_sub(1);`
+
+error: aborting due to 22 previous errors