fix typo in infinite loop lint

Takashiidobe · Takashiidobe · commit 67a33e8cc889 · 2023-12-22T18:44:20.000-05:00
diff --git a/clippy_lints/src/loops/infinite_loop.rs b/clippy_lints/src/loops/infinite_loop.rs
@@ -49,7 +49,7 @@ pub(super) fn check<'tcx>(
             if let FnRetTy::DefaultReturn(ret_span) = parent_fn_ret {
                 diag.span_suggestion(
                     ret_span,
-                    "if this is intentional, consider specifing `!` as function return",
+                    "if this is intentional, consider specifying `!` as function return",
                     " -> !",
                     Applicability::MaybeIncorrect,
                 );
diff --git a/tests/ui/infinite_loops.stderr b/tests/ui/infinite_loops.stderr
@@ -9,7 +9,7 @@ LL | |     }
    |
    = note: `-D clippy::infinite-loop` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::infinite_loop)]`
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn no_break() -> ! {
    |               ++++
@@ -26,7 +26,7 @@ LL | |         do_something();
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn all_inf() -> ! {
    |              ++++
@@ -43,7 +43,7 @@ LL | |             }
 LL | |         }
    | |_________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn all_inf() -> ! {
    |              ++++
@@ -57,7 +57,7 @@ LL | |                 do_something();
 LL | |             }
    | |_____________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn all_inf() -> ! {
    |              ++++
@@ -84,7 +84,7 @@ LL | |         do_something();
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn no_break_never_ret_noise() -> ! {
    |                               ++++
@@ -101,7 +101,7 @@ LL | |         }
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn break_inner_but_not_outer_1(cond: bool) -> ! {
    |                                            ++++
@@ -118,7 +118,7 @@ LL | |         }
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn break_inner_but_not_outer_2(cond: bool) -> ! {
    |                                            ++++
@@ -132,7 +132,7 @@ LL | |             do_something();
 LL | |         }
    | |_________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn break_outer_but_not_inner() -> ! {
    |                                ++++
@@ -149,7 +149,7 @@ LL | |             }
 LL | |         }
    | |_________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn break_wrong_loop(cond: bool) -> ! {
    |                                 ++++
@@ -166,7 +166,7 @@ LL | |         }
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn match_like() -> ! {
    |                 ++++
@@ -180,7 +180,7 @@ LL | |         let _x = matches!(result, Ok(v) if v != 0).then_some(0);
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn match_like() -> ! {
    |                 ++++
@@ -197,7 +197,7 @@ LL | |         });
 LL | |     }
    | |_____^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL | fn match_like() -> ! {
    |                 ++++
@@ -211,7 +211,7 @@ LL | |             do_something();
 LL | |         }
    | |_________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL |     fn problematic_trait_method() -> ! {
    |                                   ++++
@@ -225,7 +225,7 @@ LL | |             do_something();
 LL | |         }
    | |_________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL |     fn could_be_problematic() -> ! {
    |                               ++++
@@ -239,7 +239,7 @@ LL | |             do_something();
 LL | |         }
    | |_________^
    |
-help: if this is intentional, consider specifing `!` as function return
+help: if this is intentional, consider specifying `!` as function return
    |
 LL |     let _loop_forever = || -> ! {
    |                            ++++
@@ -256,4 +256,3 @@ LL | |     })
    = help: if this is not intended, try adding a `break` or `return` condition in the loop
 
 error: aborting due to 17 previous errors
-

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ LL \| \| }`
`9`	`9`	`\|`
`10`	`10`	= note: `-D clippy::infinite-loop` implied by `-D warnings`
`11`	`11`	= help: to override `-D warnings` add `#[allow(clippy::infinite_loop)]`
`12`		-help: if this is intentional, consider specifing `!` as function return
	`12`	+help: if this is intentional, consider specifying `!` as function return
`13`	`13`	`\|`
`14`	`14`	`LL \| fn no_break() -> ! {`
`15`	`15`	`\| ++++`
`@@ -26,7 +26,7 @@ LL \| \| do_something();`
`26`	`26`	`LL \| \| }`
`27`	`27`	`\| \|_____^`
`28`	`28`	`\|`
`29`		-help: if this is intentional, consider specifing `!` as function return
	`29`	+help: if this is intentional, consider specifying `!` as function return
`30`	`30`	`\|`
`31`	`31`	`LL \| fn all_inf() -> ! {`
`32`	`32`	`\| ++++`
`@@ -43,7 +43,7 @@ LL \| \| }`
`43`	`43`	`LL \| \| }`
`44`	`44`	`\| \|_________^`
`45`	`45`	`\|`
`46`		-help: if this is intentional, consider specifing `!` as function return
	`46`	+help: if this is intentional, consider specifying `!` as function return
`47`	`47`	`\|`
`48`	`48`	`LL \| fn all_inf() -> ! {`
`49`	`49`	`\| ++++`
`@@ -57,7 +57,7 @@ LL \| \| do_something();`
`57`	`57`	`LL \| \| }`
`58`	`58`	`\| \|_____________^`
`59`	`59`	`\|`
`60`		-help: if this is intentional, consider specifing `!` as function return
	`60`	+help: if this is intentional, consider specifying `!` as function return
`61`	`61`	`\|`
`62`	`62`	`LL \| fn all_inf() -> ! {`
`63`	`63`	`\| ++++`
`@@ -84,7 +84,7 @@ LL \| \| do_something();`
`84`	`84`	`LL \| \| }`
`85`	`85`	`\| \|_____^`
`86`	`86`	`\|`
`87`		-help: if this is intentional, consider specifing `!` as function return
	`87`	+help: if this is intentional, consider specifying `!` as function return
