fix warning message

Author: jonboh

clippy_lints/src/unnecessary_map_on_constructor.rs

@@ -88,7 +88,7 @@ impl<'tcx> LateLintPass<'tcx> for UnnecessaryMapOnConstructor {
                     cx,
                     UNNECESSARY_MAP_ON_CONSTRUCTOR,
                     expr.span,
-                    &format!("unnecessary {fun_snippet} on contstuctor {constructor_snippet}(_)"),
+                    &format!("unnecessary {} on constructor {constructor_snippet}(_)", path.ident.name),
                     "try",
                     format!("{constructor_snippet}({fun_snippet}({constructor_arg_snippet}))"),
                     applicability,

tests/ui/unnecessary_map_on_constructor.stderr

@@ -1,48 +1,48 @@
-error: unnecessary fun on contstuctor Some(_)
+error: unnecessary map on constructor Some(_)
   --> $DIR/unnecessary_map_on_constructor.rs:32:13
    |
 LL |     let a = Some(x).map(fun);
    |             ^^^^^^^^^^^^^^^^ help: try: `Some(fun(x))`
    |
    = note: `-D clippy::unnecessary-map-on-constructor` implied by `-D warnings`
 
-error: unnecessary fun on contstuctor Ok(_)
+error: unnecessary map on constructor Ok(_)
   --> $DIR/unnecessary_map_on_constructor.rs:33:27
    |
 LL |     let b: SimpleResult = Ok(x).map(fun);
    |                           ^^^^^^^^^^^^^^ help: try: `Ok(fun(x))`
 
-error: unnecessary notfun on contstuctor Err(_)
+error: unnecessary map_err on constructor Err(_)
   --> $DIR/unnecessary_map_on_constructor.rs:34:27
    |
 LL |     let c: SimpleResult = Err(err).map_err(notfun);
    |                           ^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `Err(notfun(err))`
 
-error: unnecessary fun on contstuctor Option::Some(_)
+error: unnecessary map on constructor Option::Some(_)
   --> $DIR/unnecessary_map_on_constructor.rs:36:13
    |
 LL |     let a = Option::Some(x).map(fun);
    |             ^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `Option::Some(fun(x))`
 
-error: unnecessary fun on contstuctor SimpleResult::Ok(_)
+error: unnecessary map on constructor SimpleResult::Ok(_)
   --> $DIR/unnecessary_map_on_constructor.rs:37:27
    |
 LL |     let b: SimpleResult = SimpleResult::Ok(x).map(fun);
    |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `SimpleResult::Ok(fun(x))`
 
-error: unnecessary notfun on contstuctor SimpleResult::Err(_)
+error: unnecessary map_err on constructor SimpleResult::Err(_)
   --> $DIR/unnecessary_map_on_constructor.rs:38:27
    |
 LL |     let c: SimpleResult = SimpleResult::Err(err).map_err(notfun);
    |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `SimpleResult::Err(notfun(err))`
 
-error: unnecessary fun on contstuctor Ok(_)
+error: unnecessary map on constructor Ok(_)
   --> $DIR/unnecessary_map_on_constructor.rs:39:52
    |
 LL |     let b: std::result::Result<i32, SimpleError> = Ok(x).map(fun);
    |                                                    ^^^^^^^^^^^^^^ help: try: `Ok(fun(x))`
 
-error: unnecessary notfun on contstuctor Err(_)
+error: unnecessary map_err on constructor Err(_)
   --> $DIR/unnecessary_map_on_constructor.rs:40:52
    |
 LL |     let c: std::result::Result<i32, SimpleError> = Err(err).map_err(notfun);