-use rustc::hir::{Crate, Expr, ExprKind, QPath};
-use rustc::impl_lint_pass;
-use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
-use rustc_session::declare_tool_lint;
-use syntax::ast::AttrKind;
-use syntax::symbol::sym;
+use rustc_hir::{Crate, Expr, ExprKind, QPath};
+use rustc_lint::{LateContext, LateLintPass};
+use rustc_session::{declare_tool_lint, impl_lint_pass};
-use crate::utils::{is_entrypoint_fn, snippet, span_help_and_lint};
+use crate::utils::{is_entrypoint_fn, is_no_std_crate, snippet, span_lint_and_help};
use if_chain::if_chain;
declare_clippy_lint! {
impl_lint_pass!(MainRecursion => [MAIN_RECURSION]);
-impl LateLintPass<'_, '_> for MainRecursion {
- fn check_crate(&mut self, _: &LateContext<'_, '_>, krate: &Crate) {
- self.has_no_std_attr = krate.attrs.iter().any(|attr| {
- if let AttrKind::Normal(ref attr) = attr.kind {
- attr.path == sym::no_std
- } else {
- false
- }
- });
+impl LateLintPass<'_> for MainRecursion {
+ fn check_crate(&mut self, cx: &LateContext<'_>, _: &Crate<'_>) {
+ self.has_no_std_attr = is_no_std_crate(cx);
}
- fn check_expr_post(&mut self, cx: &LateContext<'_, '_>, expr: &Expr) {
+ fn check_expr_post(&mut self, cx: &LateContext<'_>, expr: &Expr<'_>) {
if self.has_no_std_attr {
return;
}
if_chain! {
if let ExprKind::Call(func, _) = &expr.kind;
- if let ExprKind::Path(path) = &func.kind;
- if let QPath::Resolved(_, path) = &path;
+ if let ExprKind::Path(QPath::Resolved(_, path)) = &func.kind;
if let Some(def_id) = path.res.opt_def_id();
if is_entrypoint_fn(cx, def_id);
then {
- span_help_and_lint(
+ span_lint_and_help(
cx,
MAIN_RECURSION,
func.span,
&format!("recursing into entrypoint `{}`", snippet(cx, func.span, "main")),
+ None,
"consider using another function for this recursion"
)
}