Changeset 12586 in josm for trunk/tools/checkstyle/src/org

Timestamp:

2017-08-09T21:54:38+02:00 (7 years ago)

Author:

Don-vip

Message:

see #14794 - update to checkstyle 8.0 (to match the version used by Eclipse CS) + add new project to build custom Eclipse plugin (now required)

Location:

trunk/tools/checkstyle

Files:

• . (modified) (1 prop)
• src (moved) (moved from trunk/tools/checkstyle/checks )
• src/org/openstreetmap/josm/TopLevelJavadocCheck.java (modified) (2 diffs)
• src/org/openstreetmap/josm/checkstyle-metadata.properties (added)
• src/org/openstreetmap/josm/checkstyle-metadata.xml (added)
• src/org/openstreetmap/josm/messages.properties (added)

trunk/tools/checkstyle/src/org/openstreetmap/josm/TopLevelJavadocCheck.java

@@ -13 +13 @@
  * Checks that there is Javadoc for every top level class, interface or enum.
  */
-public class TopLevelJavadocCheck extends AbstractCheck
-{
+public class TopLevelJavadocCheck extends AbstractCheck {
+
     private boolean foundTopLevelClass;
 
     @Override
-    public int[] getDefaultTokens()
-    {
+    public int[] getAcceptableTokens() {
+        return getDefaultTokens();
+    }
+
+    @Override
+    public int[] getDefaultTokens() {
         return new int[]{TokenTypes.CLASS_DEF, TokenTypes.INTERFACE_DEF, TokenTypes.ENUM_DEF};
+    }
+
+    @Override
+    public int[] getRequiredTokens() {
+        return new int[0];
     }
 
@@ -75 +84 @@
 
     @Override
-    public void visitToken(DetailAST ast)
-    {
+    public void visitToken(DetailAST ast) {
         DetailAST parent = ast.getParent();
         if (parent == null || parent.getType() == TokenTypes.EOF) {