You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tn...@apache.org on 2014/06/26 22:11:58 UTC

svn commit: r1605899 - in /commons/proper/logging/trunk/src: changes/changes.xml main/java/org/apache/commons/logging/Log.java main/java/org/apache/commons/logging/LogFactory.java test/java/org/apache/commons/logging/AltHashtableTestCase.java

Author: tn
Date: Thu Jun 26 20:11:57 2014
New Revision: 1605899

URL: http://svn.apache.org/r1605899
Log:
[LOGGING-157] Fix javadoc. Thanks to Ville Skyttä

Modified:
    commons/proper/logging/trunk/src/changes/changes.xml
    commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/Log.java
    commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/LogFactory.java
    commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java

Modified: commons/proper/logging/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/logging/trunk/src/changes/changes.xml?rev=1605899&r1=1605898&r2=1605899&view=diff
==============================================================================
--- commons/proper/logging/trunk/src/changes/changes.xml (original)
+++ commons/proper/logging/trunk/src/changes/changes.xml Thu Jun 26 20:11:57 2014
@@ -44,6 +44,9 @@ The <action> type attribute can be add,u
   </properties>
   <body>
     <release version="1.2" date="TBD" description="TBD">
+      <action issue="LOGGING-157" dev="tn" type="fix" due-to="Ville Skyttä">
+        Fix javadoc to comply with javadoc tool from jdk 1.8.
+      </action>
     </release>
     <release version="1.1.3" date="2013-05-23" description="This is a maintenance release containing bug fixes.">
       <action issue="LOGGING-151" dev="tn" type="fix" due-to="Krzysztof Daniel">

Modified: commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/Log.java
URL: http://svn.apache.org/viewvc/commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/Log.java?rev=1605899&r1=1605898&r2=1605899&view=diff
==============================================================================
--- commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/Log.java (original)
+++ commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/Log.java Thu Jun 26 20:11:57 2014
@@ -43,12 +43,12 @@ package org.apache.commons.logging;
  * to be logged).
  * <p>
  * For example,
- * <code><pre>
+ * <pre>
  *    if (log.isDebugEnabled()) {
  *        ... do something expensive ...
  *        log.debug(theResult);
  *    }
- * </pre></code>
+ * </pre>
  * <p>
  * Configuration of the underlying logging system will generally be done
  * external to the Logging APIs, through whatever mechanism is supported by

Modified: commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/LogFactory.java
URL: http://svn.apache.org/viewvc/commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/LogFactory.java?rev=1605899&r1=1605898&r2=1605899&view=diff
==============================================================================
--- commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/LogFactory.java (original)
+++ commons/proper/logging/trunk/src/main/java/org/apache/commons/logging/LogFactory.java Thu Jun 26 20:11:57 2014
@@ -151,23 +151,23 @@ public abstract class LogFactory {
      * classloaders to be substituted by an alternative implementation.
      * <p>
      * <strong>Note:</strong> <code>LogFactory</code> will print:
-     * <code><pre>
-     * [ERROR] LogFactory: Load of custom hashtable failed</em>
-     * </pre></code>
+     * <pre>
+     * [ERROR] LogFactory: Load of custom hashtable failed
+     * </pre>
      * to system error and then continue using a standard Hashtable.
      * <p>
      * <strong>Usage:</strong> Set this property when Java is invoked
      * and <code>LogFactory</code> will attempt to load a new instance
      * of the given implementation class.
      * For example, running the following ant scriplet:
-     * <code><pre>
+     * <pre>
      *  &lt;java classname="${test.runner}" fork="yes" failonerror="${test.failonerror}"&gt;
      *     ...
      *     &lt;sysproperty
      *        key="org.apache.commons.logging.LogFactory.HashtableImpl"
      *        value="org.apache.commons.logging.AltHashtable"/&gt;
      *  &lt;/java&gt;
-     * </pre></code>
+     * </pre>
      * will mean that <code>LogFactory</code> will load an instance of
      * <code>org.apache.commons.logging.AltHashtable</code>.
      * <p>
@@ -951,8 +951,8 @@ public abstract class LogFactory {
      * implementation class, loaded by the specified class loader.
      * If that fails, try the class loader used to load this
      * (abstract) LogFactory.
-     * <p>
      * <h2>ClassLoader conflicts</h2>
+     * <p>
      * Note that there can be problems if the specified ClassLoader is not the
      * same as the classloader that loaded this class, ie when loading a
      * concrete LogFactory subclass via a context classloader.

Modified: commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java
URL: http://svn.apache.org/viewvc/commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java?rev=1605899&r1=1605898&r2=1605899&view=diff
==============================================================================
--- commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java (original)
+++ commons/proper/logging/trunk/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java Thu Jun 26 20:11:57 2014
@@ -23,7 +23,7 @@ import junit.framework.TestCase;
 /**
  * Test the ability to force the LogFactory class to use some
  * arbitrary Hashtable implementation to store its mapping from
- * context-classloader -> LogFactory object.
+ * context-classloader -&gt; LogFactory object.
  */
 public class AltHashtableTestCase extends TestCase {