From bc83a3a82fc14ac5f83dca972d7e48776d2b98d9 Mon Sep 17 00:00:00 2001
From: Bhavik Patel <bhavikpatel.aha@gmail.com>
Date: Sat, 27 Jun 2015 21:06:37 +0530
Subject: [PATCH] Fix AbbreviationAsWordInName Check in Checkstyle Code. #945

---
 config/checkstyle_checks.xml                                  | 2 +-
 config/suppressions.xml                                       | 4 ++++
 .../tools/checkstyle/checks/coding/RequireThisCheck.java      | 4 ++--
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/config/checkstyle_checks.xml b/config/checkstyle_checks.xml
index 5b94fc142..1d84d9ec7 100644
--- a/config/checkstyle_checks.xml
+++ b/config/checkstyle_checks.xml
@@ -261,9 +261,9 @@
     <module name="TodoComment"/>
     <module name="UncommentedMain"/>
     <module name="UnnecessaryParentheses"/>
+    <module name="AbbreviationAsWordInName"/>
 
 <!--
-    <module name="AbbreviationAsWordInName"/>
     <module name="AbstractClassName"/>
     <module name="AvoidEscapedUnicodeCharacters"/>
     <module name="AvoidInlineConditionals"/>
diff --git a/config/suppressions.xml b/config/suppressions.xml
index d4340c904..2e5671e9a 100644
--- a/config/suppressions.xml
+++ b/config/suppressions.xml
@@ -33,6 +33,10 @@
     <suppress checks="ThrowsCount"
               files="Main.java"
               lines="63"/>
+    <!-- we can not change it as, Check name is part of API (used in configurations) -->
+    <suppress checks="AbbreviationAsWordInName"
+              files="JavaNCSSCheck.java"
+              lines="40"/>
 
     <!-- Tone down the checking for test code -->
     <suppress checks="EmptyBlock" files=".*[\\/]src[\\/]test[\\/]"/>
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/RequireThisCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/RequireThisCheck.java
index 3a6afe337..9dfc6c940 100644
--- a/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/RequireThisCheck.java
+++ b/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/RequireThisCheck.java
@@ -120,7 +120,7 @@ public class RequireThisCheck extends AbstractDeclarationCollector {
     public void visitToken(DetailAST ast) {
         super.visitToken(ast);
         if (ast.getType() == TokenTypes.IDENT) {
-            processIDENT(ast);
+            processIdent(ast);
         }
     }
 
@@ -130,7 +130,7 @@ public class RequireThisCheck extends AbstractDeclarationCollector {
      *
      * @param ast IDENT to check.
      */
-    private void processIDENT(DetailAST ast) {
+    private void processIdent(DetailAST ast) {
         final int parentType = ast.getParent().getType();
         switch (parentType) {
             case TokenTypes.ANNOTATION_MEMBER_VALUE_PAIR: