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-collections.git
The following commit(s) were added to refs/heads/master by this push:
new dc1aca9 Javadoc: Replace <code></code> HTML tags with Javadoc {@code}
notation.
dc1aca9 is described below
commit dc1aca9ed9bb17cffd546f5bd3f93077ce2f8457
Author: Gary Gregory <[email protected]>
AuthorDate: Mon Dec 9 23:08:03 2019 -0500
Javadoc: Replace <code></code> HTML tags with Javadoc {@code} notation.
---
.../commons/collections4/AbstractObjectTest.java | 2 +-
.../org/apache/commons/collections4/BulkTest.java | 32 +++++++++++-----------
.../apache/commons/collections4/ListUtilsTest.java | 2 +-
.../commons/collections4/MapPerformance.java | 2 +-
.../org/apache/commons/collections4/TestUtils.java | 4 +--
.../collections4/bidimap/AbstractBidiMapTest.java | 4 +--
.../collection/AbstractCollectionTest.java | 18 ++++++------
.../functors/AbstractAnyAllOnePredicateTest.java | 6 ++--
.../functors/AbstractCompositePredicateTest.java | 20 +++++++-------
.../functors/AbstractMockPredicateTest.java | 2 +-
.../collections4/functors/AllPredicateTest.java | 2 +-
.../iterators/ArrayListIteratorTest.java | 2 +-
.../iterators/BoundedIteratorTest.java | 24 ++++++++--------
.../iterators/ObjectArrayListIteratorTest.java | 2 +-
.../iterators/SkippingIteratorTest.java | 16 +++++------
.../collections4/list/AbstractListTest.java | 4 +--
.../commons/collections4/map/AbstractMapTest.java | 20 +++++++-------
.../commons/collections4/map/IdentityMap.java | 8 +++---
.../multimap/AbstractMultiValuedMapTest.java | 10 +++----
19 files changed, 90 insertions(+), 90 deletions(-)
diff --git
a/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java
b/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java
index c473a73..ddb9546 100644
--- a/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java
+++ b/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java
@@ -218,7 +218,7 @@ public abstract class AbstractObjectTest extends BulkTest {
* Subclasses can override this variable, indicating compatibility
* with earlier Collections versions.
*
- * @return The version, or <code>null</code> if this object shouldn't be
+ * @return The version, or {@code null} if this object shouldn't be
* tested for compatibility with previous versions.
*/
public String getCompatibilityVersion() {
diff --git a/src/test/java/org/apache/commons/collections4/BulkTest.java
b/src/test/java/org/apache/commons/collections4/BulkTest.java
index a7c4b7e..bbe4f62 100644
--- a/src/test/java/org/apache/commons/collections4/BulkTest.java
+++ b/src/test/java/org/apache/commons/collections4/BulkTest.java
@@ -33,11 +33,11 @@ import junit.framework.TestSuite;
* A <I>simple test method</I> is the type of test traditionally
* supplied by by {@link TestCase}. To define a simple test, create a public
* no-argument method whose name starts with "test". You can specify the
- * the name of simple test in the constructor of <code>BulkTest</code>;
+ * the name of simple test in the constructor of {@code BulkTest};
* a subsequent call to {@link TestCase#run} will run that simple test.
* <p>
* A <I>bulk test method</I>, on the other hand, returns a new instance
- * of <code>BulkTest</code>, which can itself define new simple and bulk
+ * of {@code BulkTest}, which can itself define new simple and bulk
* test methods. By using the {@link #makeSuite} method, you can
* automatically create a hierarchical suite of tests and child bulk tests.
* <p>
@@ -89,10 +89,10 @@ import junit.framework.TestSuite;
* }
* </Pre>
*
- * In the above examples, <code>SetTest</code> defines two
- * simple test methods and no bulk test methods; <code>HashMapTest</code>
+ * In the above examples, {@code SetTest} defines two
+ * simple test methods and no bulk test methods; {@code HashMapTest}
* defines one simple test method and two bulk test methods. When
- * <code>makeSuite(HashMapTest.class).run</code> is executed,
+ * {@code makeSuite(HashMapTest.class).run} is executed,
* <I>five</I> simple test methods will be run, in this order:<P>
*
* <Ol>
@@ -124,12 +124,12 @@ import junit.framework.TestSuite;
* </UL>
*
* A subclass can override a superclass's bulk test by
- * returning <code>null</code> from the bulk test method. If you only
+ * returning {@code null} from the bulk test method. If you only
* want to override specific simple tests within a bulk test, use the
* {@link #ignoredTests} method.<P>
*
* Note that if you want to use the bulk test methods, you <I>must</I>
- * define your <code>suite()</code> method to use {@link #makeSuite}.
+ * define your {@code suite()} method to use {@link #makeSuite}.
* The ordinary {@link TestSuite} constructor doesn't know how to
* interpret bulk test methods.
*
@@ -159,7 +159,7 @@ public class BulkTest extends TestCase implements Cloneable
{
/**
- * Constructs a new <code>BulkTest</code> instance that will run the
+ * Constructs a new {@code BulkTest} instance that will run the
* specified simple test.
*
* @param name the name of the simple test method to run
@@ -171,9 +171,9 @@ public class BulkTest extends TestCase implements Cloneable
{
/**
- * Creates a clone of this <code>BulkTest</code>.<P>
+ * Creates a clone of this {@code BulkTest}.<P>
*
- * @return a clone of this <code>BulkTest</code>
+ * @return a clone of this {@code BulkTest}
*/
@Override
public Object clone() {
@@ -188,12 +188,12 @@ public class BulkTest extends TestCase implements
Cloneable {
/**
* Returns an array of test names to ignore.<P>
*
- * If a test that's defined by this <code>BulkTest</code> or
+ * If a test that's defined by this {@code BulkTest} or
* by one of its bulk test methods has a name that's in the returned
* array, then that simple test will not be executed.<P>
*
* A test's name is formed by taking the class name of the
- * root <code>BulkTest</code>, eliminating the package name, then
+ * root {@code BulkTest}, eliminating the package name, then
* appending the names of any bulk test methods that were invoked
* to get to the simple test, and then appending the simple test
* method name. The method names are delimited by periods:
@@ -203,7 +203,7 @@ public class BulkTest extends TestCase implements Cloneable
{
* </pre>
*
* is the name of one of the simple tests defined in the sample classes
- * described above. If the sample <code>HashMapTest</code> class
+ * described above. If the sample {@code HashMapTest} class
* included this method:
*
* <pre>
@@ -224,9 +224,9 @@ public class BulkTest extends TestCase implements Cloneable
{
/**
- * Returns the display name of this <code>BulkTest</code>.
+ * Returns the display name of this {@code BulkTest}.
*
- * @return the display name of this <code>BulkTest</code>
+ * @return the display name of this {@code BulkTest}
*/
@Override
public String toString() {
@@ -242,7 +242,7 @@ public class BulkTest extends TestCase implements Cloneable
{
* bulk tests are also examined recursively; and the results are stored
* in a hierarchical {@link TestSuite}.<P>
*
- * The given class must be a subclass of <code>BulkTest</code> and must
+ * The given class must be a subclass of {@code BulkTest} and must
* not be abstract.<P>
*
* @param c the class to examine for simple and bulk tests
diff --git a/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
b/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
index c8ec12d..b46575d 100644
--- a/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
@@ -316,7 +316,7 @@ public class ListUtilsTest {
}
/**
- * Tests the <code>indexOf</code> method in <code>ListUtils</code> class..
+ * Tests the {@code indexOf} method in {@code ListUtils} class..
*/
@Test
public void testIndexOf() {
diff --git a/src/test/java/org/apache/commons/collections4/MapPerformance.java
b/src/test/java/org/apache/commons/collections4/MapPerformance.java
index c0bffcf..0e12b58 100644
--- a/src/test/java/org/apache/commons/collections4/MapPerformance.java
+++ b/src/test/java/org/apache/commons/collections4/MapPerformance.java
@@ -24,7 +24,7 @@ import java.util.Set;
import org.apache.commons.collections4.map.Flat3Map;
/**
- * <code>MapPerformanceTest</code> is designed to perform basic Map
performance tests.
+ * {@code MapPerformanceTest} is designed to perform basic Map performance
tests.
*
*/
public class MapPerformance {
diff --git a/src/test/java/org/apache/commons/collections4/TestUtils.java
b/src/test/java/org/apache/commons/collections4/TestUtils.java
index 0928dbe..00a7d9f 100644
--- a/src/test/java/org/apache/commons/collections4/TestUtils.java
+++ b/src/test/java/org/apache/commons/collections4/TestUtils.java
@@ -38,7 +38,7 @@ public final class TestUtils {
* This method is especially good for testing singleton pattern on classes
* that support serialization.
*
- * @param msg the identifying message for the <code>AssertionError</code>.
+ * @param msg the identifying message for the {@code AssertionError}.
* @param o object that will be tested.
* @see #assertSameAfterSerialization(Object)
*/
@@ -72,7 +72,7 @@ public final class TestUtils {
* one that was serialized.
* <p>
* Effect of method call is the same as:
- * <code>assertSameAfterSerialization(null, o)</code>.
+ * {@code assertSameAfterSerialization(null, o)}.
*
* @param o object that will be tested.
* @see #assertSameAfterSerialization(String, Object)
diff --git
a/src/test/java/org/apache/commons/collections4/bidimap/AbstractBidiMapTest.java
b/src/test/java/org/apache/commons/collections4/bidimap/AbstractBidiMapTest.java
index cf58177..5d9a001 100644
---
a/src/test/java/org/apache/commons/collections4/bidimap/AbstractBidiMapTest.java
+++
b/src/test/java/org/apache/commons/collections4/bidimap/AbstractBidiMapTest.java
@@ -44,9 +44,9 @@ public abstract class AbstractBidiMapTest<K, V> extends
AbstractIterableMapTest<
//-----------------------------------------------------------------------
/**
- * Override to create a full <code>BidiMap</code> other than the default.
+ * Override to create a full {@code BidiMap} other than the default.
*
- * @return a full <code>BidiMap</code> implementation.
+ * @return a full {@code BidiMap} implementation.
*/
@Override
public BidiMap<K, V> makeFullMap() {
diff --git
a/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
b/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
index e62d942..37649ad 100644
---
a/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
+++
b/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
@@ -49,7 +49,7 @@ import org.apache.commons.collections4.AbstractObjectTest;
* <b>Element Population Methods</b>
* <p>
* Override these if your collection restricts what kind of elements are
- * allowed (for instance, if <code>null</code> is not permitted):
+ * allowed (for instance, if {@code null} is not permitted):
* <ul>
* <li>{@link #getFullElements()}
* <li>{@link #getOtherElements()}
@@ -72,12 +72,12 @@ import org.apache.commons.collections4.AbstractObjectTest;
* for the collection. Basically, the operation is performed against your
* collection implementation, and an identical operation is performed against a
* <i>confirmed</i> collection implementation. A confirmed collection
- * implementation is something like <code>java.util.ArrayList</code>, which is
+ * implementation is something like {@code java.util.ArrayList}, which is
* known to conform exactly to its collection interface's contract. After the
* operation takes place on both your collection implementation and the
* confirmed collection implementation, the two collections are compared to see
* if their state is identical. The comparison is usually much more involved
- * than a simple <code>equals</code> test. This verification is used to ensure
+ * than a simple {@code equals} test. This verification is used to ensure
* proper modifications are made along with ensuring that the collection does
* not change when read-only modifications are made.
* <p>
@@ -188,7 +188,7 @@ public abstract class AbstractCollectionTest<E> extends
AbstractObjectTest {
/**
* Returns true if the collections produced by
* {@link #makeObject()} and {@link #makeFullCollection()}
- * support the <code>add</code> and <code>addAll</code>
+ * support the {@code add} and {@code addAll}
* operations.<P>
* Default implementation returns true. Override if your collection
* class does not support add or addAll.
@@ -200,9 +200,9 @@ public abstract class AbstractCollectionTest<E> extends
AbstractObjectTest {
/**
* Returns true if the collections produced by
* {@link #makeObject()} and {@link #makeFullCollection()}
- * support the <code>remove</code>, <code>removeAll</code>,
- * <code>retainAll</code>, <code>clear</code> and
- * <code>iterator().remove()</code> methods.
+ * support the {@code remove}, {@code removeAll},
+ * {@code retainAll}, {@code clear} and
+ * {@code iterator().remove()} methods.
* Default implementation returns true. Override if your collection
* class does not support removal operations.
*/
@@ -358,7 +358,7 @@ public abstract class AbstractCollectionTest<E> extends
AbstractObjectTest {
* Returns a full collection to be used for testing. The collection
* returned by this method should contain every element returned by
* {@link #getFullElements()}. The default implementation, in fact,
- * simply invokes <code>addAll</code> on an empty collection with
+ * simply invokes {@code addAll} on an empty collection with
* the results of {@link #getFullElements()}. Override this default
* if your collection doesn't support addAll.
*/
@@ -1170,7 +1170,7 @@ public abstract class AbstractCollectionTest<E> extends
AbstractObjectTest {
}
/**
- * Tests <code>toString</code> on a collection.
+ * Tests {@code toString} on a collection.
*/
public void testCollectionToString() {
resetEmpty();
diff --git
a/src/test/java/org/apache/commons/collections4/functors/AbstractAnyAllOnePredicateTest.java
b/src/test/java/org/apache/commons/collections4/functors/AbstractAnyAllOnePredicateTest.java
index 12cceb2..cbb5f5b 100644
---
a/src/test/java/org/apache/commons/collections4/functors/AbstractAnyAllOnePredicateTest.java
+++
b/src/test/java/org/apache/commons/collections4/functors/AbstractAnyAllOnePredicateTest.java
@@ -33,7 +33,7 @@ import java.util.Collections;
public abstract class AbstractAnyAllOnePredicateTest<T> extends
AbstractCompositePredicateTest<T> {
/**
- * Creates a new <code>TestCompositePredicate</code>.
+ * Creates a new {@code TestCompositePredicate}.
*
* @param testValue the value which the mock predicates should expect to
see (may be null).
*/
@@ -42,7 +42,7 @@ public abstract class AbstractAnyAllOnePredicateTest<T>
extends AbstractComposit
}
/**
- * Tests whether <code>getInstance</code> with a one element array returns
the first element in the array.
+ * Tests whether {@code getInstance} with a one element array returns the
first element in the array.
*/
@Override
@SuppressWarnings("unchecked")
@@ -54,7 +54,7 @@ public abstract class AbstractAnyAllOnePredicateTest<T>
extends AbstractComposit
}
/**
- * Tests that passing a singleton collection to <code>getInstance</code>
returns the single element in the
+ * Tests that passing a singleton collection to {@code getInstance}
returns the single element in the
* collection.
*/
@Override
diff --git
a/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java
b/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java
index 978613b..db92a64 100644
---
a/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java
+++
b/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java
@@ -33,7 +33,7 @@ import org.junit.Test;
public abstract class AbstractCompositePredicateTest<T> extends
AbstractMockPredicateTest<T> {
/**
- * Creates a new <code>TestCompositePredicate</code>.
+ * Creates a new {@code TestCompositePredicate}.
*
* @param testValue the value which the mock predicates should expect to
see (may be null).
*/
@@ -44,7 +44,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
/**
* Creates an instance of the predicate to test.
*
- * @param predicates the arguments to <code>getInstance</code>.
+ * @param predicates the arguments to {@code getInstance}.
*
* @return a predicate to test.
*/
@@ -53,7 +53,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
/**
* Creates an instance of the predicate to test.
*
- * @param predicates the argument to <code>getInstance</code>.
+ * @param predicates the argument to {@code getInstance}.
*
* @return a predicate to test.
*/
@@ -76,7 +76,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
}
/**
- * Tests whether <code>getInstance</code> with a one element array returns
the first element in the array.
+ * Tests whether {@code getInstance} with a one element array returns the
first element in the array.
*/
@SuppressWarnings("unchecked")
public void singleElementArrayToGetInstance() {
@@ -86,7 +86,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
}
/**
- * Tests that passing a singleton collection to <code>getInstance</code>
returns the single element in the
+ * Tests that passing a singleton collection to {@code getInstance}
returns the single element in the
* collection.
*/
public void singletonCollectionToGetInstance() {
@@ -97,7 +97,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
}
/**
- * Tests <code>getInstance</code> with a null predicate array.
+ * Tests {@code getInstance} with a null predicate array.
*/
@Test(expected = NullPointerException.class)
public final void nullArrayToGetInstance() {
@@ -105,7 +105,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
}
/**
- * Tests <code>getInstance</code> with a single null element in the
predicate array.
+ * Tests {@code getInstance} with a single null element in the predicate
array.
*/
@SuppressWarnings({"unchecked"})
@Test(expected = NullPointerException.class)
@@ -114,7 +114,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
}
/**
- * Tests <code>getInstance</code> with two null elements in the predicate
array.
+ * Tests {@code getInstance} with two null elements in the predicate array.
*/
@SuppressWarnings({"unchecked"})
@Test(expected = NullPointerException.class)
@@ -124,7 +124,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
/**
- * Tests <code>getInstance</code> with a null predicate collection
+ * Tests {@code getInstance} with a null predicate collection
*/
@Test(expected = NullPointerException.class)
public final void nullCollectionToGetInstance() {
@@ -132,7 +132,7 @@ public abstract class AbstractCompositePredicateTest<T>
extends AbstractMockPred
}
/**
- * Tests <code>getInstance</code> with a predicate collection that
contains null elements
+ * Tests {@code getInstance} with a predicate collection that contains
null elements
*/
@Test(expected = NullPointerException.class)
public final void nullElementsInCollectionToGetInstance() {
diff --git
a/src/test/java/org/apache/commons/collections4/functors/AbstractMockPredicateTest.java
b/src/test/java/org/apache/commons/collections4/functors/AbstractMockPredicateTest.java
index 8a4aedd..d67cfeb 100644
---
a/src/test/java/org/apache/commons/collections4/functors/AbstractMockPredicateTest.java
+++
b/src/test/java/org/apache/commons/collections4/functors/AbstractMockPredicateTest.java
@@ -44,7 +44,7 @@ public abstract class AbstractMockPredicateTest<T> {
private final T testValue;
/**
- * Creates a new <code>PredicateTestBase</code>.
+ * Creates a new {@code PredicateTestBase}.
*
* @param testValue the value to pass to mock predicates.
*/
diff --git
a/src/test/java/org/apache/commons/collections4/functors/AllPredicateTest.java
b/src/test/java/org/apache/commons/collections4/functors/AllPredicateTest.java
index 1710645..4baed93 100644
---
a/src/test/java/org/apache/commons/collections4/functors/AllPredicateTest.java
+++
b/src/test/java/org/apache/commons/collections4/functors/AllPredicateTest.java
@@ -35,7 +35,7 @@ import java.util.Collections;
public class AllPredicateTest extends AbstractAnyAllOnePredicateTest<Integer> {
/**
- * Creates a new <code>TestAllPredicate</code>.
+ * Creates a new {@code TestAllPredicate}.
*/
public AllPredicateTest() {
super(42);
diff --git
a/src/test/java/org/apache/commons/collections4/iterators/ArrayListIteratorTest.java
b/src/test/java/org/apache/commons/collections4/iterators/ArrayListIteratorTest.java
index f738116..0cf8bf3 100644
---
a/src/test/java/org/apache/commons/collections4/iterators/ArrayListIteratorTest.java
+++
b/src/test/java/org/apache/commons/collections4/iterators/ArrayListIteratorTest.java
@@ -51,7 +51,7 @@ public class ArrayListIteratorTest<E> extends
ArrayIteratorTest<E> {
/**
* Test the basic ListIterator functionality - going backwards using
- * <code>previous()</code>.
+ * {@code previous()}.
*/
public void testListIterator() {
final ListIterator<E> iter = makeObject();
diff --git
a/src/test/java/org/apache/commons/collections4/iterators/BoundedIteratorTest.java
b/src/test/java/org/apache/commons/collections4/iterators/BoundedIteratorTest.java
index 7cf6372..9d5b55f 100644
---
a/src/test/java/org/apache/commons/collections4/iterators/BoundedIteratorTest.java
+++
b/src/test/java/org/apache/commons/collections4/iterators/BoundedIteratorTest.java
@@ -86,8 +86,8 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test a decorated iterator bounded such that the <code>offset</code> is
- * zero and the <code>max</code> is its size, in that the BoundedIterator
+ * Test a decorated iterator bounded such that the {@code offset} is
+ * zero and the {@code max} is its size, in that the BoundedIterator
* should return all the same elements as its decorated iterator.
*/
@Test
@@ -119,7 +119,7 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test a decorated iterator bounded to a <code>max</code> of 0. The
+ * Test a decorated iterator bounded to a {@code max} of 0. The
* BoundedIterator should behave as if there are no more elements to
return,
* since it is technically an empty iterator.
*/
@@ -135,7 +135,7 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the case if a negative <code>offset</code> is passed to the
+ * Test the case if a negative {@code offset} is passed to the
* constructor. {@link IllegalArgumentException} is expected.
*/
@Test
@@ -148,7 +148,7 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the case if a negative <code>max</code> is passed to the
+ * Test the case if a negative {@code max} is passed to the
* constructor. {@link IllegalArgumentException} is expected.
*/
@Test
@@ -161,7 +161,7 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the case if the <code>offset</code> passed to the constructor is
+ * Test the case if the {@code offset} passed to the constructor is
* greater than the decorated iterator's size. The BoundedIterator should
* behave as if there are no more elements to return.
*/
@@ -177,7 +177,7 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the case if the <code>max</code> passed to the constructor is
+ * Test the case if the {@code max} passed to the constructor is
* greater than the size of the decorated iterator. The last element
* returned should be the same as the last element of the decorated
* iterator.
@@ -208,8 +208,8 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the <code>remove()</code> method being called without
- * <code>next()</code> being called first.
+ * Test the {@code remove()} method being called without
+ * {@code next()} being called first.
*/
@Test
public void testRemoveWithoutCallingNext() {
@@ -224,8 +224,8 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the <code>remove()</code> method being called twice without calling
- * <code>next()</code> in between.
+ * Test the {@code remove()} method being called twice without calling
+ * {@code next()} in between.
*/
@Test
public void testRemoveCalledTwice() {
@@ -347,7 +347,7 @@ public class BoundedIteratorTest<E> extends
AbstractIteratorTest<E> {
/**
* Test the case if the decorated iterator does not support the
- * <code>remove()</code> method and throws an {@link
UnsupportedOperationException}.
+ * {@code remove()} method and throws an {@link
UnsupportedOperationException}.
*/
@Test
public void testRemoveUnsupported() {
diff --git
a/src/test/java/org/apache/commons/collections4/iterators/ObjectArrayListIteratorTest.java
b/src/test/java/org/apache/commons/collections4/iterators/ObjectArrayListIteratorTest.java
index e2d0003..7d9975d 100644
---
a/src/test/java/org/apache/commons/collections4/iterators/ObjectArrayListIteratorTest.java
+++
b/src/test/java/org/apache/commons/collections4/iterators/ObjectArrayListIteratorTest.java
@@ -48,7 +48,7 @@ public class ObjectArrayListIteratorTest<E> extends
ObjectArrayIteratorTest<E> {
/**
* Test the basic ListIterator functionality - going backwards using
- * <code>previous()</code>.
+ * {@code previous()}.
*/
public void testListIterator() {
final ListIterator<E> iter = makeObject();
diff --git
a/src/test/java/org/apache/commons/collections4/iterators/SkippingIteratorTest.java
b/src/test/java/org/apache/commons/collections4/iterators/SkippingIteratorTest.java
index c0ebf41..565d5b7 100644
---
a/src/test/java/org/apache/commons/collections4/iterators/SkippingIteratorTest.java
+++
b/src/test/java/org/apache/commons/collections4/iterators/SkippingIteratorTest.java
@@ -88,7 +88,7 @@ public class SkippingIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test a decorated iterator bounded such that the <code>offset</code> is
+ * Test a decorated iterator bounded such that the {@code offset} is
* zero, in that the SkippingIterator should return all the same elements
* as its decorated iterator.
*/
@@ -120,7 +120,7 @@ public class SkippingIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the case if the <code>offset</code> passed to the constructor is
+ * Test the case if the {@code offset} passed to the constructor is
* greater than the decorated iterator's size. The SkippingIterator should
* behave as if there are no more elements to return.
*/
@@ -136,7 +136,7 @@ public class SkippingIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the case if a negative <code>offset</code> is passed to the
+ * Test the case if a negative {@code offset} is passed to the
* constructor. {@link IllegalArgumentException} is expected.
*/
@Test
@@ -149,8 +149,8 @@ public class SkippingIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the <code>remove()</code> method being called without
- * <code>next()</code> being called first.
+ * Test the {@code remove()} method being called without
+ * {@code next()} being called first.
*/
@Test
public void testRemoveWithoutCallingNext() {
@@ -165,8 +165,8 @@ public class SkippingIteratorTest<E> extends
AbstractIteratorTest<E> {
}
/**
- * Test the <code>remove()</code> method being called twice without calling
- * <code>next()</code> in between.
+ * Test the {@code remove()} method being called twice without calling
+ * {@code next()} in between.
*/
@Test
public void testRemoveCalledTwice() {
@@ -276,7 +276,7 @@ public class SkippingIteratorTest<E> extends
AbstractIteratorTest<E> {
/**
* Test the case if the decorated iterator does not support the
- * <code>remove()</code> method and throws an {@link
UnsupportedOperationException}.
+ * {@code remove()} method and throws an {@link
UnsupportedOperationException}.
*/
@Test
public void testRemoveUnsupported() {
diff --git
a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
index 15b608c..1af0796 100644
--- a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
+++ b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
@@ -1120,8 +1120,8 @@ public abstract class AbstractListTest<E> extends
AbstractCollectionTest<E> {
//-----------------------------------------------------------------------
/**
* Returns a {@link BulkTest} for testing {@link List#subList(int,int)}.
- * The returned bulk test will run through every <code>TestList</code>
- * method, <i>including</i> another <code>bulkTestSubList</code>.
+ * The returned bulk test will run through every {@code TestList}
+ * method, <i>including</i> another {@code bulkTestSubList}.
* Sublists are tested until the size of the sublist is less than 10.
* Each sublist is 6 elements smaller than its parent list.
* (By default this means that two rounds of sublists will be tested).
diff --git
a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
index 5724bd3..1ef9523 100644
--- a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
@@ -117,7 +117,7 @@ import org.apache.commons.collections4.set.AbstractSetTest;
* this base set of cases. Simply override the test case (method) your map
* fails and/or the methods that define the assumptions used by the test
* cases. For example, if your map does not allow duplicate values, override
- * {@link #isAllowDuplicateValues()} and have it return <code>false</code>
+ * {@link #isAllowDuplicateValues()} and have it return {@code false}
*
*/
public abstract class AbstractMapTest<K, V> extends AbstractObjectTest {
@@ -167,7 +167,7 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
/**
* Returns true if the maps produced by
* {@link #makeObject()} and {@link #makeFullMap()}
- * support the <code>put</code> and <code>putAll</code> operations
+ * support the {@code put} and {@code putAll} operations
* adding new mappings.
* <p>
* Default implementation returns true.
@@ -180,7 +180,7 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
/**
* Returns true if the maps produced by
* {@link #makeObject()} and {@link #makeFullMap()}
- * support the <code>put</code> and <code>putAll</code> operations
+ * support the {@code put} and {@code putAll} operations
* changing existing mappings.
* <p>
* Default implementation returns true.
@@ -193,7 +193,7 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
/**
* Returns true if the maps produced by
* {@link #makeObject()} and {@link #makeFullMap()}
- * support the <code>setValue</code> operation on entrySet entries.
+ * support the {@code setValue} operation on entrySet entries.
* <p>
* Default implementation returns isPutChangeSupported().
* Override if your collection class does not support setValue but does
@@ -206,7 +206,7 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
/**
* Returns true if the maps produced by
* {@link #makeObject()} and {@link #makeFullMap()}
- * support the <code>remove</code> and <code>clear</code> operations.
+ * support the {@code remove} and {@code clear} operations.
* <p>
* Default implementation returns true.
* Override if your collection class does not support removal operations.
@@ -295,7 +295,7 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
* method must return an array with the same length as {@link
* #getSampleValues()} and all array elements must be different. The
* default implementation constructs a set of String keys, and includes a
- * single null key if {@link #isAllowNullKey()} returns <code>true</code>.
+ * single null key if {@link #isAllowNullKey()} returns {@code true}.
*/
@SuppressWarnings("unchecked")
public K[] getSampleKeys() {
@@ -348,9 +348,9 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
* method must return an array with the same length as
* {@link #getSampleKeys()}. The default implementation constructs a set
of
* String values and includes a single null value if
- * {@link #isAllowNullValue()} returns <code>true</code>, and includes
+ * {@link #isAllowNullValue()} returns {@code true}, and includes
* two values that are the same if {@link #isAllowDuplicateValues()}
returns
- * <code>true</code>.
+ * {@code true}.
*/
@SuppressWarnings("unchecked")
public V[] getSampleValues() {
@@ -371,9 +371,9 @@ public abstract class AbstractMapTest<K, V> extends
AbstractObjectTest {
* returned from this method should not be the same as those returned from
* {@link #getSampleValues()}. The default implementation constructs a
* set of String values and includes a single null value if
- * {@link #isAllowNullValue()} returns <code>true</code>, and includes two
values
+ * {@link #isAllowNullValue()} returns {@code true}, and includes two
values
* that are the same if {@link #isAllowDuplicateValues()} returns
- * <code>true</code>.
+ * {@code true}.
*/
@SuppressWarnings("unchecked")
public V[] getNewSampleValues() {
diff --git a/src/test/java/org/apache/commons/collections4/map/IdentityMap.java
b/src/test/java/org/apache/commons/collections4/map/IdentityMap.java
index cff89ae..f7aa207 100644
--- a/src/test/java/org/apache/commons/collections4/map/IdentityMap.java
+++ b/src/test/java/org/apache/commons/collections4/map/IdentityMap.java
@@ -23,8 +23,8 @@ import java.io.Serializable;
import java.util.Map;
/**
- * A <code>Map</code> implementation that matches keys and values based
- * on <code>==</code> not <code>equals()</code>.
+ * A {@code Map} implementation that matches keys and values based
+ * on {@code ==} not {@code equals()}.
* <p>
* <strong>This map will violate the detail of various Map and map view
contracts.</strong>
* As a general rule, don't compare this map to other maps. In particular, you
can't
@@ -105,7 +105,7 @@ public class IdentityMap<K, V>
/**
* Compares two keys for equals.
- * This implementation uses <code>==</code>.
+ * This implementation uses {@code ==}.
*
* @param key1 the first key to compare
* @param key2 the second key to compare
@@ -118,7 +118,7 @@ public class IdentityMap<K, V>
/**
* Compares two values for equals.
- * This implementation uses <code>==</code>.
+ * This implementation uses {@code ==}.
*
* @param value1 the first value to compare
* @param value2 the second value to compare
diff --git
a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
index ea851cc..6b16196 100644
---
a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
+++
b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
@@ -71,8 +71,8 @@ public abstract class AbstractMultiValuedMapTest<K, V>
extends AbstractObjectTes
/**
* Returns true if the maps produced by {@link #makeObject()} and
- * {@link #makeFullMap()} support the <code>put</code> and
- * <code>putAll</code> operations adding new mappings.
+ * {@link #makeFullMap()} support the {@code put} and
+ * {@code putAll} operations adding new mappings.
* <p>
* Default implementation returns true. Override if your collection class
* does not support put adding.
@@ -83,8 +83,8 @@ public abstract class AbstractMultiValuedMapTest<K, V>
extends AbstractObjectTes
/**
* Returns true if the maps produced by {@link #makeObject()} and
- * {@link #makeFullMap()} support the <code>remove</code> and
- * <code>clear</code> operations.
+ * {@link #makeFullMap()} support the {@code remove} and
+ * {@code clear} operations.
* <p>
* Default implementation returns true. Override if your collection class
* does not support removal operations.
@@ -125,7 +125,7 @@ public abstract class AbstractMultiValuedMapTest<K, V>
extends AbstractObjectTes
* must return an array with the same length as {@link #getSampleValues()}
* and all array elements must be different. The default implementation
* constructs a set of String keys, and includes a single null key if
- * {@link #isAllowNullKey()} returns <code>true</code>.
+ * {@link #isAllowNullKey()} returns {@code true}.
*/
@SuppressWarnings("unchecked")
public K[] getSampleKeys() {
