Skip to content

Commit

Permalink
Modules are either public, or private, so private modules should be s…
Browse files Browse the repository at this point in the history
…tripped out
  • Loading branch information
Aatch committed Mar 17, 2014
1 parent 9e89ffc commit 7180b5d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/librustdoc/passes.rs
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,8 @@ impl<'a> fold::DocFolder for Stripper<'a> {
}
}

clean::ViewItemItem(..) => {
clean::ViewItemItem(..) |
clean::ModuleItem(..) => {
if i.visibility != Some(ast::Public) {
return None
}
Expand All @@ -140,9 +141,6 @@ impl<'a> fold::DocFolder for Stripper<'a> {
}
}

// handled below
clean::ModuleItem(..) => {}

// trait impls for private items should be stripped
clean::ImplItem(clean::Impl{ for_: clean::ResolvedPath{ id: ref for_id, .. }, .. }) => {
if !self.exported_items.contains(for_id) {
Expand Down

9 comments on commit 7180b5d

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

saw approval from alexcrichton
at Aatch@7180b5d

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

merging Aatch/rust/rustdoc-mod-privacy = 7180b5d into auto

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

Aatch/rust/rustdoc-mod-privacy = 7180b5d merged ok, testing candidate = a269c376

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

saw approval from alexcrichton
at Aatch@7180b5d

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

merging Aatch/rust/rustdoc-mod-privacy = 7180b5d into auto

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

Aatch/rust/rustdoc-mod-privacy = 7180b5d merged ok, testing candidate = 7532d20

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

@bors
Copy link
Contributor

@bors bors commented on 7180b5d Mar 19, 2014

Choose a reason for hiding this comment

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

fast-forwarding master to auto = 7532d20

Please sign in to comment.