Use Javadoc instead of HTML tag

Author: garydgregory

src/main/java/org/apache/commons/beanutils/BaseDynaBeanMapDecorator.java

@@ -25,7 +25,7 @@
 import java.util.Set;
 
 /**
- * <p>A base class for decorators providing <code>Map</code> behavior on
+ * <p>A base class for decorators providing {@code Map} behavior on
  * {@link DynaBean}s.</p>
  *
  * <p>The motivation for this implementation is to provide access to {@link DynaBean}
@@ -107,8 +107,8 @@ public BaseDynaBeanMapDecorator(final DynaBean dynaBean) {
      * Construct a Map for the specified {@link DynaBean}.
      *
      * @param dynaBean The dyna bean being decorated
-     * @param readOnly <code>true</code> if the Map is read only
-     * otherwise <code>false</code>
+     * @param readOnly {@code true} if the Map is read only
+     * otherwise {@code false}
      * @throws IllegalArgumentException if the {@link DynaBean} is null.
      */
     public BaseDynaBeanMapDecorator(final DynaBean dynaBean, final boolean readOnly) {
@@ -134,7 +134,7 @@ public void clear() {
      * value for one (or more) of its properties.
      *
      * @param key The {@link DynaBean}'s property name
-     * @return <code>true</code> if one of the {@link DynaBean}'s
+     * @return {@code true} if one of the {@link DynaBean}'s
      * properties contains a specified value.
      */
     @Override
@@ -149,9 +149,9 @@ public boolean containsKey(final Object key) {
      * a specified value.
      *
      * @param value The value to check for.
-     * @return <code>true</code> if one of the the {@link DynaBean}'s
+     * @return {@code true} if one of the the {@link DynaBean}'s
      * properties contains the specified value, otherwise
-     * <code>false</code>.
+     * {@code false}.
      */
     @Override
     public boolean containsValue(final Object value) {
@@ -182,7 +182,7 @@ public boolean containsValue(final Object value) {
      * <p>Returns the Set of the property/value mappings
      * in the decorated {@link DynaBean}.</p>
      *
-     * <p>Each element in the Set is a <code>Map.Entry</code>
+     * <p>Each element in the Set is a {@code Map.Entry}
      * type.</p>
      *
      * @return An unmodifiable set of the DynaBean
@@ -236,8 +236,8 @@ private DynaProperty[] getDynaProperties() {
      * Indicate whether the decorated {@link DynaBean} has
      * any properties.
      *
-     * @return <code>true</code> if the {@link DynaBean} has
-     * no properties, otherwise <code>false</code>.
+     * @return {@code true} if the {@link DynaBean} has
+     * no properties, otherwise {@code false}.
      */
     @Override
     public boolean isEmpty() {
@@ -247,8 +247,8 @@ public boolean isEmpty() {
     /**
      * Indicate whether the Map is read only.
      *
-     * @return <code>true</code> if the Map is read only,
-     * otherwise <code>false</code>.
+     * @return {@code true} if the Map is read only,
+     * otherwise {@code false}.
      */
     public boolean isReadOnly() {
         return readOnly;
@@ -297,7 +297,7 @@ public Set<K> keySet() {
      * @param value The value for the specified property.
      * @return The previous property's value.
      * @throws UnsupportedOperationException if
-     * <code>isReadOnly()</code> is true.
+     * {@code isReadOnly()} is true.
      */
     @Override
     public Object put(final K key, final Object value) {
@@ -315,7 +315,7 @@ public Object put(final K key, final Object value) {
      *
      * @param map The Map of values to copy.
      * @throws UnsupportedOperationException if
-     * <code>isReadOnly()</code> is true.
+     * {@code isReadOnly()} is true.
      */
     @Override
     public void putAll(final Map<? extends K, ? extends Object> map) {

src/main/java/org/apache/commons/beanutils/BasicDynaBean.java

@@ -25,15 +25,15 @@
 import java.util.Objects;
 
 /**
- * <p>Minimal implementation of the <code>DynaBean</code> interface.  Can be
+ * <p>Minimal implementation of the {@code DynaBean} interface.  Can be
  * used as a convenience base class for more sophisticated implementations.</p>
  *
  * <p><strong>IMPLEMENTATION NOTE</strong> - Instances of this class that are
  * accessed from multiple threads simultaneously need to be synchronized.</p>
  *
  * <p><strong>IMPLEMENTATION NOTE</strong> - Instances of this class can be
  * successfully serialized and deserialized <strong>ONLY</strong> if all
- * property values are <code>Serializable</code>.</p>
+ * property values are {@code Serializable}.</p>
  *
  */
 
@@ -42,7 +42,7 @@ public class BasicDynaBean implements DynaBean, Serializable {
     private static final long serialVersionUID = 1L;
 
     /**
-     * The <code>DynaClass</code> "base class" that this DynaBean
+     * The {@code DynaClass} "base class" that this DynaBean
      * is associated with.
      */
     protected DynaClass dynaClass;
@@ -56,8 +56,8 @@ public class BasicDynaBean implements DynaBean, Serializable {
     private transient Map<String, Object> mapDecorator;
 
     /**
-     * Construct a new <code>DynaBean</code> associated with the specified
-     * <code>DynaClass</code> instance.
+     * Construct a new {@code DynaBean} associated with the specified
+     * {@code DynaClass} instance.
      *
      * @param dynaClass The DynaClass we are associated with
      */
@@ -73,8 +73,8 @@ public BasicDynaBean(final DynaClass dynaClass) {
      *
      * @param name Name of the property to check
      * @param key Name of the key to check
-     * @return <code>true</code> if the mapped property contains a value for
-     * the specified key, otherwise <code>false</code>
+     * @return {@code true} if the mapped property contains a value for
+     * the specified key, otherwise {@code false}
      *
      * @throws IllegalArgumentException if there is no property
      *  of the specified name
@@ -170,7 +170,7 @@ public Object get(final String name, final int index) {
 
     /**
      * Return the value of a mapped property with the specified name,
-     * or <code>null</code> if there is no value for the specified key.
+     * or {@code null} if there is no value for the specified key.
      *
      * @param name Name of the property whose value is to be retrieved
      * @param key Key of the value to be retrieved
@@ -191,7 +191,7 @@ public Object get(final String name, final String key) {
     }
 
     /**
-     * Return the <code>DynaClass</code> instance that describes the set of
+     * Return the {@code DynaClass} instance that describes the set of
      * properties available for this DynaBean.
      *
      * @return The associated DynaClass
@@ -226,9 +226,9 @@ protected DynaProperty getDynaProperty(final String name) {
      * Return a Map representation of this DynaBean.
      * <p>
      * This, for example, could be used in JSTL in the following way to access
-     * a DynaBean's <code>fooProperty</code>:
+     * a DynaBean's {@code fooProperty}:
      * </p>
-     * <ul><li><code>${myDynaBean.<strong>map</strong>.fooProperty}</code></li></ul>
+     * <ul><li>{@code ${myDynaBean.<strong>map</strong>.fooProperty}}</li></ul>
      *
      * @return a Map representation of this DynaBean
      * @since 1.8.0
@@ -248,8 +248,8 @@ public Map<String, Object> getMap() {
      *
      * @param dest Destination class
      * @param source Source class
-     * @return <code>true</code> if the source class is assignable to the
-     * destination class, otherwise <code>false</code>
+     * @return {@code true} if the source class is assignable to the
+     * destination class, otherwise {@code false}
      */
     protected boolean isAssignable(final Class<?> dest, final Class<?> source) {
 

src/main/java/org/apache/commons/beanutils/BasicDynaClass.java

@@ -23,11 +23,11 @@
 import java.util.HashMap;
 
 /**
- * <p>Minimal implementation of the <code>DynaClass</code> interface.  Can be
+ * <p>Minimal implementation of the {@code DynaClass} interface.  Can be
  * used as a convenience base class for more sophisticated implementations.</p> *
- * <p><strong>IMPLEMENTATION NOTE</strong> - The <code>DynaBean</code>
+ * <p><strong>IMPLEMENTATION NOTE</strong> - The {@code DynaBean}
  * implementation class supplied to our constructor MUST have a one-argument
- * constructor of its own that accepts a <code>DynaClass</code>.  This is
+ * constructor of its own that accepts a {@code DynaClass}.  This is
  * used to associate the DynaBean instance with this DynaClass.</p>
  *
  */
@@ -42,7 +42,7 @@ public class BasicDynaClass implements DynaClass, Serializable {
     protected static Class<?>[] constructorTypes = { DynaClass.class };
 
     /**
-     * The constructor of the <code>dynaBeanClass</code> that we will use
+     * The constructor of the {@code dynaBeanClass} that we will use
      * for creating new instances.
      */
     protected transient Constructor<?> constructor;
@@ -54,7 +54,7 @@ public class BasicDynaClass implements DynaClass, Serializable {
     protected Object[] constructorValues = { this };
 
     /**
-     * The <code>DynaBean</code> implementation class we will use for
+     * The {@code DynaBean} implementation class we will use for
      * creating new instances.
      */
     protected Class<?> dynaBeanClass = BasicDynaBean.class;
@@ -72,7 +72,7 @@ public class BasicDynaClass implements DynaClass, Serializable {
     /**
      * The set of dynamic properties that are part of this DynaClass,
      * keyed by the property name.  Individual descriptor instances will
-     * be the same instances as those in the <code>properties</code> list.
+     * be the same instances as those in the {@code properties} list.
      */
     protected HashMap<String, DynaProperty> propertiesMap = new HashMap<>();
 
@@ -116,8 +116,8 @@ public BasicDynaClass(final String name, Class<?> dynaBeanClass,
 
     /**
      * Return the Class object we will use to create new instances in the
-     * <code>newInstance()</code> method.  This Class <strong>MUST</strong>
-     * implement the <code>DynaBean</code> interface.
+     * {@code newInstance()} method.  This Class <strong>MUST</strong>
+     * implement the {@code DynaBean} interface.
      *
      * @return The class of the {@link DynaBean}
      */
@@ -126,12 +126,12 @@ public Class<?> getDynaBeanClass() {
     }
 
     /**
-     * <p>Return an array of <code>ProperyDescriptors</code> for the properties
+     * <p>Return an array of {@code ProperyDescriptors} for the properties
      * currently defined in this DynaClass.  If no properties are defined, a
      * zero-length array will be returned.</p>
      *
      * <p><strong>FIXME</strong> - Should we really be implementing
-     * <code>getBeanInfo()</code> instead, which returns property descriptors
+     * {@code getBeanInfo()} instead, which returns property descriptors
      * and a bunch of other stuff?</p>
      *
      * @return the set of properties for this DynaClass
@@ -143,7 +143,7 @@ public DynaProperty[] getDynaProperties() {
 
     /**
      * Return a property descriptor for the specified property, if it exists;
-     * otherwise, return <code>null</code>.
+     * otherwise, return {@code null}.
      *
      * @param name Name of the dynamic property for which a descriptor
      *  is requested
@@ -161,8 +161,8 @@ public DynaProperty getDynaProperty(final String name) {
 
     /**
      * Return the name of this DynaClass (analogous to the
-     * <code>getName()</code> method of <code>java.lang.Class</code>), which
-     * allows the same <code>DynaClass</code> implementation class to support
+     * {@code getName()} method of {@code java.lang.Class}), which
+     * allows the same {@code DynaClass} implementation class to support
      * different dynamic classes, with different sets of properties.
      *
      * @return the name of the DynaClass
@@ -176,7 +176,7 @@ public String getName() {
      * Instantiate and return a new DynaBean instance, associated
      * with this DynaClass.
      *
-     * @return A new <code>DynaBean</code> instance
+     * @return A new {@code DynaBean} instance
      * @throws IllegalAccessException if the Class or the appropriate
      *  constructor is not accessible
      * @throws InstantiationException if this Class represents an abstract
@@ -201,12 +201,12 @@ public DynaBean newInstance()
 
     /**
      * Set the Class object we will use to create new instances in the
-     * <code>newInstance()</code> method.  This Class <strong>MUST</strong>
-     * implement the <code>DynaBean</code> interface.
+     * {@code newInstance()} method.  This Class <strong>MUST</strong>
+     * implement the {@code DynaBean} interface.
      *
      * @param dynaBeanClass The new Class object
      * @throws IllegalArgumentException if the specified Class does not
-     *  implement the <code>DynaBean</code> interface
+     *  implement the {@code DynaBean} interface
      */
     protected void setDynaBeanClass(final Class<?> dynaBeanClass) {
         // Validate the argument type specified

src/main/java/org/apache/commons/beanutils/BeanAccessLanguageException.java

@@ -30,13 +30,13 @@ public class BeanAccessLanguageException extends IllegalArgumentException {
     private static final long serialVersionUID = 1L;
 
     /**
-     * Constructs a <code>BeanAccessLanguageException</code> without a detail message.
+     * Constructs a {@code BeanAccessLanguageException} without a detail message.
      */
     public BeanAccessLanguageException() {
     }
 
     /**
-     * Constructs a <code>BeanAccessLanguageException</code> without a detail message.
+     * Constructs a {@code BeanAccessLanguageException} without a detail message.
      *
      * @param message the detail message explaining this exception
      */

src/main/java/org/apache/commons/beanutils/BeanComparator.java

@@ -33,8 +33,8 @@
  * <strong>Note:</strong> The BeanComparator passes the values of the specified
  * bean property to a ComparableComparator, if no comparator is
  * specified in the constructor. If you are comparing two beans based
- * on a property that could contain "null" values, a suitable <code>Comparator</code>
- * or <code>ComparatorChain</code> should be supplied in the constructor.
+ * on a property that could contain "null" values, a suitable {@code Comparator}
+ * or {@code ComparatorChain} should be supplied in the constructor.
  * Note that the passed in {@code Comparator} must be able to handle the
  * passed in objects. Because the type of the property to be compared is not
  * known at compile time no type checks can be performed by the compiler.
@@ -77,13 +77,13 @@ public BeanComparator() {
      * <p>Constructs a property-based comparator for beans.
      * This compares two beans by the property
      * specified in the property parameter. This constructor creates
-     * a <code>BeanComparator</code> that uses a <code>ComparableComparator</code>
+     * a {@code BeanComparator} that uses a {@code ComparableComparator}
      * to compare the property values.
      * </p>
      *
      * <p>Passing "null" to this constructor will cause the BeanComparator
      * to compare objects based on natural order, that is
-     * <code>java.lang.Comparable</code>.
+     * {@code java.lang.Comparable}.
      * </p>
      *
      * @param property String Name of a bean property, which may contain the
@@ -144,7 +144,7 @@ public int compare(final T o1, final T o2) {
     }
 
     /**
-     * Two <code>BeanComparator</code>'s are equals if and only if
+     * Two {@code BeanComparator}'s are equals if and only if
      * the wrapped comparators and the property names to be compared
      * are equal.
      * @param  o Comparator to compare to

src/main/java/org/apache/commons/beanutils/BeanIntrospector.java

@@ -27,7 +27,7 @@
  * Before {@link PropertyUtils} can be used for interaction with a specific Java
  * class, the class's properties have to be determined. This is called
  * <em>introspection</em> and is initiated automatically on demand.
- * <code>PropertyUtils</code> does not perform introspection on its own, but
+ * {@code PropertyUtils} does not perform introspection on its own, but
  * delegates this task to one or more objects implementing this interface. This
  * makes it possible to customize introspection which may be useful for certain
  * code bases using non-standard conventions for accessing properties.
@@ -38,7 +38,7 @@
 public interface BeanIntrospector {
     /**
      * Performs introspection on a Java class. The current class to be inspected
-     * can be queried from the passed in <code>IntrospectionContext</code>
+     * can be queried from the passed in {@code IntrospectionContext}
      * object. A typical implementation has to obtain this class, determine its
      * properties according to the rules it implements, and add them to the
      * passed in context object.