squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Move in_macro check to end of expression since usual case is not inside macro

Browse source
This commit is contained in:
Darren Tsung 2017-12-22 10:54:52 -08:00
parent 1f36aa519e
commit b9abe028c9
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
clippy_lints/src/eq_op.rs
View file

@ -53,7 +53,7 @@ impl LintPass for EqOp {
impl<'a, 'tcx> LateLintPass<'a, 'tcx> for EqOp {
fn check_expr(&mut self, cx: &LateContext<'a, 'tcx>, e: &'tcx Expr) {
if let ExprBinary(ref op, ref left, ref right) = e.node {
if !in_macro(e.span) && is_valid_operator(op) && SpanlessEq::new(cx).ignore_fn().eq_expr(left, right) {
if is_valid_operator(op) && SpanlessEq::new(cx).ignore_fn().eq_expr(left, right) && !in_macro(e.span) {
span_lint(
cx,
EQ_OP,