`88`	`88`	`\|`
`89`	`89`	`LL \| fn no_break_never_ret_noise() -> ! {`
`90`	`90`	`\| ++++`
`@@ -101,7 +101,7 @@ LL \| \| }`
`101`	`101`	`LL \| \| }`
`102`	`102`	`\| \|_____^`
`103`	`103`	`\|`
`104`		-help: if this is intentional, consider specifing `!` as function return
	`104`	+help: if this is intentional, consider specifying `!` as function return
`105`	`105`	`\|`
`106`	`106`	`LL \| fn break_inner_but_not_outer_1(cond: bool) -> ! {`
`107`	`107`	`\| ++++`
`@@ -118,7 +118,7 @@ LL \| \| }`
`118`	`118`	`LL \| \| }`
`119`	`119`	`\| \|_____^`
`120`	`120`	`\|`
`121`		-help: if this is intentional, consider specifing `!` as function return
	`121`	+help: if this is intentional, consider specifying `!` as function return
`122`	`122`	`\|`
`123`	`123`	`LL \| fn break_inner_but_not_outer_2(cond: bool) -> ! {`
`124`	`124`	`\| ++++`
`@@ -132,7 +132,7 @@ LL \| \| do_something();`
`132`	`132`	`LL \| \| }`
`133`	`133`	`\| \|_________^`
`134`	`134`	`\|`
`135`		-help: if this is intentional, consider specifing `!` as function return
	`135`	+help: if this is intentional, consider specifying `!` as function return
`136`	`136`	`\|`
`137`	`137`	`LL \| fn break_outer_but_not_inner() -> ! {`
`138`	`138`	`\| ++++`
`@@ -149,7 +149,7 @@ LL \| \| }`
`149`	`149`	`LL \| \| }`
`150`	`150`	`\| \|_________^`
`151`	`151`	`\|`
`152`		-help: if this is intentional, consider specifing `!` as function return
	`152`	+help: if this is intentional, consider specifying `!` as function return
`153`	`153`	`\|`
`154`	`154`	`LL \| fn break_wrong_loop(cond: bool) -> ! {`
`155`	`155`	`\| ++++`
`@@ -166,7 +166,7 @@ LL \| \| }`
`166`	`166`	`LL \| \| }`
`167`	`167`	`\| \|_____^`
`168`	`168`	`\|`
`169`		-help: if this is intentional, consider specifing `!` as function return
	`169`	+help: if this is intentional, consider specifying `!` as function return
`170`	`170`	`\|`
`171`	`171`	`LL \| fn match_like() -> ! {`
`172`	`172`	`\| ++++`
`@@ -180,7 +180,7 @@ LL \| \| let _x = matches!(result, Ok(v) if v != 0).then_some(0);`
`180`	`180`	`LL \| \| }`
`181`	`181`	`\| \|_____^`
`182`	`182`	`\|`
`183`		-help: if this is intentional, consider specifing `!` as function return
	`183`	+help: if this is intentional, consider specifying `!` as function return
`184`	`184`	`\|`
`185`	`185`	`LL \| fn match_like() -> ! {`
`186`	`186`	`\| ++++`
`@@ -197,7 +197,7 @@ LL \| \| });`
`197`	`197`	`LL \| \| }`
`198`	`198`	`\| \|_____^`
`199`	`199`	`\|`
`200`		-help: if this is intentional, consider specifing `!` as function return
	`200`	+help: if this is intentional, consider specifying `!` as function return
`201`	`201`	`\|`
`202`	`202`	`LL \| fn match_like() -> ! {`
`203`	`203`	`\| ++++`
`@@ -211,7 +211,7 @@ LL \| \| do_something();`
`211`	`211`	`LL \| \| }`
`212`	`212`	`\| \|_________^`
`213`	`213`	`\|`
`214`		-help: if this is intentional, consider specifing `!` as function return
	`214`	+help: if this is intentional, consider specifying `!` as function return
`215`	`215`	`\|`
`216`	`216`	`LL \| fn problematic_trait_method() -> ! {`
`217`	`217`	`\| ++++`
`@@ -225,7 +225,7 @@ LL \| \| do_something();`
`225`	`225`	`LL \| \| }`
`226`	`226`	`\| \|_________^`
`227`	`227`	`\|`
`228`		-help: if this is intentional, consider specifing `!` as function return
	`228`	+help: if this is intentional, consider specifying `!` as function return
`229`	`229`	`\|`
`230`	`230`	`LL \| fn could_be_problematic() -> ! {`
`231`	`231`	`\| ++++`
`@@ -239,7 +239,7 @@ LL \| \| do_something();`
`239`	`239`	`LL \| \| }`
`240`	`240`	`\| \|_________^`
`241`	`241`	`\|`
`242`		-help: if this is intentional, consider specifing `!` as function return
	`242`	+help: if this is intentional, consider specifying `!` as function return
`243`	`243`	`\|`
`244`	`244`	`LL \| let _loop_forever = \|\| -> ! {`
`245`	`245`	`\| ++++`
`@@ -256,4 +256,3 @@ LL \| \| })`
`256`	`256`	= help: if this is not intended, try adding a `break` or `return` condition in the loop
`257`	`257`
`258`	`258`	`error: aborting due to 17 previous errors`
`259`		`-`