diff --git a/jetty-io/src/main/java/org/eclipse/jetty/io/ClientConnectionFactory.java b/jetty-io/src/main/java/org/eclipse/jetty/io/ClientConnectionFactory.java
index d66a5901b38..6d9d1974cbf 100644
--- a/jetty-io/src/main/java/org/eclipse/jetty/io/ClientConnectionFactory.java
+++ b/jetty-io/src/main/java/org/eclipse/jetty/io/ClientConnectionFactory.java
@@ -91,6 +91,7 @@ public interface ClientConnectionFactory
          * Tests whether one of the protocols of this class is also present in the given candidates list.
          *
          * @param candidates the candidates to match against
+         * @param secure whether the protocol should be a secure one
          * @return whether one of the protocols of this class is present in the candidates
          */
         public boolean matches(List<String> candidates, boolean secure)
diff --git a/jetty-io/src/main/java/org/eclipse/jetty/io/EndPoint.java b/jetty-io/src/main/java/org/eclipse/jetty/io/EndPoint.java
index b82f5ecff6f..85c473aad30 100644
--- a/jetty-io/src/main/java/org/eclipse/jetty/io/EndPoint.java
+++ b/jetty-io/src/main/java/org/eclipse/jetty/io/EndPoint.java
@@ -33,7 +33,7 @@ import org.eclipse.jetty.util.thread.Invocable;
 /**
  * <p>EndPoint is the abstraction for an I/O channel that transports bytes.</p>
  *
- * <h3>Asynchronous Methods</h3>
+ * <p>Asynchronous Methods</p>
  * <p>The asynchronous scheduling methods of {@link EndPoint}
  * has been influenced by NIO.2 Futures and Completion
  * handlers, but does not use those actual interfaces because they have
@@ -42,7 +42,7 @@ import org.eclipse.jetty.util.thread.Invocable;
  * implementations of {@link Callback}, such as {@link FutureCallback} and
  * {@link IteratingCallback}.</p>
  *
- * <h3>Reads</h3>
+ * <p>Reads</p>
  * <p>A {@link FutureCallback} can be used to block until an endpoint is ready
  * to fill bytes - the notification will be emitted by the NIO subsystem:</p>
  * <pre>
@@ -56,7 +56,7 @@ import org.eclipse.jetty.util.thread.Invocable;
  * int filled = endPoint.fill(byteBuffer);
  * </pre>
  *
- * <h3>Asynchronous Reads</h3>
+ * <p>Asynchronous Reads</p>
  * <p>A {@link Callback} can be used to read asynchronously in its own dispatched
  * thread:</p>
  * <pre>
@@ -77,7 +77,7 @@ import org.eclipse.jetty.util.thread.Invocable;
  * });
  * </pre>
  *
- * <h3>Blocking Writes</h3>
+ * <p>Blocking Writes</p>
  * <p>The write contract is that the callback is completed when all the bytes
  * have been written or there is a failure.
  * Blocking writes look like this:</p>
diff --git a/jetty-jaas/src/main/java/org/eclipse/jetty/jaas/spi/JDBCLoginModule.java b/jetty-jaas/src/main/java/org/eclipse/jetty/jaas/spi/JDBCLoginModule.java
index 85d86cc8193..e7ec7bddc1f 100644
--- a/jetty-jaas/src/main/java/org/eclipse/jetty/jaas/spi/JDBCLoginModule.java
+++ b/jetty-jaas/src/main/java/org/eclipse/jetty/jaas/spi/JDBCLoginModule.java
@@ -29,13 +29,10 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 /**
- * JDBCLoginModule
- * <p>
- * JAAS LoginModule to retrieve user information from
- * a database and authenticate the user.
- * <h1>Notes</h1>
- * <p>This version uses plain old JDBC connections NOT
- * Datasources.
+ * <p>JAAS LoginModule to retrieve user information from
+ * a database and authenticate the user.</p>
+ * <p>Notes</p>
+ * <p>This version uses plain old JDBC connections NOT DataSources.</p>
  */
 public class JDBCLoginModule extends AbstractDatabaseLoginModule
 {
@@ -51,7 +48,6 @@ public class JDBCLoginModule extends AbstractDatabaseLoginModule
      *
      * @return the connection for this datasource
      * @throws Exception if unable to get the connection
-     * @see AbstractDatabaseLoginModule#getConnection()
      */
     @Override
     public Connection getConnection()