From 3de6f9db202c612aad15397fb5b3320497dc2f25 Mon Sep 17 00:00:00 2001 From: Arnavion Date: Sat, 10 Dec 2016 23:57:19 -0800 Subject: [PATCH] Use inputs/output getters for rustc::ty::FnSig instead of the now non-existent fields. --- clippy_lints/src/cyclomatic_complexity.rs | 2 +- clippy_lints/src/eta_reduction.rs | 2 +- clippy_lints/src/eval_order_dependence.rs | 2 +- clippy_lints/src/len_zero.rs | 2 +- clippy_lints/src/mut_reference.rs | 2 +- clippy_lints/src/ptr.rs | 2 +- clippy_lints/src/utils/mod.rs | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/clippy_lints/src/cyclomatic_complexity.rs b/clippy_lints/src/cyclomatic_complexity.rs index 164633b7c64..fc9e25c7707 100644 --- a/clippy_lints/src/cyclomatic_complexity.rs +++ b/clippy_lints/src/cyclomatic_complexity.rs @@ -142,7 +142,7 @@ fn visit_expr(&mut self, e: &'tcx Expr) { let ty = self.cx.tcx.tables().node_id_to_type(callee.id); match ty.sty { ty::TyFnDef(_, _, ty) | - ty::TyFnPtr(ty) if ty.sig.skip_binder().output.sty == ty::TyNever => { + ty::TyFnPtr(ty) if ty.sig.skip_binder().output().sty == ty::TyNever => { self.divergence += 1; } _ => (), diff --git a/clippy_lints/src/eta_reduction.rs b/clippy_lints/src/eta_reduction.rs index 56308ed390d..2bf91f8d1bc 100644 --- a/clippy_lints/src/eta_reduction.rs +++ b/clippy_lints/src/eta_reduction.rs @@ -66,7 +66,7 @@ fn check_closure(cx: &LateContext, expr: &Expr) { ty::TyFnDef(_, _, fn_ty) | ty::TyFnPtr(fn_ty) => { if fn_ty.unsafety == Unsafety::Unsafe || - fn_ty.sig.skip_binder().output.sty == ty::TyNever { + fn_ty.sig.skip_binder().output().sty == ty::TyNever { return; } } diff --git a/clippy_lints/src/eval_order_dependence.rs b/clippy_lints/src/eval_order_dependence.rs index 9e9a900fe7a..88e1180067a 100644 --- a/clippy_lints/src/eval_order_dependence.rs +++ b/clippy_lints/src/eval_order_dependence.rs @@ -132,7 +132,7 @@ fn visit_expr(&mut self, e: &'tcx Expr) { ExprRet(_) => self.report_diverging_sub_expr(e), ExprCall(ref func, _) => match self.cx.tcx.tables().expr_ty(func).sty { ty::TyFnDef(_, _, fn_ty) | - ty::TyFnPtr(fn_ty) => if let ty::TyNever = self.cx.tcx.erase_late_bound_regions(&fn_ty.sig).output.sty { + ty::TyFnPtr(fn_ty) => if let ty::TyNever = self.cx.tcx.erase_late_bound_regions(&fn_ty.sig).output().sty { self.report_diverging_sub_expr(e); }, _ => {}, diff --git a/clippy_lints/src/len_zero.rs b/clippy_lints/src/len_zero.rs index fd58f86d42f..0242fb523ee 100644 --- a/clippy_lints/src/len_zero.rs +++ b/clippy_lints/src/len_zero.rs @@ -188,7 +188,7 @@ fn is_is_empty(cx: &LateContext, item: &ty::AssociatedItem) -> bool { if let ty::AssociatedKind::Method = item.kind { if &*item.name.as_str() == "is_empty" { let ty = cx.tcx.item_type(item.def_id).fn_sig().skip_binder(); - ty.inputs.len() == 1 + ty.inputs().len() == 1 } else { false } diff --git a/clippy_lints/src/mut_reference.rs b/clippy_lints/src/mut_reference.rs index c189ecba6de..d2cd0a32914 100644 --- a/clippy_lints/src/mut_reference.rs +++ b/clippy_lints/src/mut_reference.rs @@ -60,7 +60,7 @@ fn check_arguments(cx: &LateContext, arguments: &[Expr], type_definition: &TyS, match type_definition.sty { TypeVariants::TyFnDef(_, _, fn_type) | TypeVariants::TyFnPtr(fn_type) => { - let parameters = &fn_type.sig.skip_binder().inputs; + let parameters = fn_type.sig.skip_binder().inputs(); for (argument, parameter) in arguments.iter().zip(parameters.iter()) { match parameter.sty { TypeVariants::TyRef(_, TypeAndMut { mutbl: MutImmutable, .. }) | diff --git a/clippy_lints/src/ptr.rs b/clippy_lints/src/ptr.rs index f1fdc78aca4..1461189946c 100644 --- a/clippy_lints/src/ptr.rs +++ b/clippy_lints/src/ptr.rs @@ -94,7 +94,7 @@ fn check_fn(cx: &LateContext, decl: &FnDecl, fn_id: NodeId) { let fn_def_id = cx.tcx.map.local_def_id(fn_id); let fn_ty = cx.tcx.item_type(fn_def_id).fn_sig().skip_binder(); - for (arg, ty) in decl.inputs.iter().zip(&fn_ty.inputs) { + for (arg, ty) in decl.inputs.iter().zip(fn_ty.inputs()) { if let ty::TyRef(_, ty::TypeAndMut { ty, mutbl: MutImmutable }) = ty.sty { if match_type(cx, ty, &paths::VEC) { span_lint(cx, diff --git a/clippy_lints/src/utils/mod.rs b/clippy_lints/src/utils/mod.rs index dc64e56b2e6..d8afb96298d 100644 --- a/clippy_lints/src/utils/mod.rs +++ b/clippy_lints/src/utils/mod.rs @@ -751,7 +751,7 @@ pub fn return_ty<'a, 'tcx>(cx: &LateContext<'a, 'tcx>, fn_item: NodeId) -> ty::T let fn_def_id = cx.tcx.map.local_def_id(fn_item); let fn_sig = cx.tcx.item_type(fn_def_id).fn_sig(); let fn_sig = cx.tcx.liberate_late_bound_regions(parameter_env.free_id_outlive, fn_sig); - fn_sig.output + fn_sig.output() } /// Check if two types are the same. -- 2.44.0