diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2021-05-24 20:59:51 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-05-24 20:59:51 +0100 |
commit | 86ca1764b66e65fcb986b7188b6a77e8b53f9758 (patch) | |
tree | 9f30e5c52c33482b37bdcdff595aa38bf3a0a263 /xtask/src | |
parent | f5f24a9a2c2e9d5d5ff155e700b2bf647f926d47 (diff) | |
parent | 3641abc0c3e3ffaa1f6b5bf9b280b4217ea93e67 (diff) |
Merge #8975
8975: Use todo!() as placeholder body for generated match arms r=matklad a=jDomantas
`todo!()` seems to be a better fit for this than `{}`. Seeing that this assist predates stabilization of `todo` my guess is that simply no one bothered to change it yet.
Also fixed the issue where if the last arm was not block-like, rust-analyzer would not add a comma after it and would generate invalid code.
Co-authored-by: Domantas Jadenkus <djadenkus@gmail.com>
Diffstat (limited to 'xtask/src')
-rw-r--r-- | xtask/src/tidy.rs | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/xtask/src/tidy.rs b/xtask/src/tidy.rs index 82b33a7a0..6f687a788 100644 --- a/xtask/src/tidy.rs +++ b/xtask/src/tidy.rs | |||
@@ -275,6 +275,7 @@ fn check_todo(path: &Path, text: &str) { | |||
275 | // Some of our assists generate `todo!()`. | 275 | // Some of our assists generate `todo!()`. |
276 | "handlers/add_turbo_fish.rs", | 276 | "handlers/add_turbo_fish.rs", |
277 | "handlers/generate_function.rs", | 277 | "handlers/generate_function.rs", |
278 | "handlers/fill_match_arms.rs", | ||
278 | // To support generating `todo!()` in assists, we have `expr_todo()` in | 279 | // To support generating `todo!()` in assists, we have `expr_todo()` in |
279 | // `ast::make`. | 280 | // `ast::make`. |
280 | "ast/make.rs", | 281 | "ast/make.rs", |