From bd4ec9a98edaced73f14c5422f64c39fad3ce596 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lars=20K=C3=BChne?= Date: Sun, 4 Jul 2004 11:24:32 +0000 Subject: [PATCH] correct source location info in case whitespace between unary op and magic number --- .../checkstyle/checks/coding/MagicNumberCheck.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/MagicNumberCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/MagicNumberCheck.java index 5d7f78a1b..4e2c50791 100644 --- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/MagicNumberCheck.java +++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/MagicNumberCheck.java @@ -63,18 +63,18 @@ public class MagicNumberCheck extends Check { if (!inIgnoreList(aAST) && !isConstantDefinition(aAST)) { String text = aAST.getText(); - int columnNo = aAST.getColumnNo(); final DetailAST parent = aAST.getParent(); + DetailAST reportAST = aAST; if (parent.getType() == TokenTypes.UNARY_MINUS) { - columnNo--; + reportAST = parent; text = "-" + text; } else if (parent.getType() == TokenTypes.UNARY_PLUS) { - columnNo--; + reportAST = parent; text = "+" + text; } - log(aAST.getLineNo(), - columnNo, + log(reportAST.getLineNo(), + reportAST.getColumnNo(), "magic.number", text); }