Skip to content

Commit

Permalink
rustdoc: Show all impls of traits. rust-lang#5406
Browse files Browse the repository at this point in the history
  • Loading branch information
brson committed Mar 16, 2013
1 parent ebba8b4 commit deeeaf0
Showing 1 changed file with 23 additions and 1 deletion.
24 changes: 23 additions & 1 deletion src/librustdoc/prune_private_pass.rs
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,19 @@ fn is_visible(srv: astsrv::Srv, doc: doc::ItemDoc) -> bool {
do astsrv::exec(srv) |ctxt| {
match ctxt.ast_map.get(&id) {
ast_map::node_item(item, _) => {
item.vis == ast::public
match item.node {
ast::item_impl(_, Some(_), _, _) => {
// This is a trait implementation, make it visible
// NOTE: This is not quite right since this could be an impl
// of a private trait. We can't know that without running
// resolve though.
true
}
_ => {
// Otherwise just look at the visibility
item.vis == ast::public
}
}
}
_ => util::unreachable()
}
Expand All @@ -72,6 +84,16 @@ fn should_prune_items_without_pub_modifier() {
fail_unless!(vec::is_empty(doc.cratemod().mods()));
}

#[test]
fn unless_they_are_trait_impls() {
let doc = test::mk_doc(
~" \
trait Foo { } \
impl Foo for int { } \
");
fail_unless!(!doc.cratemod().impls().is_empty());
}
#[cfg(test)]
pub mod test {
use astsrv;
Expand Down

1 comment on commit deeeaf0

@thestinger
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

Please sign in to comment.