From e921195d93c443b20a6fcb3cb1c5b8117fe2fa1b Mon Sep 17 00:00:00 2001
From: Edwin Cheng <edwin0cheng@gmail.com>
Date: Tue, 5 May 2020 01:15:27 +0800
Subject: Change favor_types to has_constructor

---
 crates/ra_hir_def/src/body/lower.rs        | 8 ++++----
 crates/ra_hir_def/src/item_scope.rs        | 4 ++--
 crates/ra_hir_def/src/nameres/collector.rs | 6 +++---
 3 files changed, 9 insertions(+), 9 deletions(-)

(limited to 'crates/ra_hir_def/src')

diff --git a/crates/ra_hir_def/src/body/lower.rs b/crates/ra_hir_def/src/body/lower.rs
index ea0ddeebc..4671b72e9 100644
--- a/crates/ra_hir_def/src/body/lower.rs
+++ b/crates/ra_hir_def/src/body/lower.rs
@@ -575,15 +575,15 @@ impl ExprCollector<'_> {
             self.body.item_scope.define_def(def);
             if let Some(name) = name {
                 let vis = crate::visibility::Visibility::Public; // FIXME determine correctly
-                let favor_types = match def {
+                let has_constructor = match def {
                     ModuleDefId::AdtId(AdtId::StructId(s)) => {
-                        self.db.struct_data(s).variant_data.kind() == StructKind::Record
+                        self.db.struct_data(s).variant_data.kind() != StructKind::Record
                     }
-                    _ => false,
+                    _ => true,
                 };
                 self.body.item_scope.push_res(
                     name.as_name(),
-                    crate::per_ns::PerNs::from_def(def, vis, favor_types),
+                    crate::per_ns::PerNs::from_def(def, vis, has_constructor),
                 );
             }
         }
diff --git a/crates/ra_hir_def/src/item_scope.rs b/crates/ra_hir_def/src/item_scope.rs
index 6d8d1f8a3..954f2542a 100644
--- a/crates/ra_hir_def/src/item_scope.rs
+++ b/crates/ra_hir_def/src/item_scope.rs
@@ -151,7 +151,7 @@ impl ItemScope {
 }
 
 impl PerNs {
-    pub(crate) fn from_def(def: ModuleDefId, v: Visibility, favor_types: bool) -> PerNs {
+    pub(crate) fn from_def(def: ModuleDefId, v: Visibility, has_constructor: bool) -> PerNs {
         match def {
             ModuleDefId::ModuleId(_) => PerNs::types(def, v),
             ModuleDefId::FunctionId(_) => PerNs::values(def, v),
@@ -159,7 +159,7 @@ impl PerNs {
                 AdtId::UnionId(_) => PerNs::both(def, def, v),
                 AdtId::EnumId(_) => PerNs::types(def, v),
                 AdtId::StructId(_) => {
-                    if favor_types {
+                    if !has_constructor {
                         PerNs::types(def, v)
                     } else {
                         PerNs::both(def, def, v)
diff --git a/crates/ra_hir_def/src/nameres/collector.rs b/crates/ra_hir_def/src/nameres/collector.rs
index 49b33ca94..db994122a 100644
--- a/crates/ra_hir_def/src/nameres/collector.rs
+++ b/crates/ra_hir_def/src/nameres/collector.rs
@@ -844,7 +844,7 @@ impl ModCollector<'_, '_> {
         let name = def.name.clone();
         let container = ContainerId::ModuleId(module);
         let vis = &def.visibility;
-        let mut favor_types = false;
+        let mut has_constructor = false;
 
         let def: ModuleDefId = match def.kind {
             raw::DefKind::Function(ast_id) => FunctionLoc {
@@ -854,7 +854,7 @@ impl ModCollector<'_, '_> {
             .intern(self.def_collector.db)
             .into(),
             raw::DefKind::Struct(ast_id, mode) => {
-                favor_types = mode == raw::StructDefKind::Record;
+                has_constructor = mode != raw::StructDefKind::Record;
                 StructLoc { container, ast_id: AstId::new(self.file_id, ast_id) }
                     .intern(self.def_collector.db)
                     .into()
@@ -899,7 +899,7 @@ impl ModCollector<'_, '_> {
             .unwrap_or(Visibility::Public);
         self.def_collector.update(
             self.module_id,
-            &[(name, PerNs::from_def(def, vis, favor_types))],
+            &[(name, PerNs::from_def(def, vis, has_constructor))],
             vis,
         )
     }
-- 
cgit v1.2.3