diff options
| author | Maybe Waffle <waffle.lapkin@gmail.com> | 2022-10-07 17:08:29 +0000 |
|---|---|---|
| committer | Maybe Waffle <waffle.lapkin@gmail.com> | 2022-10-09 13:07:21 +0000 |
| commit | 7cfc6fa1f0c847c749929925f9def0fdc690d417 (patch) | |
| tree | 1c08501d9ca5c38691eaaa2cbd8ca933ff01ae07 /src | |
| parent | 05dcfd971a2736cf0eac04bb04f6539f36d05a44 (diff) | |
| download | rust-7cfc6fa1f0c847c749929925f9def0fdc690d417.tar.gz rust-7cfc6fa1f0c847c749929925f9def0fdc690d417.zip | |
deprecate `clippy::for_loops_over_fallibles`
Diffstat (limited to 'src')
| -rw-r--r-- | src/docs.rs | 1 | ||||
| -rw-r--r-- | src/docs/for_loops_over_fallibles.txt | 32 |
2 files changed, 0 insertions, 33 deletions
diff --git a/src/docs.rs b/src/docs.rs index 3bf488ab477..bd27bc7938f 100644 --- a/src/docs.rs +++ b/src/docs.rs @@ -170,7 +170,6 @@ docs! { "fn_to_numeric_cast_any", "fn_to_numeric_cast_with_truncation", "for_kv_map", - "for_loops_over_fallibles", "forget_copy", "forget_non_drop", "forget_ref", diff --git a/src/docs/for_loops_over_fallibles.txt b/src/docs/for_loops_over_fallibles.txt deleted file mode 100644 index c5a7508e45d..00000000000 --- a/src/docs/for_loops_over_fallibles.txt +++ /dev/null @@ -1,32 +0,0 @@ -### What it does -Checks for `for` loops over `Option` or `Result` values. - -### Why is this bad? -Readability. This is more clearly expressed as an `if -let`. - -### Example -``` -for x in opt { - // .. -} - -for x in &res { - // .. -} - -for x in res.iter() { - // .. -} -``` - -Use instead: -``` -if let Some(x) = opt { - // .. -} - -if let Ok(x) = res { - // .. -} -``` \ No newline at end of file |