Remove some unused methods from metadata

Address comments + Fix rebase

diff --git a/src/librustc/middle/cstore.rs b/src/librustc/middle/cstore.rs
index f1e9cfc0dae47b1b9c8d49b58316f0dfcfb9f6a5..dfd6256c35773ccc1227a5e2ab35ac443e488b04 100644 (file)
--- a/src/librustc/middle/cstore.rs
+++ b/src/librustc/middle/cstore.rs
@@ -201,7 +201,6 @@ fn def_index_for_def_key(&self,
                              -> Option<DefIndex>;
     fn def_key(&self, def: DefId) -> hir_map::DefKey;
     fn relative_def_path(&self, def: DefId) -> Option<hir_map::DefPath>;
-    fn struct_ctor_def_id(&self, struct_def_id: DefId) -> Option<DefId>;
     fn struct_field_names(&self, def: DefId) -> Vec<ast::Name>;
     fn item_children(&self, did: DefId) -> Vec<def::Export>;
 
@@ -377,8 +376,6 @@ fn def_key(&self, def: DefId) -> hir_map::DefKey { bug!("def_key") }
     fn relative_def_path(&self, def: DefId) -> Option<hir_map::DefPath> {
         bug!("relative_def_path")
     }
-    fn struct_ctor_def_id(&self, struct_def_id: DefId) -> Option<DefId>
-        { bug!("struct_ctor_def_id") }
     fn struct_field_names(&self, def: DefId) -> Vec<ast::Name> { bug!("struct_field_names") }
     fn item_children(&self, did: DefId) -> Vec<def::Export> { bug!("item_children") }
 

diff --git a/src/librustc_metadata/csearch.rs b/src/librustc_metadata/csearch.rs
index db2d033738651959be791ff24ab7e03571dfae6a..54664b9c04046effbc4b1d197e9be680435598d6 100644 (file)
--- a/src/librustc_metadata/csearch.rs
+++ b/src/librustc_metadata/csearch.rs
@@ -342,12 +342,6 @@ fn relative_def_path(&self, def: DefId) -> Option<hir_map::DefPath> {
         self.get_crate_data(def.krate).def_path(def.index)
     }
 
-    fn struct_ctor_def_id(&self, struct_def_id: DefId) -> Option<DefId>
-    {
-        self.dep_graph.read(DepNode::MetaData(struct_def_id));
-        self.get_crate_data(struct_def_id.krate).get_struct_ctor_def_id(struct_def_id.index)
-    }
-
     fn struct_field_names(&self, def: DefId) -> Vec<ast::Name>
     {
         self.dep_graph.read(DepNode::MetaData(def));

diff --git a/src/librustc_mir/transform/deaggregator.rs b/src/librustc_mir/transform/deaggregator.rs
index f4ef3edfa48ea6f894b00c7b5f427dbc25ac950e..198a6d256bcc325bdd914ab550fdd2fc04b671d2 100644 (file)
--- a/src/librustc_mir/transform/deaggregator.rs
+++ b/src/librustc_mir/transform/deaggregator.rs
@@ -9,7 +9,6 @@
 // except according to those terms.
 
 use rustc::ty::TyCtxt;
-use rustc::hir::def::CtorKind;
 use rustc::mir::repr::*;
 use rustc::mir::transform::{MirPass, MirSource, Pass};
 use rustc_data_structures::indexed_vec::Idx;
@@ -129,10 +128,7 @@ fn get_aggregate_statement_index<'a, 'tcx, 'b>(start: usize,
         }
         debug!("getting variant {:?}", variant);
         debug!("for adt_def {:?}", adt_def);
-        let variant_def = &adt_def.variants[variant];
-        if variant_def.ctor_kind == CtorKind::Fictive {
-            return Some(i);
-        }
+        return Some(i);
     };
     None
 }

