diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java b/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java
index dc8f19baea..155de383d2 100644
--- a/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java
+++ b/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java
@@ -26,7 +26,7 @@ import net.sourceforge.pmd.util.ResourceLoader;
  * 
  * <pre>
  * &#64;Parameters
- *     public static Collection<Object[]> data() {
+ *     public static Collection&lt;Object[]&gt; data() {
  *       return Arrays.asList(new Object[][] {
  *            { MyLanguageModule.NAME, MyLanguageModule.TERSE_NAME, "1.1",
  *              LanguageRegistry.getLanguage(MyLanguageModule.NAME).getVersion("1.1") },
@@ -39,8 +39,8 @@ import net.sourceforge.pmd.util.ResourceLoader;
  *       });
  * </pre>
  * 
- * For the parameters, see the constructor
- * {@link #AbstractLanguageVersionTest(String, String, String, LanguageVersion)}.
+ * <p>For the parameters, see the constructor
+ * {@link #AbstractLanguageVersionTest(String, String, String, LanguageVersion)}.</p>
  */
 @RunWith(Parameterized.class)
 public class AbstractLanguageVersionTest {
diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java b/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java
index e33e9e030a..0a4ca4caf2 100644
--- a/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java
+++ b/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java
@@ -19,7 +19,7 @@ import org.junit.Rule;
  * there is a workaround that must be shared by both tests (PMD and CPD's) I
  * felt compelled to move it to a single classes.
  *
- * @author Romain Pelisse <belaran@gmail.com>
+ * @author Romain Pelisse &lt;belaran@gmail.com&gt;
  *
  */
 public abstract class AbstractAntTestHelper {
diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java b/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java
index 18b8274ec7..ef7fd48918 100644
--- a/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java
+++ b/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java
@@ -19,7 +19,7 @@ import org.junit.BeforeClass;
 import net.sourceforge.pmd.PMD;
 
 /**
- * @author Romain Pelisse <belaran@gmail.com>
+ * @author Romain Pelisse &lt;belaran@gmail.com&gt;
  *
  */
 public abstract class BaseCLITest {
diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java b/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java
index 33b331e11c..62f7ad7f1b 100644
--- a/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java
+++ b/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java
@@ -25,13 +25,12 @@ import org.junit.runners.model.TestClass;
 import net.sourceforge.pmd.Rule;
 
 /**
- * A test runner for rule tests. Unlike
- * {@link SimpleAggregatorTst.CustomXmlTestClassMethodsRunner} it also reports
+ * A test runner for rule tests. It reports
  * the successful executed tests and allows to selectively execute single test
  * cases (it is {@link Filterable}).
- * <p>
- * In order to use it, you'll need to subclass {@link SimpleAggregatorTst} and
- * annotate your test class with RunWith:
+ *
+ * <p>In order to use it, you'll need to subclass {@link SimpleAggregatorTst} and
+ * annotate your test class with RunWith:</p>
  * 
  * <pre>
  * {@code @}RunWith(PMDTestRunner.class)
@@ -39,7 +38,6 @@ import net.sourceforge.pmd.Rule;
  * ...
  * }
  * </pre>
- * </p>
  */
 public class PMDTestRunner extends Runner implements Filterable {
     private final Description desc;