2016-04-30 02:01:47 +00:00
|
|
|
use rustc::hir;
|
|
|
|
use rustc::lint::*;
|
|
|
|
use syntax::codemap::Span;
|
|
|
|
use utils::span_lint;
|
|
|
|
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **What it does:** Checks for plain integer arithmetic.
|
2016-04-30 02:01:47 +00:00
|
|
|
///
|
|
|
|
/// **Why is this bad?** This is only checked against overflow in debug builds.
|
|
|
|
/// In some applications one wants explicitly checked, wrapping or saturating
|
|
|
|
/// arithmetic.
|
|
|
|
///
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **Known problems:** None.
|
2016-04-30 02:01:47 +00:00
|
|
|
///
|
|
|
|
/// **Example:**
|
2016-07-15 22:25:44 +00:00
|
|
|
/// ```rust
|
2016-04-30 02:01:47 +00:00
|
|
|
/// a + 1
|
|
|
|
/// ```
|
2016-04-30 21:54:10 +00:00
|
|
|
declare_restriction_lint! {
|
2016-04-30 02:01:47 +00:00
|
|
|
pub INTEGER_ARITHMETIC,
|
2016-08-06 08:18:36 +00:00
|
|
|
"any integer arithmetic statement"
|
2016-04-30 02:01:47 +00:00
|
|
|
}
|
|
|
|
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **What it does:** Checks for float arithmetic.
|
2016-04-30 02:01:47 +00:00
|
|
|
///
|
|
|
|
/// **Why is this bad?** For some embedded systems or kernel development, it
|
2016-08-06 07:55:04 +00:00
|
|
|
/// can be useful to rule out floating-point numbers.
|
2016-04-30 02:01:47 +00:00
|
|
|
///
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **Known problems:** None.
|
2016-04-30 02:01:47 +00:00
|
|
|
///
|
|
|
|
/// **Example:**
|
2016-07-15 22:25:44 +00:00
|
|
|
/// ```rust
|
2016-04-30 02:01:47 +00:00
|
|
|
/// a + 1.0
|
|
|
|
/// ```
|
2016-04-30 21:54:10 +00:00
|
|
|
declare_restriction_lint! {
|
2016-04-30 02:01:47 +00:00
|
|
|
pub FLOAT_ARITHMETIC,
|
2016-08-06 08:18:36 +00:00
|
|
|
"any floating-point arithmetic statement"
|
2016-04-30 02:01:47 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Copy, Clone, Default)]
|
|
|
|
pub struct Arithmetic {
|
2016-06-05 23:42:39 +00:00
|
|
|
span: Option<Span>,
|
2016-04-30 02:01:47 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
impl LintPass for Arithmetic {
|
|
|
|
fn get_lints(&self) -> LintArray {
|
|
|
|
lint_array!(INTEGER_ARITHMETIC, FLOAT_ARITHMETIC)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-12-07 12:13:40 +00:00
|
|
|
impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Arithmetic {
|
|
|
|
fn check_expr(&mut self, cx: &LateContext<'a, 'tcx>, expr: &'tcx hir::Expr) {
|
2016-10-29 16:56:12 +00:00
|
|
|
if self.span.is_some() {
|
2016-06-05 23:42:39 +00:00
|
|
|
return;
|
|
|
|
}
|
2016-04-30 02:01:47 +00:00
|
|
|
match expr.node {
|
|
|
|
hir::ExprBinary(ref op, ref l, ref r) => {
|
|
|
|
match op.node {
|
2016-06-05 23:42:39 +00:00
|
|
|
hir::BiAnd | hir::BiOr | hir::BiBitAnd | hir::BiBitOr | hir::BiBitXor | hir::BiShl |
|
|
|
|
hir::BiShr | hir::BiEq | hir::BiLt | hir::BiLe | hir::BiNe | hir::BiGe | hir::BiGt => return,
|
|
|
|
_ => (),
|
2016-04-30 02:01:47 +00:00
|
|
|
}
|
2016-11-16 20:57:56 +00:00
|
|
|
let (l_ty, r_ty) = (cx.tcx.tables().expr_ty(l), cx.tcx.tables().expr_ty(r));
|
2016-04-30 02:01:47 +00:00
|
|
|
if l_ty.is_integral() && r_ty.is_integral() {
|
2016-06-05 23:42:39 +00:00
|
|
|
span_lint(cx, INTEGER_ARITHMETIC, expr.span, "integer arithmetic detected");
|
2016-04-30 21:54:10 +00:00
|
|
|
self.span = Some(expr.span);
|
2016-04-30 02:01:47 +00:00
|
|
|
} else if l_ty.is_floating_point() && r_ty.is_floating_point() {
|
2016-06-05 23:42:39 +00:00
|
|
|
span_lint(cx, FLOAT_ARITHMETIC, expr.span, "floating-point arithmetic detected");
|
2016-04-30 21:54:10 +00:00
|
|
|
self.span = Some(expr.span);
|
2016-04-30 02:01:47 +00:00
|
|
|
}
|
2016-06-05 23:42:39 +00:00
|
|
|
}
|
2016-04-30 02:01:47 +00:00
|
|
|
hir::ExprUnary(hir::UnOp::UnNeg, ref arg) => {
|
2016-11-16 20:57:56 +00:00
|
|
|
let ty = cx.tcx.tables().expr_ty(arg);
|
2016-04-30 02:01:47 +00:00
|
|
|
if ty.is_integral() {
|
2016-06-05 23:42:39 +00:00
|
|
|
span_lint(cx, INTEGER_ARITHMETIC, expr.span, "integer arithmetic detected");
|
2016-04-30 02:01:47 +00:00
|
|
|
self.span = Some(expr.span);
|
|
|
|
} else if ty.is_floating_point() {
|
2016-06-05 23:42:39 +00:00
|
|
|
span_lint(cx, FLOAT_ARITHMETIC, expr.span, "floating-point arithmetic detected");
|
2016-04-30 02:01:47 +00:00
|
|
|
self.span = Some(expr.span);
|
|
|
|
}
|
2016-06-05 23:42:39 +00:00
|
|
|
}
|
|
|
|
_ => (),
|
2016-04-30 02:01:47 +00:00
|
|
|
}
|
|
|
|
}
|
2016-04-30 21:54:10 +00:00
|
|
|
|
2016-12-07 12:13:40 +00:00
|
|
|
fn check_expr_post(&mut self, _: &LateContext<'a, 'tcx>, expr: &'tcx hir::Expr) {
|
2016-04-30 02:01:47 +00:00
|
|
|
if Some(expr.span) == self.span {
|
|
|
|
self.span = None;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|