This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-dbcp.git

commit c294658cb39d44671e25f608fd4c615587a0c7db
Author: Gary Gregory <[email protected]>
AuthorDate: Fri Nov 21 20:26:41 2025 +0000

    Javadoc @link
---
 .../commons/dbcp2/BasicDataSourceFactory.java      |  10 +-
 .../apache/commons/dbcp2/ConnectionFactory.java    |   6 +-
 .../apache/commons/dbcp2/DelegatingConnection.java |  12 +-
 .../commons/dbcp2/DelegatingDatabaseMetaData.java  |   6 +-
 .../apache/commons/dbcp2/DelegatingResultSet.java  |   8 +-
 .../apache/commons/dbcp2/DelegatingStatement.java  |   8 +-
 .../java/org/apache/commons/dbcp2/PStmtKey.java    | 129 +++++++++++----------
 .../apache/commons/dbcp2/PoolableConnection.java   |  16 +--
 .../commons/dbcp2/PoolableConnectionFactory.java   |   2 +-
 .../apache/commons/dbcp2/PoolingConnection.java    |   4 +-
 .../apache/commons/dbcp2/PoolingDataSource.java    |   2 +-
 .../commons/dbcp2/cpdsadapter/ConnectionImpl.java  |  65 ++++++-----
 .../dbcp2/cpdsadapter/DriverAdapterCPDS.java       |  27 +++--
 .../commons/dbcp2/cpdsadapter/PStmtKeyCPDS.java    |  24 ++--
 .../dbcp2/cpdsadapter/PooledConnectionImpl.java    |  97 ++++++++--------
 .../dbcp2/datasources/CPDSConnectionFactory.java   |   3 +-
 .../dbcp2/datasources/InstanceKeyDataSource.java   |  12 +-
 .../datasources/InstanceKeyDataSourceFactory.java  |   2 +-
 .../datasources/KeyedCPDSConnectionFactory.java    |   2 +-
 .../dbcp2/datasources/PerUserPoolDataSource.java   |   5 +-
 .../datasources/PerUserPoolDataSourceFactory.java  |   2 +-
 .../dbcp2/datasources/SharedPoolDataSource.java    |   3 +-
 .../datasources/SharedPoolDataSourceFactory.java   |   2 +-
 .../commons/dbcp2/datasources/UserPassKey.java     |   2 +-
 24 files changed, 230 insertions(+), 219 deletions(-)

