Unnamed repository; edit this file 'description' to name the repository.
clippy: Enable `forget_non_drop` rule
Tetsuharu Ohzeki 2024-02-09
parent cdc3e83 · commit 3365e50
-rw-r--r--Cargo.toml1
-rw-r--r--crates/hir-ty/src/mir/lower.rs4
2 files changed, 4 insertions, 1 deletions
diff --git a/Cargo.toml b/Cargo.toml
index 8bec3893ce..d562a90b00 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -170,7 +170,6 @@ useless_asref = "allow"
## Following lints should be tackled at some point
borrowed_box = "allow"
derived_hash_with_manual_eq = "allow"
-forget_non_drop = "allow"
needless_doctest_main = "allow"
too_many_arguments = "allow"
type_complexity = "allow"
diff --git a/crates/hir-ty/src/mir/lower.rs b/crates/hir-ty/src/mir/lower.rs
index 28d26c6c8a..f4a076737a 100644
--- a/crates/hir-ty/src/mir/lower.rs
+++ b/crates/hir-ty/src/mir/lower.rs
@@ -126,6 +126,10 @@ impl DropScopeToken {
}
}
+impl Drop for DropScopeToken {
+ fn drop(&mut self) {}
+}
+
// Uncomment this to make `DropScopeToken` a drop bomb. Unfortunately we can't do this in release, since
// in cases that mir lowering fails, we don't handle (and don't need to handle) drop scopes so it will be
// actually reached. `pop_drop_scope_assert_finished` will also detect this case, but doesn't show useful