Make it compile

Author: Manishearth

clippy_lints/src/functions.rs

@@ -3,7 +3,7 @@ use std::convert::TryFrom;
 use crate::utils::{iter_input_pats, snippet, snippet_opt, span_lint, type_is_unsafe_function};
 use matches::matches;
 use rustc::hir;
-use rustc::hir::def::{Res, DefKind};
+use rustc::hir::def::Res;
 use rustc::hir::intravisit;
 use rustc::lint::{in_external_macro, LateContext, LateLintPass, LintArray, LintContext, LintPass};
 use rustc::ty;

clippy_lints/src/let_if_seq.rs

@@ -1,7 +1,7 @@
 use crate::utils::{snippet, span_lint_and_then};
 use if_chain::if_chain;
 use rustc::hir;
-use rustc::hir::def::{Res, DefKind};
+use rustc::hir::def::Res;
 use rustc::hir::BindingAnnotation;
 use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
 use rustc::{declare_lint_pass, declare_tool_lint};

clippy_lints/src/loops.rs

@@ -1774,7 +1774,7 @@ impl<'a, 'tcx> VarVisitor<'a, 'tcx> {
  }
  let res = self.cx.tables.qpath_res(seqpath, seqexpr.hir_id);
  match res {
- Res::Local(hir_id) | Res::Def(Res::Upvar(..), hir_id, ..) => {
+ Res::Local(hir_id) | Res::Upvar(hir_id, ..) => {
  let parent_id = self.cx.tcx.hir().get_parent_item(expr.hir_id);
  let parent_def_id = self.cx.tcx.hir().local_def_id_from_hir_id(parent_id);
  let extent = self.cx.tcx.region_scope_tree(parent_def_id).var_scope(hir_id.local_id);
@@ -1835,7 +1835,7 @@ impl<'a, 'tcx> Visitor<'tcx> for VarVisitor<'a, 'tcx> {
  if path.segments.len() == 1;
  then {
  match self.cx.tables.qpath_res(qpath, expr.hir_id) {
- Res::Def(Res::Upvar(..), local_id, ..) => {
+ Res::Upvar(local_id, ..) => {
  if local_id == self.var {
  // we are not indexing anything, record that
  self.nonindex = true;
@@ -2383,7 +2383,7 @@ impl<'a, 'tcx> VarCollectorVisitor<'a, 'tcx> {
  let res = self.cx.tables.qpath_res(qpath, ex.hir_id);
  then {
  match res {
- Res::Local(node_id) | Res::Def(Res::Upvar(..), node_id, ..) => {
+ Res::Local(node_id) | Res::Upvar(node_id, ..) => {
  self.ids.insert(node_id);
  },
  Res::Def(DefKind::Static, def_id) => {

clippy_lints/src/methods/unnecessary_filter_map.rs

@@ -2,7 +2,7 @@ use crate::utils::paths;
 use crate::utils::usage::mutated_variables;
 use crate::utils::{match_qpath, match_trait_method, span_lint};
 use rustc::hir;
-use rustc::hir::def::{Res, DefKind};
+use rustc::hir::def::Res;
 use rustc::hir::intravisit::{walk_expr, NestedVisitorMap, Visitor};
 use rustc::lint::LateContext;
 

clippy_lints/src/misc.rs

@@ -2,7 +2,6 @@ use if_chain::if_chain;
 use matches::matches;
 use rustc::hir::intravisit::FnKind;
 use rustc::hir::*;
-use rustc::hir::def::{Res, DefKind, CtorOf};
 use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
 use rustc::ty;
 use rustc::{declare_lint_pass, declare_tool_lint};
@@ -603,7 +602,7 @@ fn in_attributes_expansion(expr: &Expr) -> bool {
 /// Tests whether `res` is a variable defined outside a macro.
 fn non_macro_local(cx: &LateContext<'_, '_>, res: def::Res) -> bool {
  match res {
- def::Res::Local(id) | def::Res(Res::Upvar(..), id, _, _) => !in_macro(cx.tcx.hir().span_by_hir_id(id)),
+ def::Res::Local(id) | def::Res::Upvar(id, ..) => !in_macro(cx.tcx.hir().span_by_hir_id(id)),
  _ => false,
  }
 }

clippy_lints/src/question_mark.rs

@@ -117,7 +117,7 @@ impl QuestionMark {
  },
  ExprKind::Ret(Some(ref expr)) => Self::expression_returns_none(cx, expr),
  ExprKind::Path(ref qp) => {
- if let Res::Def(DefKind::Ctor(_, def::CtorOf::Variant), def_id) = cx.tables.qpath_res(qp, expression.hir_id) {
+ if let Res::Def(DefKind::Ctor(def::CtorOf::Variant, def::CtorKind::Const), def_id) = cx.tables.qpath_res(qp, expression.hir_id) {
  return cx.match_def_path(def_id, &OPTION_NONE);
  }
 

clippy_lints/src/use_self.rs

@@ -223,7 +223,7 @@ impl<'a, 'tcx> Visitor<'tcx> for UseSelfVisitor<'a, 'tcx> {
  if path.segments.last().expect(SEGMENTS_MSG).ident.name != SelfUpper.name() {
  if self.item_path.res == path.res {
  span_use_self_lint(self.cx, path);
- } else if let Res::Def(DefKind::Ctor(def::CtorOf::Struct, ctor_did), CtorKind::Fn) = path.res {
+ } else if let Res::Def(DefKind::Ctor(def::CtorOf::Struct, CtorKind::Fn), ctor_did) = path.res {
  if self.item_path.res.opt_def_id() == self.cx.tcx.parent(ctor_did) {
  span_use_self_lint(self.cx, path);
  }

clippy_lints/src/utils/usage.rs

@@ -1,4 +1,4 @@
-use rustc::hir::def::{Res, DefKind};
+use rustc::hir::def::Res;
 use rustc::hir::*;
 use rustc::lint::LateContext;
 use rustc::middle::expr_use_visitor::*;
@@ -30,7 +30,7 @@ pub fn is_potentially_mutated<'a, 'tcx: 'a>(
  cx: &'a LateContext<'a, 'tcx>,
 ) -> bool {
  let id = match variable.res {
- Res::Local(id) | Res::Def(Res::Upvar(..), id, ..) => id,
+ Res::Local(id) | Res::Upvar(id, ..) => id,
  _ => return true,
  };
  mutated_variables(expr, cx).map_or(true, |mutated| mutated.contains(&id))