Javadoc

Use HTML 'em' tag instead of 'i' tag

src/main/java/org/apache/commons/logging/LogFactory.java

@@ -228,7 +228,7 @@ public abstract class LogFactory {
      * <li>using JDK1.2+ and the calling code is loaded via the boot
      *  class loader (only likely for embedded systems work).</li>
      * </ul>
-     * Note that {@code factories} is a <i>Hashtable</i> (not a HashMap),
+     * Note that {@code factories} is a <em>Hashtable</em> (not a HashMap),
      * and hashtables don't allow null as a key.
      * @deprecated since 1.1.2
      */

src/main/java/org/apache/commons/logging/LogSource.java

@@ -161,14 +161,14 @@ public class LogSource {
     }
 
     /**
-     * Create a new {@link Log} implementation, based on the given <i>name</i>.
+     * Create a new {@link Log} implementation, based on the given <em>name</em>.
      * <p>
      * The specific {@link Log} implementation returned is determined by the
      * value of the {@code org.apache.commons.logging.log} property. The value
      * of {@code org.apache.commons.logging.log} may be set to the fully specified
      * name of a class that implements the {@link Log} interface. This class must
      * also have a public constructor that takes a single {@link String} argument
-     * (containing the <i>name</i> of the {@link Log} to be constructed.
+     * (containing the <em>name</em> of the {@link Log} to be constructed.
      * <p>
      * When {@code org.apache.commons.logging.log} is not set, or when no corresponding
      * class can be found, this method will return a Log4JLogger if the Log4j Logger
@@ -200,7 +200,7 @@ public class LogSource {
      * @throws LinkageError                if there is missing dependency.
      * @throws ExceptionInInitializerError unexpected exception has occurred in a static initializer.
      * @throws NoSuchMethodException       if a matching method is not found.
-     * @throws SecurityException           If a security manager, <i>s</i>, is present and the caller's class loader is not the same as or an ancestor of the
+     * @throws SecurityException           If a security manager, <em>s</em>, is present and the caller's class loader is not the same as or an ancestor of the
      *                                     class loader for the current class and invocation of {@link SecurityManager#checkPackageAccess
      *                                     s.checkPackageAccess()} denies access to the package of this class.
      */
@@ -215,7 +215,7 @@ public class LogSource {
      *
      * @param className class name.
      * @throws LinkageError           if there is missing dependency.
-     * @throws SecurityException      If a security manager, <i>s</i>, is present and the caller's class loader is not the same as or an ancestor of the class
+     * @throws SecurityException      If a security manager, <em>s</em>, is present and the caller's class loader is not the same as or an ancestor of the class
      *                                loader for the current class and invocation of {@link SecurityManager#checkPackageAccess s.checkPackageAccess()} denies
      *                                access to the package of this class.
      */

src/main/java/org/apache/commons/logging/impl/AvalonLogger.java

@@ -44,7 +44,7 @@ import org.apache.commons.logging.Log;
  * because the constructors available for it make this impossible to achieve in all
  * circumstances; there is no way to "reconnect" to an underlying Logger object on
  * deserialization if one was just passed in to the constructor of the original
- * object. This class <i>was</i> marked Serializable in the 1.0.4 release of
+ * object. This class <em>was</em> marked Serializable in the 1.0.4 release of
  * commons-logging, but this never actually worked (a NullPointerException would
  * be thrown as soon as the deserialized object was used), so removing this marker
  * is not considered to be an incompatible change.

src/main/java/org/apache/commons/logging/impl/LogFactoryImpl.java

@@ -666,7 +666,7 @@ public class LogFactoryImpl extends LogFactory {
      * <p>
      * This method usually returns the context class loader. However if it
      * is discovered that the class loader which loaded this class is a child
-     * of the context class loader <i>and</i> the allowFlawedContext option
+     * of the context class loader <em>and</em> the allowFlawedContext option
      * has been set then the class loader which loaded this class is returned
      * instead.
      * <p>
@@ -1213,7 +1213,7 @@ public class LogFactoryImpl extends LogFactory {
 
     /**
      * Utility method to check whether a particular logging library is
-     * present and available for use. Note that this does <i>not</i>
+     * present and available for use. Note that this does <em>not</em>
      * affect the future behavior of this class.
      */
     private boolean isLogLibraryAvailable(final String name, final String className) {

src/main/java/org/apache/commons/logging/impl/WeakHashtable.java

@@ -48,7 +48,7 @@ import java.util.Set;
  * <p>
  * <strong>Usage:</strong> typical use case is as a drop-in replacement
  * for the {@code Hashtable} used in {@code LogFactory} for J2EE environments
- * running 1.3+ JVMs. Use of this class <i>in most cases</i> (see below) will
+ * running 1.3+ JVMs. Use of this class <em>in most cases</em> (see below) will
  * allow class loaders to be collected by the garbage collector without the need
  * to call {@link org.apache.commons.logging.LogFactory#release(ClassLoader) LogFactory.release(ClassLoader)}.
  * </p>
@@ -78,7 +78,7 @@ import java.util.Set;
  * reference to the component's class loader from a key in the "global"
  * {@code LogFactory}'s factories member! If {@code LogFactory.release()}
  * is called whenever component is unloaded, the class loaders will be correctly
- * garbage collected; this <i>should</i> be done by any container that
+ * garbage collected; this <em>should</em> be done by any container that
  * bundles commons-logging by default. However, holding the class loader
  * references weakly ensures that the class loader will be garbage collected
  * without the container performing this step.

src/test/java/org/apache/commons/logging/PathableClassLoader.java

@@ -412,7 +412,7 @@ public class PathableClassLoader extends URLClassLoader {
 
     /**
      * For classes with the specified prefix, get them from the system
-     * classpath <i>which is active at the point this method is called</i>.
+     * classpath <em>which is active at the point this method is called</em>.
      * <p>
      * This method is just a shortcut for
      * <pre>

src/test/java/org/apache/commons/logging/UserClass.java

@@ -25,7 +25,7 @@ public class UserClass {
      * associated with this class' class loader.
      * <p>
      * Don't forget to set the context class loader to whatever it will be
-     * when an instance of this class is actually created <i>before</i> calling
+     * when an instance of this class is actually created <em>before</em> calling
      * this method!
      */
     public static void setAllowFlawedContext(final String state) {