Address comments
This commit is contained in:
parent
e459000bc1
commit
e301f90f22
1 changed files with 5 additions and 3 deletions
|
@ -783,7 +783,8 @@ impl<'a, 'tcx> Visitor<'tcx> for NamePrivacyVisitor<'a, 'tcx> {
|
||||||
}
|
}
|
||||||
|
|
||||||
fn visit_mod(&mut self, _m: &'tcx hir::Mod, _s: Span, _n: ast::NodeId) {
|
fn visit_mod(&mut self, _m: &'tcx hir::Mod, _s: Span, _n: ast::NodeId) {
|
||||||
// Don't visit modules inside
|
// Don't visit nested modules, since we run a separate visitor walk
|
||||||
|
// for each module in `privacy_access_levels`
|
||||||
}
|
}
|
||||||
|
|
||||||
fn visit_nested_body(&mut self, body: hir::BodyId) {
|
fn visit_nested_body(&mut self, body: hir::BodyId) {
|
||||||
|
@ -922,7 +923,8 @@ impl<'a, 'tcx> Visitor<'tcx> for TypePrivacyVisitor<'a, 'tcx> {
|
||||||
}
|
}
|
||||||
|
|
||||||
fn visit_mod(&mut self, _m: &'tcx hir::Mod, _s: Span, _n: ast::NodeId) {
|
fn visit_mod(&mut self, _m: &'tcx hir::Mod, _s: Span, _n: ast::NodeId) {
|
||||||
// Don't visit modules inside
|
// Don't visit nested modules, since we run a separate visitor walk
|
||||||
|
// for each module in `privacy_access_levels`
|
||||||
}
|
}
|
||||||
|
|
||||||
fn visit_nested_body(&mut self, body: hir::BodyId) {
|
fn visit_nested_body(&mut self, body: hir::BodyId) {
|
||||||
|
@ -1710,7 +1712,7 @@ fn privacy_access_levels<'tcx>(
|
||||||
|
|
||||||
let krate = tcx.hir().krate();
|
let krate = tcx.hir().krate();
|
||||||
|
|
||||||
for &module in tcx.hir().krate().modules.keys() {
|
for &module in krate.modules.keys() {
|
||||||
queries::check_mod_privacy::ensure(tcx, tcx.hir().local_def_id(module));
|
queries::check_mod_privacy::ensure(tcx, tcx.hir().local_def_id(module));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue