diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullCheck.java index 4d1489004..331b9f2b4 100755 --- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullCheck.java +++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullCheck.java @@ -88,12 +88,7 @@ import com.puppycrawl.tools.checkstyle.api.TokenTypes; */ public class EqualsAvoidNullCheck extends Check { - // Decides whether string literals with optional assignment - // assignment should be checked to be on the left side of the - // equalsIgnoreCase comparison. The earlier version of this class - // checked only for equals() comparison. This is added to support - // cases which may not want the equalsIgnoreCase() comparison for - // some reason (backward compatibility) + /** Whether to process equalsIgnoreCase() invocations. */ private boolean mIgnoreEqualsIgnoreCase; @Override @@ -236,10 +231,11 @@ public class EqualsAvoidNullCheck extends Check /** * Whether to ignore checking {@code String.equalsIgnoreCase(String)}. - * @param newValue whether to ignore checking + * @param aNewValue whether to ignore checking * {@code String.equalsIgnoreCase(String)}. */ - public void setIgnoreEqualsIgnoreCase(boolean newValue) { - mIgnoreEqualsIgnoreCase = newValue; + public void setIgnoreEqualsIgnoreCase(boolean aNewValue) + { + mIgnoreEqualsIgnoreCase = aNewValue; } } diff --git a/src/tests/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullTest.java b/src/tests/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullTest.java index ed1bb6c7d..38e677255 100755 --- a/src/tests/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullTest.java +++ b/src/tests/com/puppycrawl/tools/checkstyle/checks/coding/EqualsAvoidNullTest.java @@ -55,7 +55,7 @@ public class EqualsAvoidNullTest extends BaseCheckTestSupport "73:27: String literal expressions should be on the left side of an equalsIgnoreCase comparison.", "75:27: String literal expressions should be on the left side of an equalsIgnoreCase comparison.", "77:27: String literal expressions should be on the left side of an equalsIgnoreCase comparison.", - "79:27: String literal expressions should be on the left side of an equalsIgnoreCase comparison." + "79:27: String literal expressions should be on the left side of an equalsIgnoreCase comparison.", }; verify(checkConfig, getPath("coding" + File.separator + "InputEqualsAvoidNull.java"), expected); }