diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2015-04-06 16:43:55 -0700 |
|---|---|---|
| committer | Alex Crichton <alex@alexcrichton.com> | 2015-04-07 17:54:34 -0700 |
| commit | fcc89ea5006441350f0275946ff6db84a1118e49 (patch) | |
| tree | 9dc9f854b7ff4aaf76000797d42e22fe144e1ca8 /src/librustdoc/html/render.rs | |
| parent | c47bb7cd50e932a99213e004010e0f262f253446 (diff) | |
| download | rust-fcc89ea5006441350f0275946ff6db84a1118e49.tar.gz rust-fcc89ea5006441350f0275946ff6db84a1118e49.zip | |
rustdoc: Only hide possibly private modules
If an empty public module has no documentation, it shouldn't emit a page that's just a redirect loop to itself! Closes #16265
Diffstat (limited to 'src/librustdoc/html/render.rs')
| -rw-r--r-- | src/librustdoc/html/render.rs | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/librustdoc/html/render.rs b/src/librustdoc/html/render.rs index 40cc44d20eb..3796614bcf5 100644 --- a/src/librustdoc/html/render.rs +++ b/src/librustdoc/html/render.rs @@ -1352,7 +1352,9 @@ impl Context { fn ignore_private_item(&self, it: &clean::Item) -> bool { match it.inner { clean::ModuleItem(ref m) => { - (m.items.len() == 0 && it.doc_value().is_none()) || + (m.items.len() == 0 && + it.doc_value().is_none() && + it.visibility != Some(ast::Public)) || (self.passes.contains("strip-private") && it.visibility != Some(ast::Public)) } clean::PrimitiveItem(..) => it.visibility != Some(ast::Public), |