dnrops
/
rust
mirror of https://github.com/rust-lang/rust.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

rustdoc-json: discard non-local inherent impls

This commit is contained in:
schvv31n 2024-07-30 12:27:15 +01:00
parent 80d8270d84
commit 7499e21a1e
3 changed files with 20 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/librustdoc/formats/cache.rs
View File

@ -442,16 +442,16 @@ impl<'a, 'tcx> DocFolder for CacheBuilder<'a, 'tcx> {
// `public_items` map, so we can skip inserting into the // `public_items` map, so we can skip inserting into the
// paths map if there was already an entry present and we're // paths map if there was already an entry present and we're
// not a public item. // not a public item.
if !self.cache.paths.contains_key(&item.item_id.expect_def_id()) let item_def_id = item.item_id.expect_def_id();
if !self.cache.paths.contains_key(&item_def_id)
|| self || self
.cache .cache
.effective_visibilities .effective_visibilities
.is_directly_public(self.tcx, item.item_id.expect_def_id()) .is_directly_public(self.tcx, item_def_id)
{ {
self.cache.paths.insert( self.cache
item.item_id.expect_def_id(), .paths
(self.cache.stack.clone(), item.type_()), .insert(item_def_id, (self.cache.stack.clone(), item.type_()));
);
} }
} }
} }
@ -514,9 +514,7 @@ impl<'a, 'tcx> DocFolder for CacheBuilder<'a, 'tcx> {
&& adt.is_fundamental() && adt.is_fundamental()
{ {
for ty in generics { for ty in generics {
if let Some(did) = ty.def_id(self.cache) { dids.extend(ty.def_id(self.cache));
dids.insert(did);
}
} }
} }
} }
@ -529,32 +527,26 @@ impl<'a, 'tcx> DocFolder for CacheBuilder<'a, 'tcx> {
.primitive_type() .primitive_type()
.and_then(|t| self.cache.primitive_locations.get(&t).cloned()); .and_then(|t| self.cache.primitive_locations.get(&t).cloned());
if let Some(did) = did { dids.extend(did);
dids.insert(did);
}
} }
} }
if let Some(generics) = i.trait_.as_ref().and_then(|t| t.generics()) { if let Some(generics) = i.trait_.as_ref().and_then(|t| t.generics()) {
for bound in generics { for bound in generics {
if let Some(did) = bound.def_id(self.cache) { dids.extend(bound.def_id(self.cache));
dids.insert(did);
}
} }
} }
let impl_item = Impl { impl_item: item }; let impl_item = Impl { impl_item: item };
if impl_item.trait_did().map_or(true, |d| self.cache.traits.contains_key(&d)) { let impl_did = impl_item.def_id();
let trait_did = impl_item.trait_did();
if trait_did.map_or(true, |d| self.cache.traits.contains_key(&d)) {
for did in dids { for did in dids {
if self.impl_ids.entry(did).or_default().insert(impl_item.def_id()) { if self.impl_ids.entry(did).or_default().insert(impl_did) {
self.cache self.cache.impls.entry(did).or_default().push(impl_item.clone());
.impls
.entry(did)
.or_insert_with(Vec::new)
.push(impl_item.clone());
} }
} }
} else { } else {
let trait_did = impl_item.trait_did().expect("no trait did"); let trait_did = trait_did.expect("no trait did");
self.cache.orphan_trait_impls.push((trait_did, dids, impl_item)); self.cache.orphan_trait_impls.push((trait_did, dids, impl_item));
} }
None None

6
src/librustdoc/json/mod.rs
View File

@ -217,13 +217,7 @@ impl<'tcx> FormatRenderer<'tcx> for JsonRenderer<'tcx> {
fn after_krate(&mut self) -> Result<(), Error> { fn after_krate(&mut self) -> Result<(), Error> {
debug!("Done with crate"); debug!("Done with crate");
debug!("Adding Primitive impls");
for primitive in Rc::clone(&self.cache).primitive_locations.values() {
self.get_impls(*primitive);
}
let e = ExternalCrate { crate_num: LOCAL_CRATE }; let e = ExternalCrate { crate_num: LOCAL_CRATE };
let index = (*self.index).clone().into_inner(); let index = (*self.index).clone().into_inner();
debug!("Constructing Output"); debug!("Constructing Output");

5
tests/rustdoc-json/the_smallest.rs Normal file
View File

@ -0,0 +1,5 @@
// This test asserts that `index` is not polluted with unrelated items.
// See https://github.com/rust-lang/rust/issues/114039
//@ count "$.index[*]" 1
fn main() {}