Upgrade checkstyle and add new rules

Signed-off-by: Gino Augustine <[email protected]>

Author: ginoaugustine

dev/checkstyle/style.xml

@@ -25,7 +25,14 @@
     </module>
 
     <module name="TreeWalker">
+        <module name="InvalidJavadocPosition"/>
         <module name="JavadocStyle"/>
+        <module name="AtclauseOrder"/>
+        <module name="MissingJavadocType">
+            <property name="tokens"
+                      value="INTERFACE_DEF, ENUM_DEF,
+                      RECORD_DEF, ANNOTATION_DEF"/>
+        </module>
         <module name="AvoidStarImport"/>
         <module name="RedundantImport"/>
         <module name="UnusedImports"/>

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/PathTokenizer.java

@@ -92,13 +92,13 @@ public final boolean incrementToken() throws IOException {
             if (i >= buf.length) {
                 buf = Arrays.copyOf(buf, buf.length * 2);
             }
-            /**
-             * "In general, String.toLowerCase(Locale) should be used to map
-             * characters to lowercase. String case mapping methods have several
-             * benefits over Character case mapping methods. String case mapping
-             * methods can perform locale-sensitive mappings, context-sensitive
-             * mappings, and 1:M character mappings, whereas the Character case
-             * mapping methods cannot." See below.
+            /*
+              "In general, String.toLowerCase(Locale) should be used to map
+              characters to lowercase. String case mapping methods have several
+              benefits over Character case mapping methods. String case mapping
+              methods can perform locale-sensitive mappings, context-sensitive
+              mappings, and 1:M character mappings, whereas the Character case
+              mapping methods cannot." See below.
              */
             buf[i++] = (char) c;
             c = input.read();

opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/uue/UuencodeAnalyzerFactory.java

@@ -35,11 +35,11 @@ public class UuencodeAnalyzerFactory extends FileAnalyzerFactory {
     private static final String NAME = "UUEncoded";
 
     private static final String[] SUFFIXES = {
-       /**
-         * XXX:
-         * FreeBSD and DragonFly .fnt files are uuencoded;
-         * Minix3 .fnt files are binary. -- 2013-04 cnst
-         */
+            /*
+               XXX:
+               FreeBSD and DragonFly .fnt files are uuencoded;
+               Minix3 .fnt files are binary. -- 2013-04 cnst
+              */
         "UU", "UUE", /*"FNT",*/ "BASE64"
     };
 

opengrok-indexer/src/main/java/org/opengrok/indexer/authorization/AuthorizationEntity.java

@@ -420,13 +420,13 @@ protected void processTargetGroupsAndProjects() {
         Set<String> groups = new TreeSet<>();
 
         for (String x : forGroups()) {
-            /**
-             * Full group discovery takes place here. All projects/repositories
-             * in the group are added into "forProjects" and all subgroups
-             * (including projects/repositories) and parent groups (excluding
-             * the projects/repositories) are added into "forGroups".
-             *
-             * If the group does not exist then a warning is issued.
+            /*
+              Full group discovery takes place here. All projects/repositories
+              in the group are added into "forProjects" and all subgroups
+              (including projects/repositories) and parent groups (excluding
+              the projects/repositories) are added into "forGroups".
+
+              If the group does not exist then a warning is issued.
              */
             Group g;
             if ((g = Group.getByName(x)) != null) {
@@ -442,9 +442,9 @@ protected void processTargetGroupsAndProjects() {
         setForGroups(groups);
 
         forProjects().removeIf(t -> {
-            /**
-             * Check the existence of the projects and issue a warning if there
-             * is no such project.
+            /*
+              Check the existence of the projects and issue a warning if there
+              is no such project.
              */
             if ( Project.getByName(t) == null) {
                 LOGGER.log(Level.WARNING, "Configured project \"{0}\" in forProjects"
@@ -474,8 +474,7 @@ public synchronized void setWorking() {
 
     /**
      * Check if this plugin has failed during loading or is missing.
-     *
-     * This method has the same effect as !{@link isWorking()}.
+     * This method has the same effect as !{@link #isWorking()}.
      *
      * @return true if failed, true otherwise
      * @see #isWorking()

opengrok-indexer/src/main/java/org/opengrok/indexer/authorization/AuthorizationPlugin.java

@@ -171,12 +171,12 @@ public synchronized void unload() {
     public boolean isAllowed(Nameable entity,
             AuthorizationEntity.PluginDecisionPredicate pluginPredicate,
             AuthorizationEntity.PluginSkippingPredicate skippingPredicate) {
-        /**
-         * We don't check the skippingPredicate here as this instance is
-         * <b>always</b> a part of some stack (may be the default stack) and the
-         * stack checks the skipping predicate before invoking this method.
-         *
-         * @see AuthorizationStack#processStack
+        /*
+          We don't check the skippingPredicate here as this instance is
+          <b>always</b> a part of some stack (may be the default stack) and the
+          stack checks the skipping predicate before invoking this method.
+
+          @see AuthorizationStack#processStack
          */
 
         if (isFailed()) {
@@ -205,14 +205,14 @@ public boolean setPlugin(IAuthorizationPlugin plugin) {
             unload();
         }
         try {
-            /**
-             * The exception should not happen here as we already have an
-             * instance of IAuthorizationPlugin. But it is required by the
-             * compiler.
-             *
-             * NOTE: If we were to add a throws clause here we would interrupt
-             * the whole stack walk through and prevent the other authorization
-             * entities to work properly.
+            /*
+              The exception should not happen here as we already have an
+              instance of IAuthorizationPlugin. But it is required by the
+              compiler.
+
+              NOTE: If we were to add a throws clause here we would interrupt
+              the whole stack walk through and prevent the other authorization
+              entities to work properly.
              */
             this.plugin = plugin.getClass().getDeclaredConstructor().newInstance();
             return true;

opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/CommandTimeoutType.java

@@ -22,6 +22,11 @@
  */
 package org.opengrok.indexer.configuration;
 
+/**
+ * Command Time Out Type.
+ * Different time out config valid are present for each
+ * command type.
+ */
 public enum CommandTimeoutType {
     INTERACTIVE,
     INDEXER,

opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/Configuration.java

@@ -642,19 +642,18 @@ public Map<String, String> getCmds() {
     /**
      * @return int the current message limit
      * @see org.opengrok.indexer.web.messages.MessagesContainer
-     *
-     * @return int the current message limit
      */
     public int getMessageLimit() {
         return messageLimit;
     }
 
     /**
-     * @see org.opengrok.indexer.web.messages.MessagesContainer
      *
      * @param messageLimit the limit
      * @throws IllegalArgumentException when the limit is negative
+     * @see org.opengrok.indexer.web.messages.MessagesContainer
      */
+
     public void setMessageLimit(int messageLimit) throws IllegalArgumentException {
         if (messageLimit < 0) {
             throw new IllegalArgumentException(
@@ -940,12 +939,10 @@ public String getDataRoot() {
 
     /**
      * Sets data root.
-     *
      * This method also sets the pluginDirectory if it is not already set.
      *
-     * @see #setPluginDirectory(java.lang.String)
-     *
      * @param dataRoot data root path
+     * @see #setPluginDirectory(java.lang.String)
      */
     public void setDataRoot(String dataRoot) {
         if (dataRoot != null && getPluginDirectory() == null) {

opengrok-indexer/src/main/java/org/opengrok/indexer/history/AnnotationCache.java

@@ -26,6 +26,9 @@
 
 import java.io.File;
 
+/**
+ * Annotation Cache Related Operations.
+ */
 public interface AnnotationCache extends Cache {
     /**
      * Retrieve annotation from cache.

opengrok-indexer/src/main/java/org/opengrok/indexer/history/Cache.java

@@ -29,6 +29,9 @@
 import java.util.List;
 import java.util.logging.Logger;
 
+/**
+ * Cache interface for common cache related operations.
+ */
 public interface Cache {
 
     Logger LOGGER = LoggerFactory.getLogger(Cache.class);

opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryGuru.java

@@ -783,14 +783,14 @@ public boolean fillLastHistoryEntries(File directory, List<DirectoryEntry> entri
 
     /**
      * Recursively search for repositories with a depth limit, add those found to the internally used map.
-     * @see #putRepository(Repository)
      *
      * @param files list of directories to check if they contain a repository
      * @param allowedNesting number of levels of nested repos to allow
      * @param depth maximum scanning depth
      * @param isNested a value indicating if a parent {@link Repository} was already found above the {@code files}
      * @param progress {@link org.opengrok.indexer.util.Progress} instance
      * @return collection of added repositories
+     * @see #putRepository(Repository)
      */
     private Collection<RepositoryInfo> addRepositories(File[] files, int allowedNesting, int depth, boolean isNested,
                                                        Progress progress) {