about summary refs log tree commit diff
diff options
context:
space:
mode:
authorLukas Wirth <lukastw97@gmail.com>2022-04-09 01:08:13 +0200
committerLukas Wirth <lukastw97@gmail.com>2022-04-09 01:08:13 +0200
commit9050db2e80ee0190cc4a6eddc972a5e19bc9f926 (patch)
tree4765eb32e9ec9ccd391bcaef60798dc1f179f4ac
parent15e7112da31b8ffd246620831b3dc2b70a858af5 (diff)
downloadrust-9050db2e80ee0190cc4a6eddc972a5e19bc9f926.tar.gz
rust-9050db2e80ee0190cc4a6eddc972a5e19bc9f926.zip
fix: Don't create `hir::Local`s from const path patterns
-rw-r--r--crates/hir/src/semantics/source_to_def.rs9
-rw-r--r--crates/ide_assists/src/handlers/inline_local_variable.rs14
2 files changed, 21 insertions, 2 deletions
diff --git a/crates/hir/src/semantics/source_to_def.rs b/crates/hir/src/semantics/source_to_def.rs
index 4672e7db40c..5c4cfa7b45a 100644
--- a/crates/hir/src/semantics/source_to_def.rs
+++ b/crates/hir/src/semantics/source_to_def.rs
@@ -213,10 +213,15 @@ impl SourceToDefCtx<'_, '_> {
         src: InFile<ast::IdentPat>,
     ) -> Option<(DefWithBodyId, PatId)> {
         let container = self.find_pat_or_label_container(src.syntax())?;
-        let (_body, source_map) = self.db.body_with_source_map(container);
+        let (body, source_map) = self.db.body_with_source_map(container);
         let src = src.map(ast::Pat::from);
         let pat_id = source_map.node_pat(src.as_ref())?;
-        Some((container, pat_id))
+        // the pattern could resolve to a constant, verify that that is not the case
+        if let crate::Pat::Bind { .. } = body[pat_id] {
+            Some((container, pat_id))
+        } else {
+            None
+        }
     }
     pub(super) fn self_param_to_def(
         &mut self,
diff --git a/crates/ide_assists/src/handlers/inline_local_variable.rs b/crates/ide_assists/src/handlers/inline_local_variable.rs
index 0b4f46caf41..65dc2a0d511 100644
--- a/crates/ide_assists/src/handlers/inline_local_variable.rs
+++ b/crates/ide_assists/src/handlers/inline_local_variable.rs
@@ -938,4 +938,18 @@ fn f() {
 "#,
         );
     }
+
+    #[test]
+    fn test_inline_let_unit_struct() {
+        check_assist_not_applicable(
+            inline_local_variable,
+            r#"
+struct S;
+fn f() {
+    let S$0 = S;
+    S;
+}
+"#,
+        );
+    }
 }