Issue #464: Resolve code style problems: javadoc, return statements, …

…trailing comments

sevntu-checks/src/main/java/com/github/sevntu/checkstyle/checks/coding/EnumTrailingCommaCheck.java

@@ -131,17 +131,30 @@ public void visitToken(DetailAST enumDef) {
         }
     }
 
+    /**
+     * Check whether there is an illegal token after the last comma token.
+     *
+     * @param lastComma the AST containing the last comma
+     * @param nextAst the next AST
+     * @return {@code true} if there is an illegal token after the last comma,
+     * {@code false otherwise}
+     */
     private boolean isIllegalTokenAfterComma(DetailAST lastComma, DetailAST nextAst) {
+        boolean illegalToken = false;
+
         final int nextAstType = nextAst.getType();
 
         if (nextAstType == TokenTypes.SEMI) {
             if (nextAst.getLineNo() == lastComma.getLineNo()) {
                 // semi on the same line as last comma
-                return true;
+                illegalToken = true;
             }
         }
-        // next enum constant on the same line after comma
-        return nextAstType == TokenTypes.ENUM_CONSTANT_DEF;
+        else if (nextAstType == TokenTypes.ENUM_CONSTANT_DEF) {
+            // next enum constant on the same line after comma
+            illegalToken = true;
+        }
+        return illegalToken;
     }
 
     /**

sevntu-checks/src/test/java/com/github/sevntu/checkstyle/internal/CheckstyleRegressionTest.java

@@ -37,7 +37,9 @@ public class CheckstyleRegressionTest {
     /** List of checks to suppress if we dynamically add it to the configuration. */
     private static final List<String> ADD_CHECK_SUPPRESSIONS = Arrays
             .asList(
-                    "EnumTrailingCommaCheck", // conflicting with NoWhitespaceBefore, SeparatorWrap
+                    // conflicting with NoWhitespaceBefore, SeparatorWrap
+                    "EnumTrailingCommaCheck",
+
                     "ReturnCountExtendedCheck"
             );