Require empty line before tags

Author: shadromani

checkstyle.xml

@@ -320,12 +320,12 @@
                       value="^@return the *|^This method returns |^A [{]@code [a-zA-Z0-9]+[}]( is a )"/>
         </module> -->
         <!-- <module name="JavadocParagraph"/>
-        <module name="RequireEmptyLineBeforeBlockTagGroup"/>
         <module name="AtclauseOrder">
             <property name="tagOrder" value="@param, @return, @throws, @deprecated"/>
             <property name="target"
                       value="CLASS_DEF, INTERFACE_DEF, ENUM_DEF, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"/>
         </module> -->
+        <module name="RequireEmptyLineBeforeBlockTagGroup"/>
         <module name="JavadocMethod">
             <property name="accessModifiers" value="public"/>
             <property name="allowMissingParamTags" value="true"/>

src/main/java/com/maxmind/db/CacheKey.java

@@ -4,7 +4,7 @@
  * CacheKey is used as a key in the node cache. It contains the offset of the
  * node in the database file, the class of the data at that node, and the type
  * of the data at that node.
- * 
+ *
  * @param <T> the type of data stored at the node
  */
 public final class CacheKey<T> {

src/main/java/com/maxmind/db/Metadata.java

@@ -32,7 +32,7 @@ public final class Metadata {
 
     /**
      * Constructs a Metadata object.
-     * 
+     *
      * @param binaryFormatMajorVersion The major version number for the database's
      *                                 binary format.
      * @param binaryFormatMinorVersion The minor version number for the database's

src/main/java/com/maxmind/db/Network.java

@@ -60,7 +60,7 @@ public int getPrefixLength() {
         return prefixLength;
     }
 
-    /***
+    /**
      * @return A string representation of the network in CIDR notation, e.g.,
      * <code>1.2.3.0/24</code> or <code>2001::/8</code>.
      */

src/main/java/com/maxmind/db/Networks.java

@@ -11,7 +11,7 @@
 /**
  * Instances of this class provide an iterator over the networks in a database.
  * The iterator will return a {@link DatabaseRecord} for each network.
- * 
+ *
  * @param <T> The type of data returned by the iterator.
  */
 public final class Networks<T> implements Iterator<DatabaseRecord<T>> {
@@ -24,6 +24,7 @@ public final class Networks<T> implements Iterator<DatabaseRecord<T>> {
 
     /**
      * Constructs a Networks instance.
+     *
      * @param reader The reader object.
      * @param includeAliasedNetworks The boolean to include aliased networks.
      * @param typeParameterClass The type of data returned by the iterator.
@@ -36,6 +37,7 @@ public final class Networks<T> implements Iterator<DatabaseRecord<T>> {
 
     /**
      * Constructs a Networks instance.
+     *
      * @param reader The reader object.
      * @param includeAliasedNetworks The boolean to include aliased networks.
      * @param nodes The initial nodes array to start Networks iterator with.
@@ -60,6 +62,7 @@ public final class Networks<T> implements Iterator<DatabaseRecord<T>> {
 
     /**
      * Constructs a Networks instance with includeAliasedNetworks set to false by default.
+     *
      * @param reader The reader object.
      * @param typeParameterClass The type of data returned by the iterator.
      */
@@ -69,6 +72,7 @@ public final class Networks<T> implements Iterator<DatabaseRecord<T>> {
 
     /**
      * Returns the next DataRecord.
+     *
      * @return The next DataRecord.
      * @throws NetworksIterationException An exception when iterating over the networks.
      */
@@ -118,6 +122,7 @@ private boolean isInIpv4Subtree(byte[] ip) {
     * hasNext prepares the next network for reading with the Network method. It
     * returns true if there is another network to be processed and false if there
     * are no more networks.
+    *
     * @return boolean True if there is another network to be processed.
     * @throws NetworksIterationException Exception while iterating over the networks.
     */
@@ -169,6 +174,7 @@ static class NetworkNode {
 
         /**
          * Constructs a network node for internal use.
+         *
          * @param ip The ip address of the node.
          * @param prefix The prefix of the node.
          * @param pointer The node number

src/main/java/com/maxmind/db/NetworksIterationException.java

@@ -9,7 +9,7 @@
  * This exception extends RuntimeException because it is thrown by the iterator
  * methods in {@link Networks}.
  * </p>
- * 
+ *
  * @see Networks
  */
 public class NetworksIterationException extends RuntimeException { 

src/main/java/com/maxmind/db/Reader.java

@@ -197,7 +197,7 @@ int record = traverseResult[0];
      * Please note that a MaxMind DB may map IPv4 networks into several locations
      * in an IPv6 database. networks() iterates over the canonical locations and
      * not the aliases. To include the aliases, you can set includeAliasedNetworks to true.
-     * 
+     *
      * @param <T> Represents the data type(e.g., Map, HastMap, etc.).
      * @param typeParameterClass The type of data returned by the iterator.
      * @return Networks The Networks iterator.
@@ -216,7 +216,7 @@ public <T> Networks<T> networks(Class<T> typeParameterClass) throws
      * in an IPv6 database. This iterator will iterate over all of these locations
      * separately. To set the iteration over the IPv4 networks once, use the
      * includeAliasedNetworks option.
-     * 
+     *
      * @param <T> Represents the data type(e.g., Map, HastMap, etc.).
      * @param includeAliasedNetworks Enable including aliased networks.
      * @return Networks The Networks iterator.
@@ -283,6 +283,7 @@ private int findIpV4StartNode(ByteBuffer buffer)
      * in an IPv6 database. This iterator will iterate over all of these locations
      * separately. To only iterate over the IPv4 networks once, use the
      * includeAliasedNetworks option.
+     *
      * @param <T> Represents the data type(e.g., Map, HastMap, etc.).
      * @param network Specifies the network to be iterated.
      * @param includeAliasedNetworks Boolean for including aliased networks.
@@ -329,6 +330,7 @@ public <T> Networks<T> networksWithin(
 
     /**
      * Returns the node number and the prefix for the network.
+     *
      * @param ip The ip address to traverse.
      * @param bitCount The prefix.
      * @return int[]