Author: markt
Date: Mon Oct 13 15:16:55 2014
New Revision: 1631437

URL: http://svn.apache.org/r1631437
Log:
Fix some Javadoc errors reported when compiling with Java 8.

Modified:
    
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/BasicDataSource.java
    
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/datasources/package-info.java
    
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/package-info.java

Modified: 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/BasicDataSource.java
URL: 
http://svn.apache.org/viewvc/commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/BasicDataSource.java?rev=1631437&r1=1631436&r2=1631437&view=diff
==============================================================================
--- 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/BasicDataSource.java
 (original)
+++ 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/BasicDataSource.java
 Mon Oct 13 15:16:55 2014
@@ -1519,11 +1519,12 @@ public class BasicDataSource implements 
      * {@link #getRemoveAbandonedTimeout() removeAbandonedTimeout} seconds.</p>
      *
      * <p>Abandoned connections are identified and removed when
-     * {@link #getConnection()} is invoked and the following conditions hold
+     * {@link #getConnection()} is invoked and the following conditions hold:
+     * </p>
      * <ul><li>{@link #getRemoveAbandonedOnBorrow()} or
      *         {@link #getRemoveAbandonedOnMaintenance()} = true</li>
-     *     <li>{@link #getNumActive()} > {@link #getMaxTotal()} - 3 </li>
-     *     <li>{@link #getNumIdle()} < 2 </li></ul></p>
+     *     <li>{@link #getNumActive()} &gt; {@link #getMaxTotal()} - 3 </li>
+     *     <li>{@link #getNumIdle()} &lt; 2 </li></ul>
      *
      * @see #getRemoveAbandonedTimeout()
      */
@@ -1570,11 +1571,12 @@ public class BasicDataSource implements 
      * {@link #getRemoveAbandonedTimeout() removeAbandonedTimeout} seconds.</p>
      *
      * <p>Abandoned connections are identified and removed when
-     * {@link #getConnection()} is invoked and the following conditions hold
+     * {@link #getConnection()} is invoked and the following conditions hold:
+     * </p>
      * <ul><li>{@link #getRemoveAbandonedOnBorrow()} or
      *         {@link #getRemoveAbandonedOnMaintenance()} = true</li>
-     *     <li>{@link #getNumActive()} > {@link #getMaxTotal()} - 3 </li>
-     *     <li>{@link #getNumIdle()} < 2 </li></ul></p>
+     *     <li>{@link #getNumActive()} &gt; {@link #getMaxTotal()} - 3 </li>
+     *     <li>{@link #getNumIdle()} &lt; 2 </li></ul>
      *
      * @see #getRemoveAbandonedTimeout()
      */
@@ -1614,13 +1616,13 @@ public class BasicDataSource implements 
      * one of these to execute a query (using one of the execute methods)
      * resets the lastUsed property of the parent connection.</p>
      *
-     * <p>Abandoned connection cleanup happens when
-     * <code><ul>
+     * <p>Abandoned connection cleanup happens when:</p>
+     * <ul>
      * <li>{@link #getRemoveAbandonedOnBorrow()} or
      *     {@link #getRemoveAbandonedOnMaintenance()} = true</li>
      * <li>{@link #getNumIdle() numIdle} &lt; 2</li>
      * <li>{@link #getNumActive() numActive} &gt; {@link #getMaxTotal() 
maxTotal} - 3</li>
-     * </ul></code></p>
+     * </ul>
      *
      * <p>The default value is 300 seconds.</p>
      */

Modified: 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/datasources/package-info.java
URL: 
http://svn.apache.org/viewvc/commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/datasources/package-info.java?rev=1631437&r1=1631436&r2=1631437&view=diff
==============================================================================
--- 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/datasources/package-info.java
 (original)
+++ 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/datasources/package-info.java
 Mon Oct 13 15:16:55 2014
@@ -36,7 +36,7 @@
  * webapp:
  * </p>
  *
- * <code><pre>
+ * <code>
  *  &lt;Resource name="jdbc/bookstore" auth="Container"
  *             
type="org.apache.commons.dbcp2.datasources.PerUserPoolPoolDataSource"/&gt;
  *   &lt;ResourceParams name="jdbc/bookstore"&gt;
@@ -51,14 +51,14 @@
  *       &lt;name&gt;defaultMaxTotal&lt;/name&gt;&lt;value&gt;30&lt;/value&gt;
  *     &lt;/parameter&gt;
  *   &lt;/ResourceParams&gt;
- * </pre></code>
+ * </code>
  *
  * <p>
  * In web.xml.  Note that elements must be given in the order of the dtd
  * described in the servlet specification:
  * </p>
  *
- * <code><pre>
+ * <code>
  * &lt;resource-ref&gt;
  *   &lt;description&gt;
  *     Resource reference to a factory for java.sql.Connection
@@ -75,7 +75,7 @@
  *     Container
  *   &lt;/res-auth&gt;
  * &lt;/resource-ref&gt;
- * </pre></code>
+ * </code>
  *
  * <p>
  * Apache Tomcat deploys all objects configured similarly to above within the
@@ -90,7 +90,7 @@
  * below:
  * </p>
  *
- * <code><pre>
+ * <code>
  *
  *     Context ctx = new InitialContext();
  *     DataSource ds = (DataSource)
@@ -109,7 +109,7 @@
  *             con.close();
  *     }
  *
- * </pre></code>
+ * </code>
  *
  * <p>
  * The reference to the <code>DataSource</code> could be maintained, for
@@ -130,7 +130,7 @@
  * DriverAdapterCPDS as the backend source, though any CPDS is applicable.
  * </p>
  *
- * <code><pre>
+ * <code>
  *
  * public class Pool
  * {
@@ -158,13 +158,13 @@
  *     }
  * }
  *
- * </pre></code>
+ * </code>
  *
  * <p>
  * This class can then be used wherever a connection is needed:
  * </p>
  *
- * <code><pre>
+ * <code>
  *     Connection con = null;
  *     try
  *     {
@@ -178,6 +178,6 @@
  *         if (con != null)
  *             con.close();
  *     }
- * </pre></code>
+ * </code>
  */
 package org.apache.commons.dbcp2.datasources;

Modified: 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/package-info.java
URL: 
http://svn.apache.org/viewvc/commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/package-info.java?rev=1631437&r1=1631436&r2=1631437&view=diff
==============================================================================
--- 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/package-info.java
 (original)
+++ 
commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/package-info.java
 Mon Oct 13 15:16:55 2014
@@ -88,7 +88,7 @@
  *    {@link org.apache.commons.dbcp2.PoolableConnectionFactory} is for.
  * </p><p>
  *    To create the {@link org.apache.commons.dbcp2.PoolableConnectionFactory},
- *    you'll need at least two things:
+ *    you'll need at least two things:</p>
  *    <ol>
  *     <li>
  * A {@link org.apache.commons.dbcp2.ConnectionFactory} from which
@@ -97,14 +97,14 @@
  *     <li>
  * An empty and factory-less {@link org.apache.commons.pool2.ObjectPool}
  * in which the {@link java.sql.Connection}s will be stored.
- * <br></br>
+ * <br>
  * When you pass an {@link org.apache.commons.pool2.ObjectPool} into the
  * {@link org.apache.commons.dbcp2.PoolableConnectionFactory}, it will
  * automatically register itself as the {@link 
org.apache.commons.pool2.PooledObjectFactory}
  * for that pool.
  *     </li>
  *    </ol>
- * </p><p>
+ * <p>
  *    In code, that might look like this:
  * <pre>GenericObjectPool connectionPool = new GenericObjectPool(null);
  * ConnectionFactory connectionFactory = new 
DriverManagerConnectionFactory("jdbc:some:connect:string", "username", 
"password");
@@ -114,18 +114,17 @@
  *    To create a {@link org.apache.commons.dbcp2.PoolingDriver}, we do the 
same thing,
  *    except that instead of creating a {@link javax.sql.DataSource} on the 
last line,
  *    we create a {@link org.apache.commons.dbcp2.PoolingDriver}, and register 
the
- *    {@code connectionPool} with it.  E.g.,:
+ *    {@code connectionPool} with it.  E.g.,:</p>
  * <pre>GenericObjectPool connectionPool = new GenericObjectPool(null);
  * ConnectionFactory connectionFactory = new 
DriverManagerConnectionFactory("jdbc:some:connect:string", "username", 
"password");
  * PoolableConnectionFactory poolableConnectionFactory = new 
PoolableConnectionFactory(connectionFactory,connectionPool,null,null,false,true);
  * PoolingDriver driver = new PoolingDriver();
  * driver.registerPool("example",connectionPool);</pre>
- * </p><p>
+ * <p>
  *    Since the {@link org.apache.commons.dbcp2.PoolingDriver} registers itself
  *    with the {@link java.sql.DriverManager} when it is created, now you can 
just
  *    go to the {@link java.sql.DriverManager} to create your {@link 
java.sql.Connection}s,
- *    like you normally would:
+ *    like you normally would:</p>
  * <pre>Connection conn = 
DriverManager.getConnection("jdbc:apache:commons:dbcp:example");</pre>
- * </p>
  */
 package org.apache.commons.dbcp2;


Reply via email to