diff --git a/src/main/java/org/apache/commons/dbcp2/BasicDataSourceFactory.java 
b/src/main/java/org/apache/commons/dbcp2/BasicDataSourceFactory.java
index 64b0cd0c..4d5b25ed 100644
--- a/src/main/java/org/apache/commons/dbcp2/BasicDataSourceFactory.java
+++ b/src/main/java/org/apache/commons/dbcp2/BasicDataSourceFactory.java
@@ -49,12 +49,12 @@ import org.apache.commons.pool2.impl.BaseObjectPoolConfig;
 import org.apache.commons.pool2.impl.GenericObjectPoolConfig;
 
 /**
- * JNDI object factory that creates an instance of {@code BasicDataSource} 
that has been configured based on the
- * {@code RefAddr} values of the specified {@code Reference}, which must match 
the names and data types of the
- * {@code BasicDataSource} bean properties with the following exceptions:
+ * JNDI object factory that creates an instance of {@link BasicDataSource} 
that has been configured based on the
+ * {@link RefAddr} values of the specified {@code Reference}, which must match 
the names and data types of the
+ * {@link BasicDataSource} bean properties with the following exceptions:
  * <ul>
  * <li>{@code connectionInitSqls} must be passed to this factory as a single 
String using semicolon to delimit the
- * statements whereas {@code BasicDataSource} requires a collection of 
Strings.</li>
+ * statements whereas {@link BasicDataSource} requires a collection of 
Strings.</li>
  * </ul>
  *
  * @since 2.0
@@ -383,7 +383,7 @@ public class BasicDataSourceFactory implements 
ObjectFactory {
     }
 
     /**
-     * Creates and return a new {@code BasicDataSource} instance. If no 
instance can be created, return
+     * Creates and return a new {@link BasicDataSource} instance. If no 
instance can be created, return
      * {@code null} instead.
      *
      * @param obj
diff --git a/src/main/java/org/apache/commons/dbcp2/ConnectionFactory.java 
b/src/main/java/org/apache/commons/dbcp2/ConnectionFactory.java
index 13872ec3..330cad01 100644
--- a/src/main/java/org/apache/commons/dbcp2/ConnectionFactory.java
+++ b/src/main/java/org/apache/commons/dbcp2/ConnectionFactory.java
@@ -20,15 +20,15 @@ import java.sql.Connection;
 import java.sql.SQLException;
 
 /**
- * Abstract factory interface for creating {@link java.sql.Connection}s.
+ * Abstract factory interface for creating {@link Connection}s.
  *
  * @since 2.0
  */
 public interface ConnectionFactory {
     /**
-     * Create a new {@link java.sql.Connection} in an implementation specific 
fashion.
+     * Create a new {@link Connection} in an implementation specific fashion.
      *
-     * @return a new {@link java.sql.Connection}
+     * @return a new {@link Connection}
      * @throws SQLException
      *             if a database error occurs creating the connection
      */
diff --git a/src/main/java/org/apache/commons/dbcp2/DelegatingConnection.java 
b/src/main/java/org/apache/commons/dbcp2/DelegatingConnection.java
index 56ca915d..98af01cd 100644
--- a/src/main/java/org/apache/commons/dbcp2/DelegatingConnection.java
+++ b/src/main/java/org/apache/commons/dbcp2/DelegatingConnection.java
@@ -433,14 +433,14 @@ public class DelegatingConnection<C extends Connection> 
extends AbandonedTrace i
     }
 
     /**
-     * If my underlying {@link Connection} is not a {@code 
DelegatingConnection}, returns it, otherwise recursively
+     * If my underlying {@link Connection} is not a {@link 
DelegatingConnection}, returns it, otherwise recursively
      * invokes this method on my delegate.
      * <p>
-     * Hence this method will return the first delegate that is not a {@code 
DelegatingConnection}, or {@code null} when
-     * no non-{@code DelegatingConnection} delegate can be found by traversing 
this chain.
+     * Hence this method will return the first delegate that is not a {@link 
DelegatingConnection}, or {@code null} when
+     * no non-{@link DelegatingConnection} delegate can be found by traversing 
this chain.
      * </p>
      * <p>
-     * This method is useful when you may have nested {@code 
DelegatingConnection}s, and you want to make sure to obtain
+     * This method is useful when you may have nested {@link 
DelegatingConnection}s, and you want to make sure to obtain
      * a "genuine" {@link Connection}.
      * </p>
      *
@@ -549,11 +549,11 @@ public class DelegatingConnection<C extends Connection> 
extends AbandonedTrace i
     }
 
     /**
-     * Handles the given {@code SQLException}.
+     * Handles the given {@link SQLException}.
      *
      * @param <T> The throwable type.
      * @param e   The SQLException
-     * @return the given {@code SQLException}
+     * @return the given {@link SQLException}
      * @since 2.7.0
      */
     protected <T extends Throwable> T handleExceptionNoThrow(final T e) {
diff --git 
a/src/main/java/org/apache/commons/dbcp2/DelegatingDatabaseMetaData.java 
b/src/main/java/org/apache/commons/dbcp2/DelegatingDatabaseMetaData.java
index e64cf827..3b860d62 100644
--- a/src/main/java/org/apache/commons/dbcp2/DelegatingDatabaseMetaData.java
+++ b/src/main/java/org/apache/commons/dbcp2/DelegatingDatabaseMetaData.java
@@ -264,13 +264,13 @@ public class DelegatingDatabaseMetaData implements 
DatabaseMetaData {
     }
 
     /**
-     * If my underlying {@link ResultSet} is not a {@code 
DelegatingResultSet}, returns it, otherwise recursively invokes this method on 
my delegate.
+     * If my underlying {@link ResultSet} is not a {@link 
DelegatingResultSet}, returns it, otherwise recursively invokes this method on 
my delegate.
      * <p>
-     * Hence this method will return the first delegate that is not a {@code 
DelegatingResultSet}, or {@code null} when no non-{@code DelegatingResultSet}
+     * Hence this method will return the first delegate that is not a {@link 
DelegatingResultSet}, or {@code null} when no non-{@link DelegatingResultSet}
      * delegate can be found by traversing this chain.
      * </p>
      * <p>
-     * This method is useful when you may have nested {@code 
DelegatingResultSet}s, and you want to make sure to obtain a "genuine" {@link 
ResultSet}.
+     * This method is useful when you may have nested {@link 
DelegatingResultSet}s, and you want to make sure to obtain a "genuine" {@link 
ResultSet}.
      * </p>
      *
      * @return the innermost database meta data.
diff --git a/src/main/java/org/apache/commons/dbcp2/DelegatingResultSet.java 
b/src/main/java/org/apache/commons/dbcp2/DelegatingResultSet.java
index c7d5fc4b..39d6aa36 100644
--- a/src/main/java/org/apache/commons/dbcp2/DelegatingResultSet.java
+++ b/src/main/java/org/apache/commons/dbcp2/DelegatingResultSet.java
@@ -591,14 +591,14 @@ public final class DelegatingResultSet extends 
AbandonedTrace implements ResultS
     }
 
     /**
-     * If my underlying {@link ResultSet} is not a {@code 
DelegatingResultSet}, returns it, otherwise recursively
+     * If my underlying {@link ResultSet} is not a {@link 
DelegatingResultSet}, returns it, otherwise recursively
      * invokes this method on my delegate.
      * <p>
-     * Hence this method will return the first delegate that is not a {@code 
DelegatingResultSet}, or {@code null} when
-     * no non-{@code DelegatingResultSet} delegate can be found by traversing 
this chain.
+     * Hence this method will return the first delegate that is not a {@link 
DelegatingResultSet}, or {@code null} when
+     * no non-{@link DelegatingResultSet} delegate can be found by traversing 
this chain.
      * </p>
      * <p>
-     * This method is useful when you may have nested {@code 
DelegatingResultSet}s, and you want to make sure to obtain
+     * This method is useful when you may have nested {@link 
DelegatingResultSet}s, and you want to make sure to obtain
      * a "genuine" {@link ResultSet}.
      * </p>
      *
diff --git a/src/main/java/org/apache/commons/dbcp2/DelegatingStatement.java 
b/src/main/java/org/apache/commons/dbcp2/DelegatingStatement.java
index df1389f0..83d0348d 100644
--- a/src/main/java/org/apache/commons/dbcp2/DelegatingStatement.java
+++ b/src/main/java/org/apache/commons/dbcp2/DelegatingStatement.java
@@ -451,14 +451,14 @@ public class DelegatingStatement extends AbandonedTrace 
implements Statement {
     }
 
     /**
-     * If my underlying {@link Statement} is not a {@code 
DelegatingStatement}, returns it, otherwise recursively
+     * If my underlying {@link Statement} is not a {@link 
DelegatingStatement}, returns it, otherwise recursively
      * invokes this method on my delegate.
      * <p>
-     * Hence this method will return the first delegate that is not a {@code 
DelegatingStatement} or {@code null} when
-     * no non-{@code DelegatingStatement} delegate can be found by traversing 
this chain.
+     * Hence this method will return the first delegate that is not a {@link 
DelegatingStatement} or {@code null} when
+     * no non-{@link DelegatingStatement} delegate can be found by traversing 
this chain.
      * </p>
      * <p>
-     * This method is useful when you may have nested {@code 
DelegatingStatement}s, and you want to make sure to obtain
+     * This method is useful when you may have nested {@link 
DelegatingStatement}s, and you want to make sure to obtain
      * a "genuine" {@link Statement}.
      * </p>
      *
diff --git a/src/main/java/org/apache/commons/dbcp2/PStmtKey.java 
b/src/main/java/org/apache/commons/dbcp2/PStmtKey.java
index 12dfe1a3..56d58128 100644
--- a/src/main/java/org/apache/commons/dbcp2/PStmtKey.java
+++ b/src/main/java/org/apache/commons/dbcp2/PStmtKey.java
@@ -17,6 +17,7 @@
 package org.apache.commons.dbcp2;
 
 import java.sql.Connection;
+import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.Arrays;
@@ -68,20 +69,20 @@ public class PStmtKey {
     private final String sql;
 
     /**
-     * Result set type; one of {@code ResultSet.TYPE_FORWARD_ONLY}, {@code 
ResultSet.TYPE_SCROLL_INSENSITIVE}, or
-     * {@code ResultSet.TYPE_SCROLL_SENSITIVE}.
+     * Result set type; one of {@link ResultSet#TYPE_FORWARD_ONLY}, {@link 
ResultSet#TYPE_SCROLL_INSENSITIVE}, or
+     * {@link ResultSet#TYPE_SCROLL_SENSITIVE}.
      */
     private final Integer resultSetType;
 
     /**
-     * Result set concurrency. A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     * {@code ResultSet.CONCUR_UPDATABLE}.
+     * Result set concurrency. A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     * {@link ResultSet#CONCUR_UPDATABLE}.
      */
     private final Integer resultSetConcurrency;
 
     /**
-     * Result set holdability. One of the following {@code ResultSet} 
constants: {@code ResultSet.HOLD_CURSORS_OVER_COMMIT}
-     * or {@code ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     * Result set holdability. One of the following {@link ResultSet} 
constants: {@link ResultSet#HOLD_CURSORS_OVER_COMMIT}
+     * or {@link ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      */
     private final Integer resultSetHoldability;
 
@@ -96,8 +97,8 @@ public class PStmtKey {
     private final String schema;
 
     /**
-     * A flag indicating whether auto-generated keys should be returned; one 
of {@code Statement.RETURN_GENERATED_KEYS} or
-     * {@code Statement.NO_GENERATED_KEYS}.
+     * A flag indicating whether auto-generated keys should be returned; one 
of {@link Statement#RETURN_GENERATED_KEYS} or
+     * {@link Statement#NO_GENERATED_KEYS}.
      */
     private final Integer autoGeneratedKeys;
 
@@ -136,10 +137,10 @@ public class PStmtKey {
      * Constructs a key to uniquely identify a prepared statement.
      *
      * @param sql The SQL statement.
-     * @param resultSetType A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
+     * @param resultSetType A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
      * @deprecated Use {@link #PStmtKey(String, String, String, int, int)}.
      */
     @Deprecated
@@ -165,7 +166,7 @@ public class PStmtKey {
      * @param sql The SQL statement.
      * @param catalog The catalog.
      * @param autoGeneratedKeys A flag indicating whether auto-generated keys 
should be returned; one of
-     *        {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *        {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @deprecated Use {@link #PStmtKey(String, String, String, int)}.
      */
     @Deprecated
@@ -178,10 +179,10 @@ public class PStmtKey {
      *
      * @param sql The SQL statement.
      * @param catalog The catalog.
-     * @param resultSetType A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
+     * @param resultSetType A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
      * @deprecated Use {@link #PStmtKey(String, String, String, int, int)}.
      */
     @Deprecated
@@ -194,12 +195,12 @@ public class PStmtKey {
      *
      * @param sql The SQL statement.
      * @param catalog The catalog.
-     * @param resultSetType a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}
-     * @param resultSetHoldability One of the following {@code ResultSet} 
constants:
-     *        {@code ResultSet.HOLD_CURSORS_OVER_COMMIT} or {@code 
ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     * @param resultSetType a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}
+     * @param resultSetHoldability One of the following {@link ResultSet} 
constants:
+     *        {@link ResultSet#HOLD_CURSORS_OVER_COMMIT} or {@link 
ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      * @deprecated Use {@link #PStmtKey(String, String, String, int, int, 
int)}.
      */
     @Deprecated
@@ -212,12 +213,12 @@ public class PStmtKey {
      *
      * @param sql The SQL statement.
      * @param catalog The catalog.
-     * @param resultSetType a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
-     * @param resultSetHoldability One of the following {@code ResultSet} 
constants:
-     *        {@code ResultSet.HOLD_CURSORS_OVER_COMMIT} or {@code 
ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     * @param resultSetType a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
+     * @param resultSetHoldability One of the following {@link ResultSet} 
constants:
+     *        {@link ResultSet#HOLD_CURSORS_OVER_COMMIT} or {@link 
ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      * @param statementType The SQL statement type, prepared or callable.
      * @deprecated Use {@link #PStmtKey(String, String, String, int, int, int, 
PoolingConnection.StatementType)}
      */
@@ -233,10 +234,10 @@ public class PStmtKey {
      *
      * @param sql The SQL statement.
      * @param catalog The catalog.
-     * @param resultSetType A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
+     * @param resultSetType A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
      * @param statementType The SQL statement type, prepared or callable.
      * @deprecated Use {@link #PStmtKey(String, String, String, int, int, 
PoolingConnection.StatementType)}.
      */
@@ -280,7 +281,7 @@ public class PStmtKey {
      * @param catalog The catalog.
      * @param statementType The SQL statement type, prepared or callable.
      * @param autoGeneratedKeys A flag indicating whether auto-generated keys 
should be returned; one of
-     *        {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *        {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @deprecated Use {@link #PStmtKey(String, String, String, 
PoolingConnection.StatementType, Integer)}
      */
     @Deprecated
@@ -308,7 +309,7 @@ public class PStmtKey {
      * @param catalog The catalog.
      * @param schema The schema
      * @param autoGeneratedKeys A flag indicating whether auto-generated keys 
should be returned; one of
-     *        {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *        {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @since 2.5.0
      */
     public PStmtKey(final String sql, final String catalog, final String 
schema, final int autoGeneratedKeys) {
@@ -321,10 +322,10 @@ public class PStmtKey {
      * @param sql The SQL statement.
      * @param catalog The catalog.
      * @param schema The schema
-     * @param resultSetType A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
+     * @param resultSetType A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
      */
     public PStmtKey(final String sql, final String catalog, final String 
schema, final int resultSetType, final int resultSetConcurrency) {
         this(sql, catalog, schema, resultSetType, resultSetConcurrency, 
StatementType.PREPARED_STATEMENT);
@@ -336,12 +337,12 @@ public class PStmtKey {
      * @param sql The SQL statement.
      * @param catalog The catalog.
      * @param schema The schema
-     * @param resultSetType a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}
-     * @param resultSetHoldability One of the following {@code ResultSet} 
constants:
-     *        {@code ResultSet.HOLD_CURSORS_OVER_COMMIT} or {@code 
ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     * @param resultSetType a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}
+     * @param resultSetHoldability One of the following {@link ResultSet} 
constants:
+     *        {@link ResultSet#HOLD_CURSORS_OVER_COMMIT} or {@link 
ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      * @since 2.5.0
      */
     public PStmtKey(final String sql, final String catalog, final String 
schema, final int resultSetType, final int resultSetConcurrency,
@@ -355,12 +356,12 @@ public class PStmtKey {
      * @param sql The SQL statement.
      * @param catalog The catalog.
      * @param schema The schema.
-     * @param resultSetType a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
-     * @param resultSetHoldability One of the following {@code ResultSet} 
constants:
-     *        {@code ResultSet.HOLD_CURSORS_OVER_COMMIT} or {@code 
ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     * @param resultSetType a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
+     * @param resultSetHoldability One of the following {@link ResultSet} 
constants:
+     *        {@link ResultSet#HOLD_CURSORS_OVER_COMMIT} or {@link 
ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      * @param statementType The SQL statement type, prepared or callable.
      * @since 2.5.0
      */
@@ -376,10 +377,10 @@ public class PStmtKey {
      * @param sql The SQL statement.
      * @param catalog The catalog.
      * @param schema The schema.
-     * @param resultSetType A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *        {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
-     * @param resultSetConcurrency A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *        {@code ResultSet.CONCUR_UPDATABLE}.
+     * @param resultSetType A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *        {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
+     * @param resultSetConcurrency A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *        {@link ResultSet#CONCUR_UPDATABLE}.
      * @param statementType The SQL statement type, prepared or callable.
      * @since 2.5.0
      */
@@ -456,7 +457,7 @@ public class PStmtKey {
      * @param schema The schema.
      * @param statementType The SQL statement type, prepared or callable.
      * @param autoGeneratedKeys A flag indicating whether auto-generated keys 
should be returned; one of
-     *        {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *        {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @since 2.5.0
      */
     public PStmtKey(final String sql, final String catalog, final String 
schema, final StatementType statementType, final Integer autoGeneratedKeys) {
@@ -535,8 +536,8 @@ public class PStmtKey {
     }
 
     /**
-     * Gets a flag indicating whether auto-generated keys should be returned; 
one of {@code Statement.RETURN_GENERATED_KEYS}
-     * or {@code Statement.NO_GENERATED_KEYS}.
+     * Gets a flag indicating whether auto-generated keys should be returned; 
one of {@link Statement#RETURN_GENERATED_KEYS}
+     * or {@link Statement#NO_GENERATED_KEYS}.
      *
      * @return a flag indicating whether auto-generated keys should be 
returned.
      */
@@ -572,8 +573,8 @@ public class PStmtKey {
     }
 
     /**
-     * Gets the result set concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     * {@code ResultSet.CONCUR_UPDATABLE}.
+     * Gets the result set concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     * {@link ResultSet#CONCUR_UPDATABLE}.
      *
      * @return The result set concurrency type.
      */
@@ -582,8 +583,8 @@ public class PStmtKey {
     }
 
     /**
-     * Gets the result set holdability, one of the following {@code ResultSet} 
constants:
-     * {@code ResultSet.HOLD_CURSORS_OVER_COMMIT} or {@code 
ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     * Gets the result set holdability, one of the following {@link ResultSet} 
constants:
+     * {@link ResultSet#HOLD_CURSORS_OVER_COMMIT} or {@link 
ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      *
      * @return The result set holdability.
      */
@@ -592,8 +593,8 @@ public class PStmtKey {
     }
 
     /**
-     * Gets the result set type, one of {@code ResultSet.TYPE_FORWARD_ONLY}, 
{@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or
-     * {@code ResultSet.TYPE_SCROLL_SENSITIVE}.
+     * Gets the result set type, one of {@link ResultSet#TYPE_FORWARD_ONLY}, 
{@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or
+     * {@link ResultSet#TYPE_SCROLL_SENSITIVE}.
      *
      * @return the result set type.
      */
diff --git a/src/main/java/org/apache/commons/dbcp2/PoolableConnection.java 
b/src/main/java/org/apache/commons/dbcp2/PoolableConnection.java
index 971fd5eb..e39ea1bc 100644
--- a/src/main/java/org/apache/commons/dbcp2/PoolableConnection.java
+++ b/src/main/java/org/apache/commons/dbcp2/PoolableConnection.java
@@ -394,11 +394,11 @@ public class PoolableConnection extends 
DelegatingConnection<Connection> impleme
      * Validates the connection, using the following algorithm:
      * <ol>
      * <li>If {@code fastFailValidation} (constructor argument) is {@code 
true} and this connection has previously
-     * thrown a fatal disconnection exception, a {@code SQLException} is 
thrown.</li>
+     * thrown a fatal disconnection exception, a {@link SQLException} is 
thrown.</li>
      * <li>If {@code sql} is null, the driver's #{@link 
Connection#isValid(int) isValid(timeout)} is called. If it
-     * returns {@code false}, {@code SQLException} is thrown; otherwise, this 
method returns successfully.</li>
-     * <li>If {@code sql} is not null, it is executed as a query and if the 
resulting {@code ResultSet} contains at
-     * least one row, this method returns successfully. If not, {@code 
SQLException} is thrown.</li>
+     * returns {@code false}, {@link SQLException} is thrown; otherwise, this 
method returns successfully.</li>
+     * <li>If {@code sql} is not null, it is executed as a query and if the 
resulting {@link ResultSet} contains at
+     * least one row, this method returns successfully. If not, {@link 
SQLException} is thrown.</li>
      * </ol>
      *
      * @param sql
@@ -448,11 +448,11 @@ public class PoolableConnection extends 
DelegatingConnection<Connection> impleme
      * Validates the connection, using the following algorithm:
      * <ol>
      * <li>If {@code fastFailValidation} (constructor argument) is {@code 
true} and this connection has previously
-     * thrown a fatal disconnection exception, a {@code SQLException} is 
thrown.</li>
+     * thrown a fatal disconnection exception, a {@link SQLException} is 
thrown.</li>
      * <li>If {@code sql} is null, the driver's #{@link 
Connection#isValid(int) isValid(timeout)} is called. If it
-     * returns {@code false}, {@code SQLException} is thrown; otherwise, this 
method returns successfully.</li>
-     * <li>If {@code sql} is not null, it is executed as a query and if the 
resulting {@code ResultSet} contains at
-     * least one row, this method returns successfully. If not, {@code 
SQLException} is thrown.</li>
+     * returns {@code false}, {@link SQLException} is thrown; otherwise, this 
method returns successfully.</li>
+     * <li>If {@code sql} is not null, it is executed as a query and if the 
resulting {@link ResultSet} contains at
+     * least one row, this method returns successfully. If not, {@link 
SQLException} is thrown.</li>
      * </ol>
      *
      * @param sql
diff --git 
a/src/main/java/org/apache/commons/dbcp2/PoolableConnectionFactory.java 
b/src/main/java/org/apache/commons/dbcp2/PoolableConnectionFactory.java
index 6f21326d..bdcdddc6 100644
--- a/src/main/java/org/apache/commons/dbcp2/PoolableConnectionFactory.java
+++ b/src/main/java/org/apache/commons/dbcp2/PoolableConnectionFactory.java
@@ -99,7 +99,7 @@ public class PoolableConnectionFactory implements 
PooledObjectFactory<PoolableCo
     private Duration defaultQueryTimeoutDuration;
 
     /**
-     * Creates a new {@code PoolableConnectionFactory}.
+     * Creates a new {@link PoolableConnectionFactory}.
      *
      * @param connFactory
      *            the {@link ConnectionFactory} from which to obtain base 
{@link Connection}s
diff --git a/src/main/java/org/apache/commons/dbcp2/PoolingConnection.java 
b/src/main/java/org/apache/commons/dbcp2/PoolingConnection.java
index 89d99055..60e02ae1 100644
--- a/src/main/java/org/apache/commons/dbcp2/PoolingConnection.java
+++ b/src/main/java/org/apache/commons/dbcp2/PoolingConnection.java
@@ -169,7 +169,7 @@ public class PoolingConnection extends 
DelegatingConnection<Connection>
      *            the SQL string used to define the statement
      * @param autoGeneratedKeys
      *            A flag indicating whether auto-generated keys should be 
returned; one of
-     *            {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *            {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      *
      * @return the PStmtKey created for the given arguments.
      */
@@ -514,7 +514,7 @@ public class PoolingConnection extends 
DelegatingConnection<Connection>
      *            the SQL string used to define the PreparedStatement
      * @param autoGeneratedKeys
      *            A flag indicating whether auto-generated keys should be 
returned; one of
-     *            {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *            {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @return a {@link PoolablePreparedStatement}
      * @throws SQLException
      *             Wraps an underlying exception.
diff --git a/src/main/java/org/apache/commons/dbcp2/PoolingDataSource.java 
b/src/main/java/org/apache/commons/dbcp2/PoolingDataSource.java
index 35d8e57d..d72ddb90 100644
--- a/src/main/java/org/apache/commons/dbcp2/PoolingDataSource.java
+++ b/src/main/java/org/apache/commons/dbcp2/PoolingDataSource.java
@@ -129,7 +129,7 @@ public class PoolingDataSource<C extends Connection> 
implements DataSource, Auto
     }
 
     /**
-     * Returns a {@link java.sql.Connection} from my pool, according to the 
contract specified by
+     * Returns a {@link Connection} from my pool, according to the contract 
specified by
      * {@link ObjectPool#borrowObject}.
      */
     @Override
diff --git 
a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/ConnectionImpl.java 
b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/ConnectionImpl.java
index bf97c28e..5009d0cc 100644
--- a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/ConnectionImpl.java
+++ b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/ConnectionImpl.java
@@ -19,6 +19,7 @@ package org.apache.commons.dbcp2.cpdsadapter;
 import java.sql.CallableStatement;
 import java.sql.Connection;
 import java.sql.PreparedStatement;
+import java.sql.ResultSet;
 import java.sql.SQLException;
 
 import org.apache.commons.dbcp2.DelegatingCallableStatement;
@@ -26,9 +27,9 @@ import org.apache.commons.dbcp2.DelegatingConnection;
 import org.apache.commons.dbcp2.DelegatingPreparedStatement;
 
 /**
- * This class is the {@code Connection} that will be returned from
- * {@code PooledConnectionImpl.getConnection()}. Most methods are wrappers 
around the JDBC 1.x
- * {@code Connection}. A few exceptions include preparedStatement and close. 
In accordance with the JDBC
+ * This class is the {@link Connection} that will be returned from
+ * {@link PooledConnectionImpl#getConnection()}. Most methods are wrappers 
around the JDBC 1.x
+ * {@link Connection}. A few exceptions include preparedStatement and close. 
In accordance with the JDBC
  * specification this Connection cannot be used after closed() is called. Any 
further usage will result in an
  * SQLException.
  * <p>
@@ -131,13 +132,13 @@ final class ConnectionImpl extends 
DelegatingConnection<Connection> {
     }
 
     /**
-     * If pooling of {@code CallableStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
-     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
java.sql.Connection}.
+     * If pooling of {@link CallableStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
+     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
Connection}.
      *
      * @param sql
      *            an SQL statement that may contain one or more '?' parameter 
placeholders. Typically, this statement is
      *            specified using JDBC call escape syntax.
-     * @return a default {@code CallableStatement} object containing the 
pre-compiled SQL statement.
+     * @return a default {@link CallableStatement} object containing the 
pre-compiled SQL statement.
      * @throws SQLException
      *                Thrown if a database access error occurs or this method 
is called on a closed connection.
      * @since 2.4.0
@@ -154,23 +155,23 @@ final class ConnectionImpl extends 
DelegatingConnection<Connection> {
     }
 
     /**
-     * If pooling of {@code CallableStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
-     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
java.sql.Connection}.
+     * If pooling of {@link CallableStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
+     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
Connection}.
      *
      * @param sql
-     *            a {@code String} object that is the SQL statement to be sent 
to the database; may contain on or
+     *            a {@link String} object that is the SQL statement to be sent 
to the database; may contain on or
      *            more '?' parameters.
      * @param resultSetType
-     *            a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            a concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
-     * @return a {@code CallableStatement} object containing the pre-compiled 
SQL statement that will produce
-     *         {@code ResultSet} objects with the given type and concurrency.
+     *            a concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
+     * @return a {@link CallableStatement} object containing the pre-compiled 
SQL statement that will produce
+     *         {@link ResultSet} objects with the given type and concurrency.
      * @throws SQLException
      *             Thrown if a database access error occurs, this method is 
called on a closed connection or the given
-     *             parameters are not {@code ResultSet} constants indicating 
type and concurrency.
+     *             parameters are not {@link ResultSet} constants indicating 
type and concurrency.
      * @since 2.4.0
      */
     @Override
@@ -187,26 +188,26 @@ final class ConnectionImpl extends 
DelegatingConnection<Connection> {
     }
 
     /**
-     * If pooling of {@code CallableStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
-     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
java.sql.Connection}.
+     * If pooling of {@link CallableStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
+     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
Connection}.
      *
      * @param sql
-     *            a {@code String} object that is the SQL statement to be sent 
to the database; may contain on or
+     *            a {@link String} object that is the SQL statement to be sent 
to the database; may contain on or
      *            more '?' parameters.
      * @param resultSetType
-     *            one of the following {@code ResultSet} constants: {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            one of the following {@link ResultSet} constants: {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            one of the following {@code ResultSet} constants: {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            one of the following {@link ResultSet} constants: {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      * @param resultSetHoldability
-     *            one of the following {@code ResultSet} constants: {@code 
ResultSet.HOLD_CURSORS_OVER_COMMIT}
-     *            or {@code ResultSet.CLOSE_CURSORS_AT_COMMIT}.
-     * @return a new {@code CallableStatement} object, containing the 
pre-compiled SQL statement, that will
-     *         generate {@code ResultSet} objects with the given type, 
concurrency, and holdability.
+     *            one of the following {@link ResultSet} constants: {@link 
ResultSet#HOLD_CURSORS_OVER_COMMIT}
+     *            or {@link ResultSet#CLOSE_CURSORS_AT_COMMIT}.
+     * @return a new {@link CallableStatement} object, containing the 
pre-compiled SQL statement, that will
+     *         generate {@link ResultSet} objects with the given type, 
concurrency, and holdability.
      * @throws SQLException
      *             Thrown if a database access error occurs, this method is 
called on a closed connection or the given
-     *             parameters are not {@code ResultSet} constants indicating 
type, concurrency, and holdability.
+     *             parameters are not {@link ResultSet} constants indicating 
type, concurrency, and holdability.
      * @since 2.4.0
      */
     @Override
@@ -223,8 +224,8 @@ final class ConnectionImpl extends 
DelegatingConnection<Connection> {
     }
 
     /**
-     * If pooling of {@code PreparedStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
-     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
java.sql.Connection}.
+     * If pooling of {@link PreparedStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
+     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
Connection}.
      *
      * @param sql
      *            SQL statement to be prepared
@@ -259,8 +260,8 @@ final class ConnectionImpl extends 
DelegatingConnection<Connection> {
     //
 
     /**
-     * If pooling of {@code PreparedStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
-     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
java.sql.Connection}.
+     * If pooling of {@link PreparedStatement}s is turned on in the {@link 
DriverAdapterCPDS}, a pooled object may
+     * be returned, otherwise delegate to the wrapped JDBC 1.x {@link 
Connection}.
      *
      * @throws SQLException
      *             if this connection is closed or an error occurs in the 
wrapped connection.
diff --git 
a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/DriverAdapterCPDS.java 
b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/DriverAdapterCPDS.java
index 38bb07d7..db1340da 100644
--- a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/DriverAdapterCPDS.java
+++ b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/DriverAdapterCPDS.java
@@ -18,7 +18,9 @@ package org.apache.commons.dbcp2.cpdsadapter;
 
 import java.io.PrintWriter;
 import java.io.Serializable;
+import java.sql.Connection;
 import java.sql.DriverManager;
+import java.sql.PreparedStatement;
 import java.sql.SQLException;
 import java.sql.SQLFeatureNotSupportedException;
 import java.time.Duration;
@@ -35,6 +37,7 @@ import javax.naming.Referenceable;
 import javax.naming.StringRefAddr;
 import javax.naming.spi.ObjectFactory;
 import javax.sql.ConnectionPoolDataSource;
+import javax.sql.DataSource;
 import javax.sql.PooledConnection;
 
 import org.apache.commons.dbcp2.Constants;
@@ -49,22 +52,22 @@ import 
org.apache.commons.pool2.impl.GenericKeyedObjectPoolConfig;
 /**
  * <p>
  * An adapter for JDBC drivers that do not include an implementation of {@link 
javax.sql.ConnectionPoolDataSource}, but
- * still include a {@link java.sql.DriverManager} implementation. {@code 
ConnectionPoolDataSource}s are not used
- * within general applications. They are used by {@code DataSource} 
implementations that pool
- * {@code Connection}s, such as {@link 
org.apache.commons.dbcp2.datasources.SharedPoolDataSource}. A J2EE container
- * will normally provide some method of initializing the {@code 
ConnectionPoolDataSource} whose attributes are
+ * still include a {@link java.sql.DriverManager} implementation. {@link 
ConnectionPoolDataSource}s are not used
+ * within general applications. They are used by {@link DataSource} 
implementations that pool
+ * {@link Connection}s, such as {@link 
org.apache.commons.dbcp2.datasources.SharedPoolDataSource}. A J2EE container
+ * will normally provide some method of initializing the {@link 
ConnectionPoolDataSource} whose attributes are
  * presented as bean getters/setters and then deploying it via JNDI. It is 
then available as a source of physical
- * connections to the database, when the pooling {@code DataSource} needs to 
create a new physical connection.
+ * connections to the database, when the pooling {@link DataSource} needs to 
create a new physical connection.
  * </p>
  * <p>
  * Although normally used within a JNDI environment, the DriverAdapterCPDS can 
be instantiated and initialized as any
- * bean and then attached directly to a pooling {@code DataSource}. {@code 
Jdbc2PoolDataSource} can use the
- * {@code ConnectionPoolDataSource} with or without the use of JNDI.
+ * bean and then attached directly to a pooling {@link DataSource}. {@code 
Jdbc2PoolDataSource} can use the
+ * {@link ConnectionPoolDataSource} with or without the use of JNDI.
  * </p>
  * <p>
- * The DriverAdapterCPDS also provides {@code PreparedStatement} pooling which 
is not generally available in jdbc2
- * {@code ConnectionPoolDataSource} implementation, but is addressed within 
the JDBC 3 specification. The
- * {@code PreparedStatement} pool in DriverAdapterCPDS has been in the DBCP 
package for some time, but it has not
+ * The DriverAdapterCPDS also provides {@link PreparedStatement} pooling which 
is not generally available in jdbc2
+ * {@link ConnectionPoolDataSource} implementation, but is addressed within 
the JDBC 3 specification. The
+ * {@link PreparedStatement} pool in DriverAdapterCPDS has been in the DBCP 
package for some time, but it has not
  * undergone extensive testing in the configuration used here. It should be 
considered experimental and can be toggled
  * with the poolPreparedStatements attribute.
  * </p>
@@ -548,7 +551,7 @@ public class DriverAdapterCPDS implements 
ConnectionPoolDataSource, Referenceabl
     }
 
     /**
-     * Tests whether to toggle the pooling of {@code PreparedStatement}s
+     * Tests whether to toggle the pooling of {@link PreparedStatement}s
      *
      * @return value of poolPreparedStatements.
      */
@@ -747,7 +750,7 @@ public class DriverAdapterCPDS implements 
ConnectionPoolDataSource, Referenceabl
     }
 
     /**
-     * Sets whether to toggle the pooling of {@code PreparedStatement}s
+     * Sets whether to toggle the pooling of {@link PreparedStatement}s
      *
      * @param poolPreparedStatements true to pool statements.
      * @throws IllegalStateException if {@link #getPooledConnection()} has 
been called
diff --git 
a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PStmtKeyCPDS.java 
b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PStmtKeyCPDS.java
index c948343a..b04d9b8f 100644
--- a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PStmtKeyCPDS.java
+++ b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PStmtKeyCPDS.java
@@ -17,6 +17,8 @@
 package org.apache.commons.dbcp2.cpdsadapter;
 
 import java.sql.PreparedStatement;
+import java.sql.ResultSet;
+import java.sql.Statement;
 
 import org.apache.commons.dbcp2.PStmtKey;
 
@@ -46,7 +48,7 @@ public class PStmtKeyCPDS extends PStmtKey {
      *            The SQL statement.
      * @param autoGeneratedKeys
      *            A flag indicating whether auto-generated keys should be 
returned; one of
-     *            {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *            {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      */
     public PStmtKeyCPDS(final String sql, final int autoGeneratedKeys) {
         super(sql, null, autoGeneratedKeys);
@@ -58,11 +60,11 @@ public class PStmtKeyCPDS extends PStmtKey {
      * @param sql
      *            The SQL statement.
      * @param resultSetType
-     *            A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      */
     public PStmtKeyCPDS(final String sql, final int resultSetType, final int 
resultSetConcurrency) {
         super(sql, resultSetType, resultSetConcurrency);
@@ -74,14 +76,14 @@ public class PStmtKeyCPDS extends PStmtKey {
      * @param sql
      *            The SQL statement.
      * @param resultSetType
-     *            a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}
+     *            A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}
      * @param resultSetHoldability
-     *            One of the following {@code ResultSet} constants: {@code 
ResultSet.HOLD_CURSORS_OVER_COMMIT}
-     *            or {@code ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     *            One of the following {@link ResultSet} constants: {@link 
ResultSet#HOLD_CURSORS_OVER_COMMIT}
+     *            or {@link ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      */
     public PStmtKeyCPDS(final String sql, final int resultSetType, final int 
resultSetConcurrency,
             final int resultSetHoldability) {
diff --git 
a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PooledConnectionImpl.java 
b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PooledConnectionImpl.java
index 1d27f855..3ba38694 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PooledConnectionImpl.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/cpdsadapter/PooledConnectionImpl.java
@@ -19,6 +19,7 @@ package org.apache.commons.dbcp2.cpdsadapter;
 import java.sql.CallableStatement;
 import java.sql.Connection;
 import java.sql.PreparedStatement;
+import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.ArrayList;
@@ -146,8 +147,8 @@ final class PooledConnectionImpl
     }
 
     /**
-     * Closes the physical connection and marks this {@code PooledConnection} 
so that it may not be used to
-     * generate any more logical {@code Connection}s.
+     * Closes the physical connection and marks this {@link PooledConnection} 
so that it may not be used to
+     * generate any more logical {@link Connection}s.
      *
      * @throws SQLException
      *             Thrown when an error occurs or the connection is already 
closed.
@@ -195,7 +196,7 @@ final class PooledConnectionImpl
      *            The SQL statement.
      * @param autoGeneratedKeys
      *            A flag indicating whether auto-generated keys should be 
returned; one of
-     *            {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *            {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @return a key to uniquely identify a prepared statement.
      */
     protected PStmtKey createKey(final String sql, final int 
autoGeneratedKeys) {
@@ -208,11 +209,11 @@ final class PooledConnectionImpl
      * @param sql
      *            The SQL statement.
      * @param resultSetType
-     *            A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      * @return a key to uniquely identify a prepared statement.
      */
     protected PStmtKey createKey(final String sql, final int resultSetType, 
final int resultSetConcurrency) {
@@ -225,14 +226,14 @@ final class PooledConnectionImpl
      * @param sql
      *            The SQL statement.
      * @param resultSetType
-     *            a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}
+     *            A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}
      * @param resultSetHoldability
-     *            One of the following {@code ResultSet} constants: {@code 
ResultSet.HOLD_CURSORS_OVER_COMMIT}
-     *            or {@code ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     *            One of the following {@link ResultSet} constants: {@link 
ResultSet#HOLD_CURSORS_OVER_COMMIT}
+     *            or {@link ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      * @return a key to uniquely identify a prepared statement.
      */
     protected PStmtKey createKey(final String sql, final int resultSetType, 
final int resultSetConcurrency, final int resultSetHoldability) {
@@ -245,14 +246,14 @@ final class PooledConnectionImpl
      * @param sql
      *            The SQL statement.
      * @param resultSetType
-     *            a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}
+     *            a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}
      * @param resultSetConcurrency
-     *            A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      * @param resultSetHoldability
-     *            One of the following {@code ResultSet} constants: {@code 
ResultSet.HOLD_CURSORS_OVER_COMMIT}
-     *            or {@code ResultSet.CLOSE_CURSORS_AT_COMMIT}.
+     *            One of the following {@link ResultSet} constants: {@link 
ResultSet#HOLD_CURSORS_OVER_COMMIT}
+     *            or {@link ResultSet#CLOSE_CURSORS_AT_COMMIT}.
      * @param statementType
      *            The SQL statement type, prepared or callable.
      * @return a key to uniquely identify a prepared statement.
@@ -269,11 +270,11 @@ final class PooledConnectionImpl
      * @param sql
      *            The SQL statement.
      * @param resultSetType
-     *            A result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            A result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            A concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            A concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      * @param statementType
      *            The SQL statement type, prepared or callable.
      * @return a key to uniquely identify a prepared statement.
@@ -470,7 +471,7 @@ final class PooledConnectionImpl
      * @param sql
      *            an SQL statement that may contain one or more '?' parameter 
placeholders. Typically, this statement is
      *            specified using JDBC call escape syntax.
-     * @return a default {@code CallableStatement} object containing the 
pre-compiled SQL statement.
+     * @return a default {@link CallableStatement} object containing the 
pre-compiled SQL statement.
      * @throws SQLException
      *                Thrown if a database access error occurs or this method 
is called on a closed connection.
      * @since 2.4.0
@@ -493,19 +494,19 @@ final class PooledConnectionImpl
      * Creates or obtains a {@link CallableStatement} from my pool.
      *
      * @param sql
-     *            a {@code String} object that is the SQL statement to be sent 
to the database; may contain on or
+     *            a {@link String} object that is the SQL statement to be sent 
to the database; may contain on or
      *            more '?' parameters.
      * @param resultSetType
-     *            a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            a concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
-     * @return a {@code CallableStatement} object containing the pre-compiled 
SQL statement that will produce
-     *         {@code ResultSet} objects with the given type and concurrency.
+     *            a concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
+     * @return a {@link CallableStatement} object containing the pre-compiled 
SQL statement that will produce
+     *         {@link ResultSet} objects with the given type and concurrency.
      * @throws SQLException
      *             Thrown if a database access error occurs, this method is 
called on a closed connection or the given
-     *             parameters are not {@code ResultSet} constants indicating 
type and concurrency.
+     *             parameters are not {@link ResultSet} constants indicating 
type and concurrency.
      * @since 2.4.0
      */
     @SuppressWarnings("resource") // getRawConnection() does not allocate
@@ -528,22 +529,22 @@ final class PooledConnectionImpl
      * Creates or obtains a {@link CallableStatement} from my pool.
      *
      * @param sql
-     *            a {@code String} object that is the SQL statement to be sent 
to the database; may contain on or
+     *            a {@link String} object that is the SQL statement to be sent 
to the database; may contain on or
      *            more '?' parameters.
      * @param resultSetType
-     *            one of the following {@code ResultSet} constants: {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            one of the following {@link ResultSet} constants: {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            one of the following {@code ResultSet} constants: {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            one of the following {@link ResultSet} constants: {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      * @param resultSetHoldability
-     *            one of the following {@code ResultSet} constants: {@code 
ResultSet.HOLD_CURSORS_OVER_COMMIT}
-     *            or {@code ResultSet.CLOSE_CURSORS_AT_COMMIT}.
-     * @return a new {@code CallableStatement} object, containing the 
pre-compiled SQL statement, that will
-     *         generate {@code ResultSet} objects with the given type, 
concurrency, and holdability.
+     *            one of the following {@link ResultSet} constants: {@link 
ResultSet#HOLD_CURSORS_OVER_COMMIT}
+     *            or {@link ResultSet#CLOSE_CURSORS_AT_COMMIT}.
+     * @return a new {@link CallableStatement} object, containing the 
pre-compiled SQL statement, that will
+     *         generate {@link ResultSet} objects with the given type, 
concurrency, and holdability.
      * @throws SQLException
      *             Thrown if a database access error occurs, this method is 
called on a closed connection or the given
-     *             parameters are not {@code ResultSet} constants indicating 
type, concurrency, and holdability.
+     *             parameters are not {@link ResultSet} constants indicating 
type, concurrency, and holdability.
      * @since 2.4.0
      */
     @SuppressWarnings("resource") // getRawConnection() does not allocate
@@ -591,7 +592,7 @@ final class PooledConnectionImpl
      *            an SQL statement that may contain one or more '?' IN 
parameter placeholders.
      * @param autoGeneratedKeys
      *            a flag indicating whether auto-generated keys should be 
returned; one of
-     *            {@code Statement.RETURN_GENERATED_KEYS} or {@code 
Statement.NO_GENERATED_KEYS}.
+     *            {@link Statement#RETURN_GENERATED_KEYS} or {@link 
Statement#NO_GENERATED_KEYS}.
      * @return a {@link PoolablePreparedStatement}
      * @throws SQLException Thrown if a database access error occurs, this 
method is called on a closed connection, or
      *         the borrow failed.
@@ -615,14 +616,14 @@ final class PooledConnectionImpl
      * Creates or obtains a {@link PreparedStatement} from my pool.
      *
      * @param sql
-     *            a {@code String} object that is the SQL statement to be sent 
to the database; may contain one or
+     *            a {@link String} object that is the SQL statement to be sent 
to the database; may contain one or
      *            more '?' IN parameters.
      * @param resultSetType
-     *            a result set type; one of {@code 
ResultSet.TYPE_FORWARD_ONLY},
-     *            {@code ResultSet.TYPE_SCROLL_INSENSITIVE}, or {@code 
ResultSet.TYPE_SCROLL_SENSITIVE}.
+     *            a result set type; one of {@link 
ResultSet#TYPE_FORWARD_ONLY},
+     *            {@link ResultSet#TYPE_SCROLL_INSENSITIVE}, or {@link 
ResultSet#TYPE_SCROLL_SENSITIVE}.
      * @param resultSetConcurrency
-     *            a concurrency type; one of {@code 
ResultSet.CONCUR_READ_ONLY} or
-     *            {@code ResultSet.CONCUR_UPDATABLE}.
+     *            a concurrency type; one of {@link 
ResultSet#CONCUR_READ_ONLY} or
+     *            {@link ResultSet#CONCUR_UPDATABLE}.
      *
      * @return a {@link PoolablePreparedStatement}.
      * @throws SQLException Thrown if a database access error occurs, this 
method is called on a closed connection, or
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/CPDSConnectionFactory.java 
b/src/main/java/org/apache/commons/dbcp2/datasources/CPDSConnectionFactory.java
index 580fb9d7..b00c016e 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/CPDSConnectionFactory.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/CPDSConnectionFactory.java
@@ -25,6 +25,7 @@ import javax.sql.ConnectionEventListener;
 import javax.sql.ConnectionPoolDataSource;
 import javax.sql.PooledConnection;
 
+import org.apache.commons.dbcp2.PoolableConnectionFactory;
 import org.apache.commons.pool2.ObjectPool;
 import org.apache.commons.pool2.PooledObject;
 import org.apache.commons.pool2.PooledObjectFactory;
@@ -44,7 +45,7 @@ final class CPDSConnectionFactory extends 
AbstractConnectionFactory
     private UserPassKey userPassKey;
 
     /**
-     * Creates a new {@code PoolableConnectionFactory}.
+     * Creates a new {@link PoolableConnectionFactory}.
      *
      * @param cpds
      *            the ConnectionPoolDataSource from which to obtain 
PooledConnection's
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSource.java 
b/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSource.java
index 4e6415fc..f7bdd916 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSource.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSource.java
@@ -41,30 +41,30 @@ import org.apache.commons.pool2.impl.GenericObjectPool;
 
 /**
  * <p>
- * The base class for {@code SharedPoolDataSource} and {@code 
PerUserPoolDataSource}. Many of the
+ * The base class for {@link SharedPoolDataSource} and {@link 
PerUserPoolDataSource}. Many of the
  * configuration properties are shared and defined here. This class is 
declared public in order to allow particular
  * usage with commons-beanutils; do not make direct use of it outside of 
<em>commons-dbcp2</em>.
  * </p>
  *
  * <p>
- * A J2EE container will normally provide some method of initializing the 
{@code DataSource} whose attributes are
+ * A J2EE container will normally provide some method of initializing the 
{@link DataSource} whose attributes are
  * presented as bean getters/setters and then deploying it via JNDI. It is 
then available to an application as a source
  * of pooled logical connections to the database. The pool needs a source of 
physical connections. This source is in the
- * form of a {@code ConnectionPoolDataSource} that can be specified via the 
{@link #setDataSourceName(String)} used
+ * form of a {@link ConnectionPoolDataSource} that can be specified via the 
{@link #setDataSourceName(String)} used
  * to lookup the source via JNDI.
  * </p>
  *
  * <p>
  * Although normally used within a JNDI environment, A DataSource can be 
instantiated and initialized as any bean. In
- * this case the {@code ConnectionPoolDataSource} will likely be instantiated 
in a similar manner. This class
+ * this case the {@link ConnectionPoolDataSource} will likely be instantiated 
in a similar manner. This class
  * allows the physical source of connections to be attached directly to this 
pool using the
  * {@link #setConnectionPoolDataSource(ConnectionPoolDataSource)} method.
  * </p>
  *
  * <p>
  * The dbcp package contains an adapter, {@link 
org.apache.commons.dbcp2.cpdsadapter.DriverAdapterCPDS}, that can be
- * used to allow the use of {@code DataSource}'s based on this class with JDBC 
driver implementations that do not
- * supply a {@code ConnectionPoolDataSource}, but still provide a {@link 
java.sql.Driver} implementation.
+ * used to allow the use of {@link DataSource}'s based on this class with JDBC 
driver implementations that do not
+ * supply a {@link ConnectionPoolDataSource}, but still provide a {@link 
java.sql.Driver} implementation.
  * </p>
  *
  * <p>
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSourceFactory.java
 
b/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSourceFactory.java
index 8fca509f..81a8971e 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSourceFactory.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/InstanceKeyDataSourceFactory.java
@@ -37,7 +37,7 @@ import org.apache.commons.dbcp2.ListException;
 import org.apache.commons.dbcp2.Utils;
 
 /**
- * A JNDI ObjectFactory which creates {@code SharedPoolDataSource}s or {@code 
PerUserPoolDataSource}s
+ * A JNDI ObjectFactory which creates {@link SharedPoolDataSource}s or {@link 
PerUserPoolDataSource}s
  *
  * @since 2.0
  */
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/KeyedCPDSConnectionFactory.java
 
b/src/main/java/org/apache/commons/dbcp2/datasources/KeyedCPDSConnectionFactory.java
index f029a34d..06b2b5d1 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/KeyedCPDSConnectionFactory.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/KeyedCPDSConnectionFactory.java
@@ -45,7 +45,7 @@ final class KeyedCPDSConnectionFactory extends 
AbstractConnectionFactory
     private KeyedObjectPool<UserPassKey, PooledConnectionAndInfo> pool;
 
     /**
-     * Creates a new {@code KeyedPoolableConnectionFactory}.
+     * Creates a new {@code KeyedCPDSConnectionFactory}.
      *
      * @param cpds
      *            the ConnectionPoolDataSource from which to obtain 
PooledConnections
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSource.java 
b/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSource.java
index f12ea256..fddfc98e 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSource.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSource.java
@@ -30,6 +30,7 @@ import javax.naming.NamingException;
 import javax.naming.Reference;
 import javax.naming.StringRefAddr;
 import javax.sql.ConnectionPoolDataSource;
+import javax.sql.DataSource;
 
 import org.apache.commons.dbcp2.SwallowedExceptionLogger;
 import org.apache.commons.dbcp2.Utils;
@@ -41,7 +42,7 @@ import org.apache.commons.pool2.impl.GenericObjectPool;
 
 /**
  * <p>
- * A pooling {@code DataSource} appropriate for deployment within J2EE 
environment. There are many configuration
+ * A pooling {@link DataSource} appropriate for deployment within J2EE 
environment. There are many configuration
  * options, most of which are defined in the parent class. This datasource 
uses individual pools per user, and some
  * properties can be set specifically for a given user, if the deployment 
environment can support initialization of
  * mapped properties. So for example, a pool of admin or write-access 
Connections can be guaranteed a certain number of
@@ -645,7 +646,7 @@ public class PerUserPoolDataSource extends 
InstanceKeyDataSource {
     }
 
     /**
-     * Returns a {@code PerUserPoolDataSource} {@link Reference}.
+     * Returns a {@link PerUserPoolDataSource} {@link Reference}.
      */
     @Override
     public Reference getReference() throws NamingException {
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSourceFactory.java
 
b/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSourceFactory.java
index e690413c..8677419e 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSourceFactory.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/PerUserPoolDataSourceFactory.java
@@ -24,7 +24,7 @@ import javax.naming.RefAddr;
 import javax.naming.Reference;
 
 /**
- * A JNDI ObjectFactory which creates {@code SharedPoolDataSource}s
+ * A JNDI ObjectFactory which creates {@link SharedPoolDataSource}s
  *
  * @since 2.0
  */
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSource.java 
b/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSource.java
index ff42cd0c..c24b1ccf 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSource.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSource.java
@@ -25,6 +25,7 @@ import javax.naming.NamingException;
 import javax.naming.Reference;
 import javax.naming.StringRefAddr;
 import javax.sql.ConnectionPoolDataSource;
+import javax.sql.DataSource;
 
 import org.apache.commons.dbcp2.PoolableConnection;
 import org.apache.commons.pool2.KeyedObjectPool;
@@ -34,7 +35,7 @@ import 
org.apache.commons.pool2.impl.GenericKeyedObjectPoolConfig;
 
 /**
  * <p>
- * A pooling {@code DataSource} appropriate for deployment within J2EE 
environment. There are many configuration
+ * A pooling {@link DataSource} appropriate for deployment within J2EE 
environment. There are many configuration
  * options, most of which are defined in the parent class. All users (based on 
user name) share a single maximum number
  * of Connections in this data source.
  * </p>
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSourceFactory.java
 
b/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSourceFactory.java
index 8bc6b388..688da7fe 100644
--- 
a/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSourceFactory.java
+++ 
b/src/main/java/org/apache/commons/dbcp2/datasources/SharedPoolDataSourceFactory.java
@@ -20,7 +20,7 @@ import javax.naming.RefAddr;
 import javax.naming.Reference;
 
 /**
- * A JNDI ObjectFactory which creates {@code SharedPoolDataSource}s
+ * A JNDI ObjectFactory which creates {@link SharedPoolDataSource}s
  *
  * @since 2.0
  */
diff --git 
a/src/main/java/org/apache/commons/dbcp2/datasources/UserPassKey.java 
b/src/main/java/org/apache/commons/dbcp2/datasources/UserPassKey.java
index 93f066de..1288cb6e 100644
--- a/src/main/java/org/apache/commons/dbcp2/datasources/UserPassKey.java
+++ b/src/main/java/org/apache/commons/dbcp2/datasources/UserPassKey.java
@@ -26,7 +26,7 @@ import org.apache.commons.pool2.KeyedObjectPool;
  * Holds a user name and password pair. Serves as a poolable object key for 
the {@link KeyedObjectPool} backing a
  * {@link SharedPoolDataSource}. Two instances with the same user name are 
considered equal. This ensures that there
  * will be only one keyed pool for each user in the pool. The password is used 
(along with the user name) by the
- * {@code KeyedCPDSConnectionFactory} when creating new connections.
+ * {@link KeyedCPDSConnectionFactory} when creating new connections.
  * </p>
  *
  * <p>

Reply via email to