diff --git a/src/librustc_resolve/build_reduced_graph.rs b/src/librustc_resolve/build_reduced_graph.rs
index ea894556b7fb5b6c8202d835d02d71b3486a8dc1..321b814238cfdb0c605ce86570ea2db9c51ac571 100644 (file)
--- a/src/librustc_resolve/build_reduced_graph.rs
+++ b/src/librustc_resolve/build_reduced_graph.rs
@@ -31,6 +31,7 @@
 
 use syntax::ast::Name;
 use syntax::attr;
+use syntax::parse::token;
 
 use syntax::ast::{self, Block, ForeignItem, ForeignItemKind, Item, ItemKind};
 use syntax::ast::{Mutability, StmtKind, TraitItem, TraitItemKind};

diff --git a/src/librustc_typeck/check/method/suggest.rs b/src/librustc_typeck/check/method/suggest.rs
index 4fd1341f09b727f62ee33f6605252b1754320e5f..ac8a35f08b0d97646e8409ea7f75243aa743a299 100644 (file)
--- a/src/librustc_typeck/check/method/suggest.rs
+++ b/src/librustc_typeck/check/method/suggest.rs
@@ -451,27 +451,26 @@ fn visit_item(&mut self, i: &'v hir::Item) {
         fn handle_external_def(ccx: &CrateCtxt,
                                traits: &mut AllTraitsVec,
                                external_mods: &mut FnvHashSet<DefId>,
-                               def_id: DefId) {
-            match ccx.tcx.sess.cstore.describe_def(def_id) {
-                Some(Def::Trait(_)) => {
+                               def: Def) {
+            let def_id = def.def_id();
+            match def {
+                Def::Trait(..) => {
                     traits.push(TraitInfo::new(def_id));
                 }
-                Some(Def::Mod(_)) => {
+                Def::Mod(..) => {
                     if !external_mods.insert(def_id) {
                         return;
                     }
                     for child in ccx.tcx.sess.cstore.item_children(def_id) {
-                        handle_external_def(ccx, traits, external_mods, child.def.def_id())
+                        handle_external_def(ccx, traits, external_mods, child.def)
                     }
                 }
                 _ => {}
             }
         }
         for cnum in ccx.tcx.sess.cstore.crates() {
-            handle_external_def(ccx, &mut traits, &mut external_mods, DefId {
-                krate: cnum,
-                index: CRATE_DEF_INDEX
-            });
+            let def_id = DefId { krate: cnum, index: CRATE_DEF_INDEX };
+            handle_external_def(ccx, &mut traits, &mut external_mods, Def::Mod(def_id));
         }
 
         *ccx.all_traits.borrow_mut() = Some(traits);

diff --git a/src/librustdoc/visit_lib.rs b/src/librustdoc/visit_lib.rs
index 1e262bb490635074f1858128a0f83440d8ef96f0..6d2830c56192bb434b73485384b6a9681d8d7d8b 100644 (file)
--- a/src/librustdoc/visit_lib.rs
+++ b/src/librustdoc/visit_lib.rs
@@ -66,11 +66,12 @@ fn update(&mut self, did: DefId, level: Option<AccessLevel>) -> Option<AccessLev
 
     pub fn visit_mod(&mut self, def_id: DefId) {
         for item in self.cstore.item_children(def_id) {
-            self.visit_item(item.def.def_id());
+            self.visit_item(item.def);
         }
     }
 
-    fn visit_item(&mut self, def_id: DefId) {
+    fn visit_item(&mut self, def: Def) {
+        let def_id = def.def_id();
         let vis = self.cstore.visibility(def_id);
         let inherited_item_level = if vis == Visibility::Public {
             self.prev_level
@@ -80,7 +81,7 @@ fn visit_item(&mut self, def_id: DefId) {
 
         let item_level = self.update(def_id, inherited_item_level);
 
-        if let Some(Def::Mod(_)) = self.cstore.describe_def(def_id) {
+        if let Def::Mod(..) = def {
             let orig_level = self.prev_level;
 
             self.prev_level = item_level;