BUG-4109: Correct checkstyle dependency and avoid a couple of NPEs

[yangtools.git] / common / checkstyle-logging / src / main / java / org / opendaylight / yangtools / checkstyle / LoggerVariableNameCheck.java

diff --git a/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerVariableNameCheck.java b/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerVariableNameCheck.java
index 615f9148c03843cc50fe4df3718085bec0723d3f..6e69da05880c0762b7ce52a185db85bca624571b 100644 (file)
--- a/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerVariableNameCheck.java
+++ b/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerVariableNameCheck.java
@@ -25,8 +25,8 @@ public class LoggerVariableNameCheck extends Check {
 
     @Override
     public void visitToken(DetailAST aAST) {
-        final String variableName = CheckLoggingUtil.getVariableName(aAST);
-        if (CheckLoggingUtil.isAFieldVariable(aAST) && CheckLoggingUtil.isLoggerType(aAST) && !variableName.equals(LOGGER_VAR_NAME)) {
+        if (CheckLoggingUtil.isAFieldVariable(aAST) && CheckLoggingUtil.isLoggerType(aAST)
+                && !LOGGER_VAR_NAME.equals(CheckLoggingUtil.getVariableName(aAST))) {
             log(aAST.getLineNo(), LOG_MESSAGE);
         }
     }