rust-lang/rust
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Cleanup: node_id -> hir_id

Browse Source
This commit is contained in:
Philipp Hansch
2020-04-19 14:19:11 +02:00
parent 1c0e4e5b97
commit c3c00e5028
3 changed files with 10 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
clippy_lints/src/derive.rs
View File

@@ -120,9 +120,9 @@ fn check_hash_peq<'a, 'tcx>(
cx, DERIVE_HASH_XOR_EQ, span, cx, DERIVE_HASH_XOR_EQ, span,
mess, mess,
|diag| { |diag| {
if let Some(node_id) = cx.tcx.hir().as_local_hir_id(impl_id) { if let Some(hir_id) = cx.tcx.hir().as_local_hir_id(impl_id) {
diag.span_note( diag.span_note(
cx.tcx.hir().span(node_id), cx.tcx.hir().span(hir_id),
"`PartialEq` implemented here" "`PartialEq` implemented here"
); );
} }

14
clippy_lints/src/loops.rs
View File

@@ -1652,14 +1652,14 @@ fn check_for_mutability(cx: &LateContext<'_, '_>, bound: &Expr<'_>) -> Option<Hi
if let QPath::Resolved(None, _) = *qpath; if let QPath::Resolved(None, _) = *qpath;
then { then {
let res = qpath_res(cx, qpath, bound.hir_id); let res = qpath_res(cx, qpath, bound.hir_id);
if let Res::Local(node_id) = res { if let Res::Local(hir_id) = res {
let node_str = cx.tcx.hir().get(node_id); let node_str = cx.tcx.hir().get(hir_id);
if_chain! { if_chain! {
if let Node::Binding(pat) = node_str; if let Node::Binding(pat) = node_str;
if let PatKind::Binding(bind_ann, ..) = pat.kind; if let PatKind::Binding(bind_ann, ..) = pat.kind;
if let BindingAnnotation::Mutable = bind_ann; if let BindingAnnotation::Mutable = bind_ann;
then { then {
return Some(node_id); return Some(hir_id);
} }
} }
} }
@@ -2184,8 +2184,8 @@ impl<'a, 'tcx> Visitor<'tcx> for InitializeVisitor<'a, 'tcx> {
fn var_def_id(cx: &LateContext<'_, '_>, expr: &Expr<'_>) -> Option<HirId> { fn var_def_id(cx: &LateContext<'_, '_>, expr: &Expr<'_>) -> Option<HirId> {
if let ExprKind::Path(ref qpath) = expr.kind { if let ExprKind::Path(ref qpath) = expr.kind {
let path_res = qpath_res(cx, qpath, expr.hir_id); let path_res = qpath_res(cx, qpath, expr.hir_id);
if let Res::Local(node_id) = path_res { if let Res::Local(hir_id) = path_res {
return Some(node_id); return Some(hir_id);
} }
} }
None None
@@ -2422,8 +2422,8 @@ impl<'a, 'tcx> VarCollectorVisitor<'a, 'tcx> {
let res = qpath_res(self.cx, qpath, ex.hir_id); let res = qpath_res(self.cx, qpath, ex.hir_id);
then { then {
match res { match res {
Res::Local(node_id) => { Res::Local(hir_id) => {
self.ids.insert(node_id); self.ids.insert(hir_id);
}, },
Res::Def(DefKind::Static, def_id) => { Res::Def(DefKind::Static, def_id) => {
let mutable = self.cx.tcx.is_mutable_static(def_id); let mutable = self.cx.tcx.is_mutable_static(def_id);

3
clippy_lints/src/missing_inline.rs
View File

@@ -152,8 +152,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for MissingInline {
}; };
if let Some(trait_def_id) = trait_def_id { if let Some(trait_def_id) = trait_def_id {
if cx.tcx.hir().as_local_node_id(trait_def_id).is_some() && !cx.access_levels.is_exported(impl_item.hir_id) if cx.tcx.hir().as_local_hir_id(trait_def_id).is_some() && !cx.access_levels.is_exported(impl_item.hir_id) {
{
// If a trait is being implemented for an item, and the // If a trait is being implemented for an item, and the
// trait is not exported, we don't need #[inline] // trait is not exported, we don't need #[inline]
return; return;