about summary refs log tree commit diff
diff options
context:
space:
mode:
authorboolean_coercion <booleancoercion@gmail.com>2021-02-12 11:53:52 +0200
committerboolean_coercion <booleancoercion@gmail.com>2021-02-12 11:54:22 +0200
commit642efabfbbba6577a3698a305bac8ce0c693e67f (patch)
tree898acc16f53055e7c0af4bce49710b3e3d1c0b2e
parent9194c11d69aa98c9d85fd0979bf9b93d3d7de809 (diff)
downloadrust-642efabfbbba6577a3698a305bac8ce0c693e67f.tar.gz
rust-642efabfbbba6577a3698a305bac8ce0c693e67f.zip
Fixed typos and updated to matches! where applicable
-rw-r--r--clippy_lints/src/from_str_radix_10.rs12
-rw-r--r--tests/ui/from_str_radix_10.stderr10
2 files changed, 7 insertions, 15 deletions
diff --git a/clippy_lints/src/from_str_radix_10.rs b/clippy_lints/src/from_str_radix_10.rs
index 53cb1e3ecd9..de9add4b6b6 100644
--- a/clippy_lints/src/from_str_radix_10.rs
+++ b/clippy_lints/src/from_str_radix_10.rs
@@ -46,7 +46,7 @@ impl LateLintPass<'tcx> for FromStrRadix10 {
             if let TyKind::Path(ty_qpath) = &ty.kind;
             let ty_res = cx.qpath_res(ty_qpath, ty.hir_id);
             if let def::Res::PrimTy(prim_ty) = ty_res;
-            if is_primitive_integer_ty(prim_ty);
+            if matches!(prim_ty, PrimTy::Int(_) | PrimTy::Uint(_));
 
             // check if the second part of the path indeed calls the associated
             // function `from_str_radix`
@@ -63,7 +63,7 @@ impl LateLintPass<'tcx> for FromStrRadix10 {
                     cx,
                     FROM_STR_RADIX_10,
                     exp.span,
-                    "This call to `from_str_radix` can be shortened to a call to str::parse",
+                    "this call to `from_str_radix` can be replaced with a call to `str::parse`",
                     "try",
                     format!("({}).parse()", orig_string),
                     Applicability::MaybeIncorrect
@@ -72,11 +72,3 @@ impl LateLintPass<'tcx> for FromStrRadix10 {
         }
     }
 }
-
-fn is_primitive_integer_ty(ty: PrimTy) -> bool {
-    match ty {
-        PrimTy::Int(_) => true,
-        PrimTy::Uint(_) => true,
-        _ => false,
-    }
-}
diff --git a/tests/ui/from_str_radix_10.stderr b/tests/ui/from_str_radix_10.stderr
index 376d0dd56ea..5557cd3b9ef 100644
--- a/tests/ui/from_str_radix_10.stderr
+++ b/tests/ui/from_str_radix_10.stderr
@@ -1,4 +1,4 @@
-error: This call to `from_str_radix` can be shortened to a call to str::parse
+error: this call to `from_str_radix` can be replaced with a call to `str::parse`
   --> $DIR/from_str_radix_10.rs:17:5
    |
 LL |     u32::from_str_radix("30", 10)?;
@@ -6,25 +6,25 @@ LL |     u32::from_str_radix("30", 10)?;
    |
    = note: `-D clippy::from-str-radix-10` implied by `-D warnings`
 
-error: This call to `from_str_radix` can be shortened to a call to str::parse
+error: this call to `from_str_radix` can be replaced with a call to `str::parse`
   --> $DIR/from_str_radix_10.rs:18:5
    |
 LL |     i64::from_str_radix("24", 10)?;
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `("24").parse()`
 
-error: This call to `from_str_radix` can be shortened to a call to str::parse
+error: this call to `from_str_radix` can be replaced with a call to `str::parse`
   --> $DIR/from_str_radix_10.rs:19:5
    |
 LL |     isize::from_str_radix("100", 10)?;
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `("100").parse()`
 
-error: This call to `from_str_radix` can be shortened to a call to str::parse
+error: this call to `from_str_radix` can be replaced with a call to `str::parse`
   --> $DIR/from_str_radix_10.rs:20:5
    |
 LL |     u8::from_str_radix("7", 10)?;
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `("7").parse()`
 
-error: This call to `from_str_radix` can be shortened to a call to str::parse
+error: this call to `from_str_radix` can be replaced with a call to `str::parse`
   --> $DIR/from_str_radix_10.rs:23:5
    |
 LL |     i32::from_str_radix(string, 10)?;