Small changes to reduce instances of long comment suppression

gedbrowser-renderer/src/main/java/org/schoellerfamily/gedbrowser/renderer/DefaultRenderer.java

@@ -5,12 +5,10 @@
 /**
  * @author Dick Schoeller
  */
-@SuppressWarnings("PMD.CommentSize")
 public final class DefaultRenderer extends GedRenderer<GedObject> {
     /**
-     * This constructor is public for testing purposes only. Do
-     * not try to call it outside of the context of the rendering
-     * engine.
+     * This constructor is public for testing purposes only. Do not try to
+     * call it outside of the context of the rendering engine.
      *
      * @param gedObject the gedObject from the parent renderer
      * @param rendererFactory the renderFactory from the parent renderer

gedbrowser-renderer/src/main/java/org/schoellerfamily/gedbrowser/renderer/FamilyRenderer.java

@@ -12,7 +12,6 @@
  *
  * @author Dick Schoeller
  */
-@SuppressWarnings({ "PMD.CommentSize" })
 public final class FamilyRenderer extends GedRenderer<Family> {
     /** */
     private static final int INDENT_INCREMENT = 2;
@@ -62,9 +61,8 @@ public List<GedRenderer<?>> getAttributes() {
     }
 
     /**
-     * This method is public for testing purposes only. Do
-     * not try to call it outside of the context of the rendering
-     * engine.
+     * This method is public for testing purposes only. Do not try to call it
+     * outside of the context of the rendering engine.
      *
      * @param builder Buffer for holding the rendition
      * @param pageRenderer The page renderer that is invoking this

gedbrowser-renderer/src/main/java/org/schoellerfamily/gedbrowser/renderer/NameNameHtmlRenderer.java

@@ -7,7 +7,6 @@
  *
  * @author Dick Schoeller
  */
-@SuppressWarnings("PMD.CommentSize")
 public class NameNameHtmlRenderer implements NameHtmlRenderer {
     /**
      * Holder for the NameRenderer that is using this helper.
@@ -17,9 +16,8 @@ public class NameNameHtmlRenderer implements NameHtmlRenderer {
     /**
      * Constructor.
      *
-     * This constructor is public for testing purposes only. Do
-     * not try to call it outside of the context of the rendering
-     * engine.
+     * This constructor is public for testing purposes only. Do not try to call
+     * it outside of the context of the rendering engine.
      *
      * @param nameRenderer the renderer that this is associated with.
      */

gedbrowser-renderer/src/main/java/org/schoellerfamily/gedbrowser/renderer/PersonRenderer.java

@@ -15,7 +15,7 @@
  *
  * @author Dick Schoeller
  */
-@SuppressWarnings({ "PMD.CommentSize", "PMD.GodClass" })
+@SuppressWarnings({ "PMD.GodClass" })
 public final class PersonRenderer extends GedRenderer<Person> {
     /**
      * Number of generations, including the root when rendering the tree.
@@ -44,9 +44,8 @@ public PersonRenderer(final Person gedObject,
     }
 
     /**
-     * This method is public for testing purposes only. Do
-     * not try to call it outside of the context of the rendering
-     * engine.
+     * This method is public for testing purposes only. Do not try to call it
+     * outside of the context of the rendering engine.
      *
      * @param builder Buffer for holding the rendition
      * @param pad Minimum number spaces for padding each line of the output
@@ -58,9 +57,8 @@ public void renderFather(final StringBuilder builder,
     }
 
     /**
-     * This method is public for testing purposes only. Do
-     * not try to call it outside of the context of the rendering
-     * engine.
+     * This method is public for testing purposes only. Do not try to call it
+     * outside of the context of the rendering engine.
      *
      * @param builder Buffer for holding the rendition
      * @param pad Minimum number spaces for padding each line of the output

gedbrowser-renderer/src/test/java/org/schoellerfamily/gedbrowser/renderer/test/AnonymousModePersonNameHtmlRendererTest.java

@@ -14,13 +14,12 @@
 import org.schoellerfamily.gedbrowser.renderer.RenderingContext;
 
 /**
- * All of these tests should return Living. The reason
- * is that the user created is one who shows up living
- * in the test. Therefore, the renderer will punt, no
+ * All of these tests should return Living. The reason is that the user created
+ * is one who shows up living in the test. Therefore, the renderer will punt, no
  * matter what the input string.
  *
- * There is a separate set of tests with a logged in
- * user. Those should get real rendering.
+ * There is a separate set of tests with a logged in user. Those should get real
+ * rendering.
  *
  * @author Dick Schoeller
  */

gedbrowser-renderer/src/test/java/org/schoellerfamily/gedbrowser/renderer/test/IndexRendererTest.java

@@ -12,7 +12,6 @@
 /**
  * @author Dick Schoeller
  */
-@SuppressWarnings("PMD.CommentSize")
 public final class IndexRendererTest {
 
     /**
@@ -72,7 +71,6 @@ public void testSurnames() throws IOException {
 
     /**
      * Prepare the expect string for index html test.
-     *
      * @param id person's ID
      * @param surname person's surname
      * @param prefix person's given name