You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pc...@apache.org on 2006/07/14 01:45:02 UTC

svn commit: r421741 [3/15] - in /incubator/openjpa/trunk: openjpa-kernel-5/src/main/java/org/apache/openjpa/enhance/ openjpa-kernel/src/main/java/org/apache/openjpa/abstractstore/ openjpa-kernel/src/main/java/org/apache/openjpa/ant/ openjpa-kernel/src/...

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryCacheStoreQuery.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryCacheStoreQuery.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryCacheStoreQuery.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryCacheStoreQuery.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -46,8 +46,7 @@
 
 /**
  * A {@link StoreQuery} implementation that caches the OIDs involved in
- * the query, and can determine whether or not the query has been
- * dirtied.
+ * the query, and can determine whether or not the query has been dirtied.
  *
  * @author Patrick Linskey
  * @since 2.5.0
@@ -70,8 +69,7 @@
     }
 
     /**
-     * Return the {@link QueryCache} that this object is associated
-     * with.
+     * Return the {@link QueryCache} that this object is associated with.
      */
     public QueryCache getCache() {
         return _cache;
@@ -85,30 +83,25 @@
     }
 
     /**
-     * <p>Look in the query cache for a result for the given query
+     * Look in the query cache for a result for the given query
      * key. Only look if this query is being executed outside a
      * transaction or in a transaction with IgnoreChanges set to true
      * or in a transaction with IgnoreChanges set to false but in which
-     * none of the classes involved in this query have been
-     * touched.</p>
-     * <p/>
-     * <p>Caching is not used when using object locking.
+     * none of the classes involved in this query have been touched.
+     *  Caching is not used when using object locking.
      * This is because we must obtain locks on the
      * data, and it is likely that making n trips to the database to
      * make the locks will be slower than running the query against
-     * the database.</p>
-     * <p/>
-     * <p>If the fetch configuration has query caching disabled,
-     * then this method returns <code>null</code>.</p>
-     * <p/>
-     * <p>Return the list if we meet the above criteria and if a list
+     * the database.
+     *  If the fetch configuration has query caching disabled,
+     * then this method returns <code>null</code>.
+     *  Return the list if we meet the above criteria and if a list
      * is found for <code>qk</code>. Else, return
-     * <code>null</code>.</p>
-     * <p/>
-     * <p>This implementation means that queries against the cache
+     * <code>null</code>.
+     *  This implementation means that queries against the cache
      * are of READ_COMMITTED isolation level. It'd be nice to support
      * READ_SERIALIZABLE -- to do so, we'd just return false when in
-     * a transaction.</p>
+     * a transaction.
      */
     private List checkCache(QueryKey qk) {
         if (qk == null)
@@ -451,7 +444,7 @@
     }
 
     /**
-     * Result list implementation for a cached query result.  Package-protected
+     * Result list implementation for a cached query result. Package-protected
      * for testing.
      */
     public static class CachedList
@@ -495,8 +488,7 @@
      * A wrapper around a {@link ResultObjectProvider} that builds up a list of
      * all the OIDs in this list and registers that list with the
      * query cache. Abandons monitoring and registering if one of the classes
-     * in the access path is modified while the query results are being
-     * loaded.
+     * in the access path is modified while the query results are being loaded.
      */
     private class CachingResultObjectProvider
         implements ResultObjectProvider, TypesChangedListener {
@@ -514,7 +506,7 @@
         private int _size = Integer.MAX_VALUE;
 
         /**
-         * Constructor.  Supply delegate result provider and our query key.
+         * Constructor. Supply delegate result provider and our query key.
          */
         public CachingResultObjectProvider(ResultObjectProvider rop,
             boolean proj, QueryKey key) {
@@ -666,7 +658,7 @@
     }
 
     /**
-     *	Struct to recognize cached oids.
+     * Struct to recognize cached oids.
      */
     private static class CachedObjectId {
 

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -45,11 +45,11 @@
 import org.apache.openjpa.util.ImplHelper;
 
 /**
- * <p>This class stores information about a particular invocation of
+ * This class stores information about a particular invocation of
  * a query. It contains a reference to the external properties of the
  * query that was executed, as well as any parameters used to execute
  * that query, with one exception: first-class objects used as
- * parameter values are converted to OIDs.</p>
+ * parameter values are converted to OIDs.
  *
  * @author Patrick Linskey
  */

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryResult.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryResult.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryResult.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryResult.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -19,7 +19,7 @@
 import java.util.Collection;
 
 /**
- * <p>A query result.</p>
+ * A query result.
  *
  * @author Abe White
  */
@@ -56,7 +56,7 @@
     }
 
     /**
-     *	Whether this data is timed out.
+     * Whether this data is timed out.
      */
     public boolean isTimedOut() {
         return _ex != -1 && _ex < System.currentTimeMillis();

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedEvent.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedEvent.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedEvent.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedEvent.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -19,8 +19,8 @@
 import java.util.EventObject;
 
 /**
- * <p>An event indicating that instances of given persistent types have
- * been modified.</p>
+ * An event indicating that instances of given persistent types have
+ * been modified.
  *
  * @author Abe White
  */
@@ -32,8 +32,8 @@
     /**
      * Constructor.
      *
-     * @param    source    the data or query cache
-     * @param    types    the changed types
+     * @param source the data or query cache
+     * @param types the changed types
      */
     public TypesChangedEvent(Object source, Collection types) {
         super(source);

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/TypesChangedListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.datacache;
 
 /**
- * <p>An entity that wishes to be notified when types change.</p>
+ * An entity that wishes to be notified when types change.
  *
  * @author Abe White
  * @since 3.0

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/AutomaticManagedRuntime.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/AutomaticManagedRuntime.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/AutomaticManagedRuntime.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/AutomaticManagedRuntime.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -25,11 +25,10 @@
 import org.apache.openjpa.util.InvalidStateException;
 
 /**
- * <p>Implementation of the {@link ManagedRuntime} interface that searches
+ * Implementation of the {@link ManagedRuntime} interface that searches
  * through a set of known JNDI locations and method invocations to locate the
- * appropriate mechanism for obtaining a TransactionManager.</p>
- * <p/>
- * <p>Built in support is provided for the following Application Servers:
+ * appropriate mechanism for obtaining a TransactionManager.
+ *  Built in support is provided for the following Application Servers:
  * <ul>
  * <li>Bluestone</li>
  * <li>HP Application Server</li>
@@ -41,7 +40,7 @@
  * <li>SunONE</li>
  * <li>Weblogic</li>
  * <li>Websphere</li>
- * </ul></p>
+ * </ul>
  *
  * @author Marc Prud'hommeaux
  */
@@ -49,12 +48,12 @@
     implements ManagedRuntime, Configurable {
 
     private static final String [] JNDI_LOCS = new String []{
-        "javax.transaction.TransactionManager",        // weblogic
-        "java:/TransactionManager",                    // jboss & jrun
-        "java:/DefaultDomain/TransactionManager",    // jrun too
-        "java:comp/pm/TransactionManager",            // orion & oracle
-        "java:comp/TransactionManager",                // generic
-        "java:pm/TransactionManager",                // borland
+        "javax.transaction.TransactionManager", // weblogic
+        "java:/TransactionManager", // jboss & jrun
+        "java:/DefaultDomain/TransactionManager", // jrun too
+        "java:comp/pm/TransactionManager", // orion & oracle
+        "java:comp/TransactionManager", // generic
+        "java:pm/TransactionManager", // borland
     };
     private static final String [] METHODS = new String[]{
         "com.arjuna.jta.JTA_TransactionManager.transactionManager", // hp
@@ -66,7 +65,7 @@
         "org.openejb.OpenEJB.getTransactionManager",
         "com.sun.jts.jta.TransactionManagerImpl.getTransactionManagerImpl",
         "com.inprise.visitransact.jta.TransactionManagerImpl."
-            + "getTransactionManagerImpl",                            // borland
+            + "getTransactionManagerImpl", // borland
     };
     private static final ManagedRuntime WLS;
     private static final ManagedRuntime SUNONE;

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/InvocationManagedRuntime.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/InvocationManagedRuntime.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/InvocationManagedRuntime.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/InvocationManagedRuntime.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -23,12 +23,11 @@
 import org.apache.openjpa.lib.conf.Configuration;
 
 /**
- * <p>Implementation of the {@link ManagedRuntime} interface that uses
- * a static method call to find the TransactionManager.</p>
- * <p/>
- * <p>For example, to configure it to use IBM Websphere's TransactionManager,
+ * Implementation of the {@link ManagedRuntime} interface that uses
+ * a static method call to find the TransactionManager.
+ *  For example, to configure it to use IBM Websphere's TransactionManager,
  * use the method:<br />
- * <code>com.ibm.ejs.jts.jta.JTSXA.getTransactionManager</code></p>
+ * <code>com.ibm.ejs.jts.jta.JTSXA.getTransactionManager</code>
  *
  * @author Marc Prud'hommeaux
  */
@@ -49,8 +48,7 @@
 
     /**
      * Set the method to invoke to get the {@link TransactionManager}.
-     * <p/>
-     * E.g.: com.ibm.ejs.jts.jta.JTSXA.getTransactionManager
+     *  E.g.: com.ibm.ejs.jts.jta.JTSXA.getTransactionManager
      */
     public void setTransactionManagerMethod(String methodName) {
         _clazz = methodName.substring(0, methodName.lastIndexOf('.'));

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/JNDIManagedRuntime.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/JNDIManagedRuntime.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/JNDIManagedRuntime.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/JNDIManagedRuntime.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -20,8 +20,8 @@
 import javax.transaction.TransactionManager;
 
 /**
- * <p>Implementation of the {@link ManagedRuntime} interface that uses JNDI to
- * find the TransactionManager.</p>
+ * Implementation of the {@link ManagedRuntime} interface that uses JNDI to
+ * find the TransactionManager.
  *
  * @author Abe White
  */

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/ManagedRuntime.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/ManagedRuntime.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/ManagedRuntime.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/ManagedRuntime.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -18,15 +18,15 @@
 import javax.transaction.TransactionManager;
 
 /**
- * <p>This interface must be implemented by concrete plugins to application
- * servers in order to integrate the OpenJPA runtime in a managed environment.</p>
+ * This interface must be implemented by concrete plugins to application
+ * servers in order to integrate the OpenJPA runtime in a managed environment.
  *
  * @author Abe White
  */
 public interface ManagedRuntime {
 
     /**
-     * Return the TransactionManager for the managed runtime.  This
+     * Return the TransactionManager for the managed runtime. This
      * manager is used to register synchronization listeners, to
      * map transactional PersistenceManagers to the current transaction,
      * and possibly to enlist XA resources.

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/SunOneManagedRuntime.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/SunOneManagedRuntime.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/SunOneManagedRuntime.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/SunOneManagedRuntime.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/WLSManagedRuntime.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/WLSManagedRuntime.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/WLSManagedRuntime.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ee/WLSManagedRuntime.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/ApplicationIdTool.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/ApplicationIdTool.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/ApplicationIdTool.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/ApplicationIdTool.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -58,8 +58,7 @@
 import serp.util.Strings;
 
 /**
- * <p>Generates a class appropriate for use as an application identity
- * class.</p>
+ * Generates a class appropriate for use as an application identity class.
  *
  * @author Patrick Linskey
  * @author Abe White
@@ -174,8 +173,8 @@
     }
 
     /**
-     * The directory to write source to.  Defaults to the directory
-     * of the Java file for the set type.  If the given directory does not
+     * The directory to write source to. Defaults to the directory
+     * of the Java file for the set type. If the given directory does not
      * match the package of the object id, the package structure will be
      * created below the directory.
      */
@@ -184,8 +183,8 @@
     }
 
     /**
-     * The directory to write source to.  Defaults to the directory
-     * of the Java file for the set type.  If the given directory does not
+     * The directory to write source to. Defaults to the directory
+     * of the Java file for the set type. If the given directory does not
      * match the package of the object id, the package structure will be
      * created below the directory.
      */
@@ -244,8 +243,7 @@
     }
 
     /**
-     * Returns true if the application identity class should
-     * be an inner class.
+     * Returns true if the application identity class should be an inner class.
      */
     public boolean isInnerClass() {
         Class oidClass = _meta.getObjectIdType();
@@ -605,7 +603,7 @@
 
     /**
      * Create the fromString method that parses the result of our toString
-     * method.  If we have superclasses with id fields, this will call
+     * method. If we have superclasses with id fields, this will call
      * super.fromString() so that the parent class can parse its own fields.
      */
     private String getFromStringCode(boolean hasSuperclass) {
@@ -702,7 +700,7 @@
         else if (!type.isPrimitive()) {
             parse.append("new ").append(Strings.getClassName(type)).
                 openParen(true).append(var).closeParen();
-        } else    // primitive
+        } else // primitive
         {
             switch (type.getName().charAt(0)) {
                 case 'b':
@@ -852,8 +850,7 @@
 
     /**
      * Return a hashCode method that takes into account all
-     * primary key values.  Must deal correctly with both
-     * primitives and objects.
+     * primary key values. Must deal correctly with both primitives and objects.
      */
     private String getHashCodeCode(boolean hasSuperclass) {
         // if we are below a concrete class then we cannot declare any
@@ -1025,7 +1022,7 @@
     /**
      * Code to convert a string to a byte array.
      *
-     * @see    org.apache.openjpa.lib.util.Base16Encoder#decode
+     * @see org.apache.openjpa.lib.util.Base16Encoder#decode
      */
     private String getToBytesByteArrayCode() {
         CodeFormat code = newCodeFormat();
@@ -1071,7 +1068,7 @@
     /**
      * Code to convert a byte array to a string.
      *
-     * @see    org.apache.openjpa.lib.util.Base16Encoder#encode
+     * @see org.apache.openjpa.lib.util.Base16Encoder#encode
      */
     private String getToStringByteArrayCode() {
         CodeFormat code = newCodeFormat();
@@ -1222,20 +1219,19 @@
     }
 
     /**
-     * <p>Usage: java org.apache.openjpa.enhance.ApplicationIdTool [option]*
-     * &lt;class name | .java file | .class file | .jdo file&gt;+</p>
-     * <p/>
-     * <p>Where the following options are recognized.
+     * Usage: java org.apache.openjpa.enhance.ApplicationIdTool [option]*
+     * &lt;class name | .java file | .class file | .jdo file&gt;+
+     *  Where the following options are recognized.
      * <ul>
      * <li><i>-properties/-p &lt;properties file&gt;</i>: The path to a OpenJPA
-     * properties file containing information as outlined in 
+     * properties file containing information as outlined in
      * {@link Configuration}; optional.</li>
      * <li><i>-&lt;property name&gt; &lt;property value&gt;</i>: All bean
      * properties of the standard OpenJPA {@link OpenJPAConfiguration} can be
      * set by using their names and supplying a value.</li>
      * <li><i>-directory/-d &lt;output directory&gt;</i>: Path to the base
-     * source directory.  The package structure will be created beneath
-     * this directory if necessary.  If not specified, the tool will try
+     * source directory. The package structure will be created beneath
+     * this directory if necessary. If not specified, the tool will try
      * to locate the .java file in the CLASSPATH and output to the same
      * directory; failing that, it will use the current directory as
      * the base directory.
@@ -1246,24 +1242,23 @@
      * <li><i>-token/-t &lt;token&gt;</i>: The token to use to separate
      * stingified primary key field values in the stringified oid.</li>
      * <li><i>-name/-n &lt;id class name&gt;</i>: The name of the identity
-     * class to generate.  If this option is specified, you must only
-     * give a single class argument.  If the class metadata names an object
+     * class to generate. If this option is specified, you must only
+     * give a single class argument. If the class metadata names an object
      * id class, this argument is ignored.</li>
      * <li><i>-suffix/-s &lt;id class suffix&gt;</i>: A string to suffix each
-     * persistent class with to create the identity class name.  This is
+     * persistent class with to create the identity class name. This is
      * overridden by <code>-name</code> or by any identity class name
      * specified in metadata.</li>
      * <li><i>-codeFormat/-cf.&lt;property name&gt; &lt; property value&gt;</i>
      * : Arguments like this will be used to configure the bean
      * properties of the internal {@link CodeFormat}.</li>
-     * </ul></p>
-     * <p/>
-     * <p>Each additional argument can be either the full class name of the
+     * </ul>
+     *  Each additional argument can be either the full class name of the
      * type to create an id class for, the path to the .java file for the type,
      * the path to the .class file for the type, or the path to a .jdo file
-     * listing one or more types.  If a .java file already exists for an
+     * listing one or more types. If a .java file already exists for an
      * application id, it will be backed up to a file named
-     * &lt;orig file name&gt;~.</p>
+     * &lt;orig file name&gt;~.
      */
     public static void main(String[] args)
         throws IOException, ClassNotFoundException {
@@ -1281,8 +1276,7 @@
 
     /**
      * Run the application id tool with the given command-line and
-     * given configuration. Returns false if invalid options were
-     * given.
+     * given configuration. Returns false if invalid options were given.
      */
     public static boolean run(OpenJPAConfiguration conf, String[] args,
         Options opts)
@@ -1326,7 +1320,7 @@
     }
 
     /**
-     * Run the tool.  Returns false if invalid options were given.
+     * Run the tool. Returns false if invalid options were given.
      */
     public static boolean run(OpenJPAConfiguration conf, String[] args,
         Flags flags, ClassLoader loader)
@@ -1450,14 +1444,13 @@
     }
 
     /**
-     *	Interface implemented by metadata factories that can load non-existant
-     *	object id classes.
+     * Interface implemented by metadata factories that can load non-existant
+     * object id classes.
      */
     public static interface ObjectIdLoader
 	{
 		/**
-		 *	Turn on the loading of all identity classes, even if they don't
-		 *	exist.
+		 * Turn on the loading of all identity classes, even if they don't exist.
 	 	 */
 		public void setLoadObjectIds ();
 	}

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/CodeGenerator.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/CodeGenerator.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/CodeGenerator.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/CodeGenerator.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -34,7 +34,7 @@
 import serp.util.Strings;
 
 /**
- * <p>Generates Java class code from metadata.</p>
+ * Generates Java class code from metadata.
  *
  * @author Abe White
  * @author Stephen Kim
@@ -50,14 +50,14 @@
     private ParameterTemplate _code = null;
 
     /**
-     * Constructor.  Supply configuration and class to generate code for.
+     * Constructor. Supply configuration and class to generate code for.
      */
     public CodeGenerator(OpenJPAConfiguration conf, Class type) {
         this(new MetaDataRepository(conf).getMetaData(type, null, true));
     }
 
     /**
-     * Constructor.  Supply configuration and metadata to generate code for.
+     * Constructor. Supply configuration and metadata to generate code for.
      */
     public CodeGenerator(ClassMetaData meta) {
         _meta = meta;
@@ -65,7 +65,7 @@
     }
 
     /**
-     * The directory to write source to.  Defaults to the current directory.
+     * The directory to write source to. Defaults to the current directory.
      * If the given directory does not match the package of the metadata, the
      * package structure will be created below the directory.
      */
@@ -74,7 +74,7 @@
     }
 
     /**
-     * The directory to write source to.  Defaults to the current directory.
+     * The directory to write source to. Defaults to the current directory.
      * If the given directory does not match the package of the metadata, the
      * package structure will be created below the directory.
      */
@@ -293,8 +293,7 @@
     }
 
     /**
-     * Append the declaration and code for the given field to the given
-     * buffers.
+     * Append the declaration and code for the given field to the given buffers.
      */
     private void appendFieldCode(FieldMetaData fmd, CodeFormat decs,
         CodeFormat code) {
@@ -462,25 +461,22 @@
 
     /**
      * Return a code template for the given class, or null to use the standard
-     * system-generated Java code.  To facilitate template reuse, the
+     * system-generated Java code. To facilitate template reuse, the
      * following parameters can appear in the template; the proper values
      * will be subtituted by the system:
      * <ul>
      * <li>${packageDec}: The package declaration, in the form
-     * "package &lt;package name &gt;;", or empty string if no
-     * package.</li>
+     * "package &lt;package name &gt;;", or empty string if no package.</li>
      * <li>${imports}: Imports for the packages used by the declared
      * field types.</li>
      * <li>${className}: The name of the class, without package.</li>
      * <li>${extendsDec}: Extends declaration, in the form
      * "extends &lt;superclass&gt;", or empty string if no superclass.</li>
      * <li>${constructor}: A constructor that takes in all primary key fields
-     * of the class, or empty string if the class uses datastore
-     * identity.</li>
+     * of the class, or empty string if the class uses datastore identity.</li>
      * <li>${fieldDecs}: Declarations of all the declared fields.</li>
      * <li>${fieldCode}: Get/set methods for all the declared fields.</li>
-     * </ul>
-     * Returns null by default.
+     * </ul> Returns null by default.
      */
     protected String getClassCode() {
         return null;
@@ -488,7 +484,7 @@
 
     /**
      * Return code for the initial value for the given field, or null to use
-     * the default generated by the system.  Returns null by default.
+     * the default generated by the system. Returns null by default.
      */
     protected String getInitialValue(FieldMetaData field) {
         return null;
@@ -506,26 +502,25 @@
      * <li>${fieldType}: The field's type name.</li>
      * <li>${fieldValue}: The field's initial value, in the form
      * " = &lt;value&gt;", or empty string if none.</li>
-     * </ul>
-     * Returns null by default.
+     * </ul> Returns null by default.
      */
     protected String getDeclaration(FieldMetaData field) {
         return null;
     }
 
     /**
-     *	Return a code template for the get/set methods of the given field, or
-     *	null to use the system-generated default Java code.
-     *	To facilitate template reuse, the following parameters can appear in
-     *	your template; the proper values will be subtituted by the system:
-     *	<ul>
-     *	<li>${fieldName}: The name of the field.</li>
-     *	<li>${capFieldName}: The capitalized field name.</li>
-     *	<li>${propertyName}: The field name without leading '_', if any.</li>
-     *	<li>${fieldType}: The field's type name.</li>
-     *	<li>${fieldValue}: The field's initial value, in the form
-     *		"= &lt;value&gt;", or empty string if none.</li>
-     *	</ul>
+     * Return a code template for the get/set methods of the given field, or
+     * null to use the system-generated default Java code.
+     * To facilitate template reuse, the following parameters can appear in
+     * your template; the proper values will be subtituted by the system:
+     * <ul>
+     * <li>${fieldName}: The name of the field.</li>
+     * <li>${capFieldName}: The capitalized field name.</li>
+     * <li>${propertyName}: The field name without leading '_', if any.</li>
+     * <li>${fieldType}: The field's type name.</li>
+     * <li>${fieldValue}: The field's initial value, in the form
+     * "= &lt;value&gt;", or empty string if none.</li>
+     * </ul>
      */
     protected String getFieldCode (FieldMetaData field)
 	{

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorage.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorage.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorage.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorage.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,9 +16,9 @@
 package org.apache.openjpa.enhance;
 
 /**
- * <p>Interface for dynamically generated classes.  Certain getters/setters
+ * Interface for dynamically generated classes. Certain getters/setters
  * may either return null or throw an exception depending on the
- * {@link DynamicStorageGenerator}'s field policy.</p>
+ * {@link DynamicStorageGenerator}'s field policy.
  *
  * @author Steve Kim
  * @nojavadoc
@@ -132,7 +132,7 @@
     public void setObject(int field, Object val);
 
     /**
-     *  Ensure object capacity
+     * Ensure object capacity
      */
     public void initialize ();
 }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorageGenerator.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorageGenerator.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorageGenerator.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/DynamicStorageGenerator.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -33,8 +33,8 @@
 import serp.bytecode.TableSwitchInstruction;
 
 /**
- * <p>Factory for creating new {@link DynamicStorage} classes.  Can be
- * extended to decorate/modify the generated instances behavior.</p>
+ * Factory for creating new {@link DynamicStorage} classes. Can be
+ * extended to decorate/modify the generated instances behavior.
  *
  * @author Steve Kim
  * @nojavadoc
@@ -58,7 +58,7 @@
 
     /**
      * Constant to be as silent as possible during invalid index passed
-     * to set/get type methods.  On getting an Object, for example,
+     * to set/get type methods. On getting an Object, for example,
      * null will be returned.
      * However, on primitive gets, an exception will be thrown.
      */
@@ -119,7 +119,7 @@
     }
 
     /**
-     * Return a class name to use for the given user key.  By default,
+     * Return a class name to use for the given user key. By default,
      * returns the stringified key prefixed by PREFIX.
      */
     protected String getClassName(Object obj) {
@@ -135,7 +135,7 @@
     }
 
     /**
-     * Return the name for the generated field at the given index.  Returns
+     * Return the name for the generated field at the given index. Returns
      * <code>"field" + i</code> by default.
      */
     protected String getFieldName(int index) {
@@ -413,7 +413,7 @@
 
     /**
      * Clear code associated with the given method signature, and return
-     * the empty code.  Will return null if the method should be empty.
+     * the empty code. Will return null if the method should be empty.
      */
     protected Code replaceMethod(BCClass bc, String name, Class retType,
         Class[] args, boolean remove) {
@@ -518,7 +518,7 @@
     }
 
     /**
-     *  Get the wrapper for the given type.
+     * Get the wrapper for the given type.
      */
     protected Class getWrapper(Class c) {
         for (int i = 0; i < WRAPPERS.length; i++) {

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldConsumer.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldConsumer.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldConsumer.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldConsumer.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.enhance;
 
 /**
- * <p>Consumes persistent field values.</p>
+ * Consumes persistent field values.
  */
 public interface FieldConsumer {
 
@@ -66,7 +66,7 @@
     void storeStringField(int fieldIndex, String value);
 
     /**
-     *	Set the value of the given field.
+     * Set the value of the given field.
      */
     void storeObjectField(int fieldIndex, Object value);
 }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldManager.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldManager.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldManager.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldManager.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.enhance;
 
 /**
- * <p>Manages persistent fields.</p>
+ * Manages persistent fields.
  */
 public interface FieldManager
     extends FieldConsumer, FieldSupplier {

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldSupplier.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldSupplier.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldSupplier.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/FieldSupplier.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.enhance;
 
 /**
- * <p>Supplies persistent field values.</p>
+ * Supplies persistent field values.
  */
 public interface FieldSupplier {
 
@@ -66,7 +66,7 @@
     String fetchStringField(int fieldIndex);
 
     /**
-     *	Return the value of the given field.
+     * Return the value of the given field.
      */
     Object fetchObjectField (int fieldIndex);
 }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCDataGenerator.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCDataGenerator.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCDataGenerator.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCDataGenerator.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -48,9 +48,9 @@
 import serp.bytecode.LookupSwitchInstruction;
 
 /**
- * <p>Generates {@link PCData} instances which avoid primitve wrappers
+ * Generates {@link PCData} instances which avoid primitve wrappers
  * to optimize memory use and performance at the cost of slightly higher
- * startup time.</p>
+ * startup time.
  *
  * @author Steve Kim
  * @nojavadoc
@@ -116,8 +116,7 @@
     }
 
     /**
-     * Perform any final actions before the pcdata is returned to client
-     * code.
+     * Perform any final actions before the pcdata is returned to client code.
      */
     protected void finish(DynamicPCData data, ClassMetaData meta) {
     }
@@ -196,7 +195,7 @@
 
     /**
      * Have to load the type since it may not be available to the
-     * same classloader (i.e. rar vs. ear).  The context classloader
+     * same classloader (i.e. rar vs. ear). The context classloader
      * (i.e. the user app classloader) should be fine.
      */
     private void addGetType(BCClass bc, ClassMetaData meta) {
@@ -371,7 +370,7 @@
             lswitch.setDefaultTarget(target);
 
             // if (obj != null)
-            code.next();    // jump back over target
+            code.next(); // jump back over target
             ifins = code.ifnonnull();
             code.vreturn();
 
@@ -417,7 +416,7 @@
                 code.go2().setTarget(switchTarget);
             }
             lswitch.setDefaultTarget(switchTarget);
-            code.next();    // step over switch target
+            code.next(); // step over switch target
 
             // if (arrIdx != -1)
             code.constant().setValue(-1);
@@ -465,7 +464,7 @@
 
             // if (fieldImpl != null)
             // 		fieldImpl[index] = null;
-            code.next();    // step over nullTarget
+            code.next(); // step over nullTarget
             code.getfield().setField(impl);
             ifins = code.ifnonnull();
             code.vreturn();
@@ -753,7 +752,7 @@
             boolean.class, new Class[]{ int.class });
         jumps2.add(code.ifne());
         //			sm.setIntermediate (index, inter);
-        //	}  // end else
+        //	} // end else
         code.aload().setParam(0);
         code.constant().setValue(index);
         code.aload().setLocal(inter);
@@ -1067,8 +1066,8 @@
     }
 
     /**
-     *  Dynamic {@link PCData}s generated will implement this interface
-     *  to simplify initialization.
+     * Dynamic {@link PCData}s generated will implement this interface
+     * to simplify initialization.
      */
     public static interface DynamicPCData extends PCData {
 

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -79,9 +79,9 @@
 import serp.util.Numbers;
 
 /**
- * <p>Bytecode enhancer used to enhance persistent classes from metadata.  The
+ * Bytecode enhancer used to enhance persistent classes from metadata. The
  * enhancer must be invoked on all persistence-capable and persistence aware
- * classes.</p>
+ * classes.
  *
  * @author Abe White
  */
@@ -123,14 +123,14 @@
     private Set _violations = null;
 
     /**
-     * Constructor.  Supply configuration and type to enhance.
+     * Constructor. Supply configuration and type to enhance.
      */
     public PCEnhancer(OpenJPAConfiguration conf, Class type) {
         this(conf, new Project().loadClass(type), null);
     }
 
     /**
-     * Constructor.  Supply configuration and type to enhance.
+     * Constructor. Supply configuration and type to enhance.
      */
     public PCEnhancer(OpenJPAConfiguration conf, ClassMetaData type) {
         this(conf, new Project().loadClass(type.getDescribedType()),
@@ -138,11 +138,11 @@
     }
 
     /**
-     * Constructor.  Supply configuration.
+     * Constructor. Supply configuration.
      *
-     * @param    type    the bytecode representation fo the type to
+     * @param type the bytecode representation fo the type to
      * enhance; this can be created from any stream or file
-     * @param    repos    a metadata repository to use for metadata access,
+     * @param repos a metadata repository to use for metadata access,
      * or null to create a new reporitory; the repository
      * from the given configuration isn't used by default
      * because the configuration might be an
@@ -187,7 +187,7 @@
 
     /**
      * A boolean indicating whether the enhancer should add a no-args
-     * constructor if one is not already present in the class.  OpenJPA
+     * constructor if one is not already present in the class. OpenJPA
      * requires that a no-arg constructor (whether created by the compiler
      * or by the user) be present in a PC.
      */
@@ -197,7 +197,7 @@
 
     /**
      * A boolean indicating whether the enhancer should add a no-args
-     * constructor if one is not already present in the class.  OpenJPA
+     * constructor if one is not already present in the class. OpenJPA
      * requires that a no-arg constructor (whether created by the compiler
      * or by the user) be present in a PC.
      */
@@ -224,8 +224,8 @@
     }
 
     /**
-     * The base build directory to generate code to.  The proper package
-     * structure will be created beneath this directory.  Defaults to
+     * The base build directory to generate code to. The proper package
+     * structure will be created beneath this directory. Defaults to
      * overwriting the existing class file if null.
      */
     public File getDirectory() {
@@ -233,8 +233,8 @@
     }
 
     /**
-     * The base build directory to generate code to.  The proper package
-     * structure will be creaed beneath this directory.  Defaults to
+     * The base build directory to generate code to. The proper package
+     * structure will be creaed beneath this directory. Defaults to
      * overwriting the existing class file if null.
      */
     public void setDirectory(File dir) {
@@ -242,16 +242,14 @@
     }
 
     /**
-     * Return the current {@link BytecodeWriter} to write to
-     * or null if none.
+     * Return the current {@link BytecodeWriter} to write to or null if none.
      */
     public BytecodeWriter getBytecodeWriter() {
         return _writer;
     }
 
     /**
-     * Set the {@link BytecodeWriter} to write the bytecode
-     * to or null if none.
+     * Set the {@link BytecodeWriter} to write the bytecode to or null if none.
      */
     public void setBytecodeWriter(BytecodeWriter writer) {
         _writer = writer;
@@ -260,7 +258,7 @@
     /**
      * Perform bytecode enhancements.
      *
-     * @return    <code>ENHANCE_*</code> constant
+     * @return <code>ENHANCE_*</code> constant
      */
     public int run() {
         if (_log.isTraceEnabled())
@@ -343,7 +341,7 @@
 
     /**
      * Validate that the methods in this property-access instance are
-     * written correctly.  This method also gathers information on each
+     * written correctly. This method also gathers information on each
      * property's backing field.
      */
     private void validateProperties() {
@@ -435,7 +433,7 @@
     }
 
     /**
-     * Return the field returned / assigned by <code>meth</code>.  Returns
+     * Return the field returned / assigned by <code>meth</code>. Returns
      * null if non-fields (methods, literals, parameters, variables) are
      * returned, or if non-parameters are assigned to fields.
      */
@@ -527,7 +525,7 @@
 
     /**
      * Replaced all direct access to managed fields with the appropriate
-     * pcGet/pcSet method.  Note that this includes access to fields
+     * pcGet/pcSet method. Note that this includes access to fields
      * owned by PersistenceCapable classes other than this one.
      */
     private void replaceAndValidateFieldAccess() {
@@ -558,14 +556,13 @@
      * Replaces all instructions matching the given template in the given
      * code block with calls to the appropriate generated getter/setter.
      *
-     * @param    code    the code block to modify; the code iterator will
+     * @param code the code block to modify; the code iterator will
      * be placed before the first instruction on method start,
-     * and will be after the last instruction on method
-     * completion
-     * @param    ins        the template instruction to search for; either a
+     * and will be after the last instruction on method completion
+     * @param ins the template instruction to search for; either a
      * getfield or putfield instruction
-     * @param    get        boolean indicating if this is a get instruction
-     * @param    stat    template invokestatic instruction to replace with
+     * @param get boolean indicating if this is a get instruction
+     * @param stat template invokestatic instruction to replace with
      */
     private void replaceAndValidateFieldAccess(Code code, Instruction ins,
         boolean get, Instruction stat) {
@@ -645,11 +642,10 @@
      * Helper method to return the declaring PersistenceCapable class of
      * the given field.
      *
-     * @param    fieldName    the name of the field
-     * @param    owner        the nominal owner of the field
+     * @param fieldName the name of the field
+     * @param owner the nominal owner of the field
      * @return the metadata for the PersistenceCapable type that
-     * declares the field (and therefore has the static
-     * method), or null if none
+     * declares the field (and therefore has the static method), or null if none
      */
     private ClassMetaData getPersistenceCapableOwner(String fieldName,
         Class owner) {
@@ -672,7 +668,7 @@
 
     /**
      * Adds all synthetic methods to the bytecode by delegating to
-     * the various addXXXMethods () functions in this class.  Includes
+     * the various addXXXMethods () functions in this class. Includes
      * all static field access methods.
      * Note that the 'stock' methods like <code>pcIsTransactional</code>,
      * <code>pcFetchObjectId</code>, etc are defined only in the
@@ -775,7 +771,7 @@
      * These methods are used by the impl helper to create new
      * managed instances efficiently without reflection.
      *
-     * @param    oid        set to true to mimic the method version that takes
+     * @param oid set to true to mimic the method version that takes
      * an oid value as well as a state manager
      */
     private void addNewInstanceMethod(boolean oid) {
@@ -867,8 +863,7 @@
 
     /**
      * Adds the {@link PersistenceCapable#pcProvideField} and
-     * {@link PersistenceCapable#pcProvideFields} methods to
-     * the bytecode.
+     * {@link PersistenceCapable#pcProvideFields} methods to the bytecode.
      */
     private void addProvideFieldsMethods()
         throws NoSuchMethodException {
@@ -918,8 +913,7 @@
 
     /**
      * Adds the {@link PersistenceCapable#pcReplaceField} and
-     * {@link PersistenceCapable#pcReplaceFields} methods to
-     * the bytecode.
+     * {@link PersistenceCapable#pcReplaceFields} methods to the bytecode.
      */
     private void addReplaceFieldsMethods()
         throws NoSuchMethodException {
@@ -973,8 +967,7 @@
     }
 
     /**
-     * Adds the {@link PersistenceCapable#pcCopyFields} method to the
-     * bytecode.
+     * Adds the {@link PersistenceCapable#pcCopyFields} method to the bytecode.
      */
     private void addCopyFieldsMethod()
         throws NoSuchMethodException {
@@ -1024,7 +1017,7 @@
 
     /**
      * Helper method to add the code common to the beginning of both the
-     * pcReplaceField method and the pcProvideField method.  This includes
+     * pcReplaceField method and the pcProvideField method. This includes
      * calculating the relative field number of the desired field and calling
      * the superclass if necessary.
      *
@@ -1070,7 +1063,7 @@
      * This helper method, given the pcReplaceField or pcProvideField
      * method, adds the bytecode for the corresponding 'plural' version
      * of the method -- the version that takes an int[] of fields to
-     * to access rather than a single field.  The multiple fields version
+     * to access rather than a single field. The multiple fields version
      * simply loops through the provided indexes and delegates to the
      * singular version for each one.
      */
@@ -1198,7 +1191,7 @@
     }
 
     /**
-     * Helper method to add a stock method to the bytecode.  Each
+     * Helper method to add a stock method to the bytecode. Each
      * stock method simply delegates to a corresponding StateManager method.
      * Given the StateManager method, then, this function translates it into
      * the wrapper method that should be added to the bytecode.
@@ -1236,8 +1229,7 @@
     }
 
     /**
-     * Adds the {@link PersistenceCapable#pcGetVersion}
-     * method to the bytecode.
+     * Adds the {@link PersistenceCapable#pcGetVersion} method to the bytecode.
      */
     private void addGetVersionMethod()
         throws NoSuchMethodException {
@@ -1668,7 +1660,7 @@
 
     /**
      * If the given field is a wrapper-type single field identity primary key,
-     * return its corresponding primitive class.  Else return the field type.
+     * return its corresponding primitive class. Else return the field type.
      */
     private Class unwrapSingleFieldIdentity(FieldMetaData fmd) {
         if (!fmd.getDefiningMetaData().isOpenJPAIdentity())
@@ -1781,18 +1773,17 @@
 
     /**
      * When communicating with the StateManager, many methods are used
-     * depending on the class of state being passed.  This method,
+     * depending on the class of state being passed. This method,
      * given the type of information being passed and the prefix
      * ('provided', 'replace', etc) of the method to
      * call, returns the StateManager method that should be used.
      *
-     * @param    type        the type of state being passed
-     * @param    prefix        the prefix of the method to call; all methods
-     * end in '[state type]Field'; only the prefix
-     * varies
-     * @param    get            true if receiving information from the
+     * @param type the type of state being passed
+     * @param prefix the prefix of the method to call; all methods
+     * end in '[state type]Field'; only the prefix varies
+     * @param get true if receiving information from the
      * StateManager, false if passing it to the SM
-     * @param    curValue    true if the current state value is passed to
+     * @param curValue true if the current state value is passed to
      * the StateManager as an extra argument
      */
     private Method getStateManagerMethod(Class type, String prefix,
@@ -1802,17 +1793,15 @@
     }
 
     /**
-     * Return the method of the given owner type matching the given
-     * criteria.
+     * Return the method of the given owner type matching the given criteria.
      *
-     * @param    type        the type of state being passed
-     * @param    prefix        the prefix of the method to call; all methods
-     * end in '[state type]Field'; only the prefix
-     * varies
-     * @param    get            true if receiving information from the
+     * @param type the type of state being passed
+     * @param prefix the prefix of the method to call; all methods
+     * end in '[state type]Field'; only the prefix varies
+     * @param get true if receiving information from the
      * owner, false if passing it to the owner
-     * @param    haspc        true if the pc is passed as an extra argument
-     * @param    curValue    true if the current state value is passed to
+     * @param haspc true if the pc is passed as an extra argument
+     * @param curValue true if the current state value is passed to
      * the owner as an extra argument
      */
     private Method getMethod(Class owner, Class type, String prefix,
@@ -2036,7 +2025,7 @@
 
     /**
      * Adds the code to properly handle PersistenceCapable serialization
-     * to the bytecode.  This includes creating and initializing the
+     * to the bytecode. This includes creating and initializing the
      * static <code>serialVersionUID</code> constant if not already defined,
      * as well as creating a custom <code>writeObject</code> method if the
      * class is Serializable and does not define them.
@@ -2375,7 +2364,7 @@
 
     /**
      * Compare the given field to its Java default, returning the
-     * comparison instruction.  The field value will already be on the stack.
+     * comparison instruction. The field value will already be on the stack.
      */
     private static JumpInstruction ifDefaultValue(Code code,
         FieldMetaData fmd) {
@@ -2440,7 +2429,7 @@
      * Adds bytecode modifying the cloning behavior of the class being
      * enhanced to correctly replace the <code>pcFlags</code> and
      * <code>pcStateManager</code> instance fields of any clone created with
-     * their default values.  Also, if this class is the base PC type
+     * their default values. Also, if this class is the base PC type
      * and does not declared a clone method, one will be added.
      */
     private void addCloningCode() {
@@ -2664,8 +2653,8 @@
      * The generated method interacts with the instance state and the
      * StateManager to get the value of the field.
      *
-     * @param    index    the relative number of the field
-     * @param    fmd        metadata about the field to get
+     * @param index the relative number of the field
+     * @param fmd metadata about the field to get
      */
     private void addGetMethod(int index, FieldMetaData fmd)
         throws NoSuchMethodException {
@@ -2733,8 +2722,8 @@
      * The generated method interacts with the instance state and the
      * StateManager to set the value of the field.
      *
-     * @param    index    the relative number of the field
-     * @param    fmd        metadata about the field to set
+     * @param index the relative number of the field
+     * @param fmd metadata about the field to set
      */
     private void addSetMethod(int index, FieldMetaData fmd)
         throws NoSuchMethodException {
@@ -3209,7 +3198,7 @@
         throws NoSuchMethodException {
         if (_meta.getAccessType() == ClassMetaData.ACCESS_FIELD)
             code.getfield().setField(fmd.getName(), fmd.getDeclaredType());
-        else    // property
+        else // property
         {
             Method meth = (Method) fmd.getBackingMember();
             code.invokevirtual().setMethod(PRE + meth.getName(),
@@ -3227,7 +3216,7 @@
         throws NoSuchMethodException {
         if (_meta.getAccessType() == ClassMetaData.ACCESS_FIELD)
             code.putfield().setField(fmd.getName(), fmd.getDeclaredType());
-        else    // property
+        else // property
             code.invokevirtual().setMethod(PRE + getSetterName(fmd),
                 void.class, new Class[]{ fmd.getDeclaredType() });
     }
@@ -3338,47 +3327,44 @@
     }
 
     /**
-     * <p>Usage: java org.apache.openjpa.enhance.PCEnhancer [option]*
-     * &lt;class name | .java file | .class file | .jdo file&gt;+</p>
-     * <p/>
-     * <p>Where the following options are recognized.
+     * Usage: java org.apache.openjpa.enhance.PCEnhancer [option]*
+     * &lt;class name | .java file | .class file | .jdo file&gt;+
+     *  Where the following options are recognized.
      * <ul>
      * <li><i>-properties/-p &lt;properties file&gt;</i>: The path to a OpenJPA
-     * properties file containing information as outlined in 
+     * properties file containing information as outlined in
      * {@link Configuration}; optional.</li>
      * <li><i>-&lt;property name&gt; &lt;property value&gt;</i>: All bean
      * properties of the standard OpenJPA {@link OpenJPAConfiguration} can be
      * set by using their names and supplying a value; for example:
      * <li><i>-directory/-d &lt;build directory&gt;</i>: The path to the base
-     * directory where enhanced classes are stored.  By default, the
+     * directory where enhanced classes are stored. By default, the
      * enhancer overwrites the original .class file with the enhanced
-     * version.  Use this option to store the generated .class file in
-     * another directory.  The package structure will be created beneath
+     * version. Use this option to store the generated .class file in
+     * another directory. The package structure will be created beneath
      * the given directory.</li>
      * <li><i>-addDefaultConstructor/-adc [true/t | false/f]</i>: Whether to
      * add a default constructor to persistent classes missing one, as
-     * opposed to throwing an exception.  Defaults to true.</li>
+     * opposed to throwing an exception. Defaults to true.</li>
      * <li><i>-tmpClassLoader/-tcl [true/t | false/f]</i>: Whether to
      * load the pre-enhanced classes using a temporary class loader.
-     * Defaults to true.  Set this to false when attempting to debug
+     * Defaults to true. Set this to false when attempting to debug
      * class loading errors.</li>
      * <li><i>-enforcePropertyRestrictions/-epr [true/t | false/f]</i>:
      * Whether to throw an exception if a PROPERTY access entity appears
-     * to be violating standard property restrictions.  Defaults to
-     * false.</li>
-     * </ul></p>
-     * <p/>
-     * <p>Each additional argument can be either the full class name of the
+     * to be violating standard property restrictions. Defaults to false.</li>
+     * </ul>
+     *  Each additional argument can be either the full class name of the
      * type to enhance, the path to the .java file for the type, the path to
      * the .class file for the type, or the path to a .jdo file listing one
      * or more types to enhance.
      * If the type being enhanced has metadata, it will be enhanced as a
-     * persistence capable class.  If not, it will be considered a persistence
+     * persistence capable class. If not, it will be considered a persistence
      * aware class, and all access to fields of persistence capable classes
-     * will be replaced by the appropriate	get/set method.  If the type
+     * will be replaced by the appropriate	get/set method. If the type
      * explicitly declares the persistence-capable interface, it will
-     * not be enhanced.  Thus, it is safe to invoke the enhancer on classes
-     * that are already enhanced.</p>
+     * not be enhanced. Thus, it is safe to invoke the enhancer on classes
+     * that are already enhanced.
      */
     public static void main(String[] args)
         throws IOException {
@@ -3395,7 +3381,7 @@
     }
 
     /**
-     * Run the tool.  Returns false if invalid options given.
+     * Run the tool. Returns false if invalid options given.
      */
     public static boolean run(OpenJPAConfiguration conf, String[] args,
         Options opts)
@@ -3493,7 +3479,7 @@
     }
 
     /**
-     *	Run flags.
+     * Run flags.
      */
     public static class Flags {
 
@@ -3503,9 +3489,8 @@
 		public boolean enforcePropertyRestrictions = false;
 	}
 
-
 	/**
- 	 *	Plugin interface for additional enhancement.
+ 	 * Plugin interface for additional enhancement.
 	 */
 	public static interface AuxiliaryEnhancer
 	{

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCRegistry.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCRegistry.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCRegistry.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCRegistry.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -23,7 +23,7 @@
 import java.util.WeakHashMap;
 
 /**
- * <p>Tracks registered persistence-capable classes.</p>
+ * Tracks registered persistence-capable classes.
  *
  * @since 4.0
  * @author Abe White
@@ -171,10 +171,10 @@
      * @param fieldTypes managed field types
      * @param fieldFlags managed field flags
      * @param sup the most immediate persistent superclass
-     * @param    pcClass        the <code>PersistenceCapable</code> class
-     * @param    fieldNames managed field names
-     * @param    alias        the class alias
-     * @param    pc an instance of the class, if not abstract
+     * @param pcClass the <code>PersistenceCapable</code> class
+     * @param fieldNames managed field names
+     * @param alias the class alias
+     * @param pc an instance of the class, if not abstract
      */
     public static void register(Class pcClass, String[] fieldNames,
         Class[] fieldTypes, byte[] fieldFlags, Class sup, String alias,
@@ -234,8 +234,7 @@
     }
 
     /**
-     *	This is a helper class to manage metadata per persistence-capable
-     *	class.
+     * This is a helper class to manage metadata per persistence-capable class.
      */
     private static class Meta {
 

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PersistenceCapable.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PersistenceCapable.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PersistenceCapable.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PersistenceCapable.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/StateManager.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/StateManager.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/StateManager.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/StateManager.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -19,7 +19,7 @@
 import java.io.ObjectOutput;
 
 /**
- * <p>Internal state manager for managed instances.</p>
+ * Internal state manager for managed instances.
  */
 public interface StateManager {
     // DO NOT ADD ADDITIONAL DEPENDENCIES TO THIS CLASS
@@ -270,13 +270,12 @@
     public double replaceDoubleField(PersistenceCapable pc, int idx);
 
     /**
-     *	Replace state callback.
+     * Replace state callback.
      */
     public String replaceStringField(PersistenceCapable pc, int idx);
 
-
 	/**
-	 *	Replace state callback.
+	 * Replace state callback.
 	 */
     public Object replaceObjectField (PersistenceCapable pc, int idx);
 }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractLifecycleListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractLifecycleListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractLifecycleListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractLifecycleListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,8 +16,8 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Abstract implementation of the {@link LifecycleListener} interface
- * which delegates events to a single method.</p>
+ * Abstract implementation of the {@link LifecycleListener} interface
+ * which delegates events to a single method.
  *
  * @author Steve Kim
  * @author Abe White

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractRemoteCommitProvider.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractRemoteCommitProvider.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractRemoteCommitProvider.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractRemoteCommitProvider.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -24,7 +24,7 @@
 import org.apache.openjpa.lib.util.Localizer;
 
 /**
- * Abstract implementation of {@link RemoteCommitProvider}.  Obtains handles
+ * Abstract implementation of {@link RemoteCommitProvider}. Obtains handles
  * to the event manager and log.
  *
  * @author Patrick Linskey

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractTransactionListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractTransactionListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractTransactionListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AbstractTransactionListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,8 +16,8 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Abstract implementation of the {@link TransactionListener} interface
- * that provides no-op implementations of all methods.</p>
+ * Abstract implementation of the {@link TransactionListener} interface
+ * that provides no-op implementations of all methods.
  *
  * @author Abe White
  * @since 3.0
@@ -26,8 +26,8 @@
     implements TransactionListener {
 
     /**
-     * Catch-all for unhandled events.  This method is called by all other
-     * event methods if you do not override them.  Does nothing by default.
+     * Catch-all for unhandled events. This method is called by all other
+     * event methods if you do not override them. Does nothing by default.
      */
     protected void eventOccurred(TransactionEvent event) {
     }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AttachListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AttachListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AttachListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/AttachListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Listener for when a detached instance is attached.</p>
+ * Listener for when a detached instance is attached.
  *
  * @author Steve Kim
  */

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeanLifecycleCallbacks.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeanLifecycleCallbacks.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeanLifecycleCallbacks.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeanLifecycleCallbacks.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -34,7 +34,7 @@
     private Object _listener;
 
     /**
-     * Constructor.  Make the callback on an instance of the given type.
+     * Constructor. Make the callback on an instance of the given type.
      *
      * @arg whether another argunent is expected such as AfterDetach
      */
@@ -44,7 +44,7 @@
     }
 
     /**
-     * Constructor.  Make the callback on an instance of the given type.
+     * Constructor. Make the callback on an instance of the given type.
      */
     public BeanLifecycleCallbacks(Class cls, Method method, boolean arg) {
         super(method, arg);

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeginTransactionListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeginTransactionListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeginTransactionListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/BeginTransactionListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Notified when transactions begin.</p>
+ * Notified when transactions begin.
  *
  * @author Patrick Linskey
  * @author Abe White
@@ -27,7 +27,7 @@
     /**
      * Notification that a transaction has begun.
      *
-     * @see    TransactionEvent#AFTER_BEGIN
+     * @see TransactionEvent#AFTER_BEGIN
      */
     public void afterBegin(TransactionEvent event);
 }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/CallbackModes.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/CallbackModes.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/CallbackModes.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/CallbackModes.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Callback handling constants.</p>
+ * Callback handling constants.
  *
  * @author Steve Kim
  * @since 4.0
@@ -44,7 +44,7 @@
     public static final int CALLBACK_RETHROW = 2 << 3;
 
     /**
-     *	Flag to always rollback on a callback exception.
+     * Flag to always rollback on a callback exception.
      */
     public static final int CALLBACK_ROLLBACK = 2 << 4;
 }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/ClearListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/ClearListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/ClearListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/ClearListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Listener for when persistent state is cleared from an instance.</p>
+ * Listener for when persistent state is cleared from an instance.
  *
  * @author Steve Kim
  * @author Abe White

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DeleteListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DeleteListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DeleteListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DeleteListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Listener for when a persistent instance is deleted.</p>
+ * Listener for when a persistent instance is deleted.
  *
  * @author Steve Kim
  * @author Abe White

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DetachListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DetachListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DetachListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DetachListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Listener for when a persistent instance is detached.</p>
+ * Listener for when a persistent instance is detached.
  *
  * @author Steve Kim
  */

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DirtyListener.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DirtyListener.java?rev=421741&r1=421740&r2=421741&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DirtyListener.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/event/DirtyListener.java Thu Jul 13 16:44:20 2006
@@ -5,7 +5,7 @@
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *     http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -16,7 +16,7 @@
 package org.apache.openjpa.event;
 
 /**
- * <p>Listener for when a persistent instance becomes dirty.</p>
+ * Listener for when a persistent instance becomes dirty.
  *
  * @author Steve Kim
  * @author Abe White
@@ -39,7 +39,7 @@
     public void beforeDirtyFlushed(LifecycleEvent event);
 
     /**
-     *	Invoked after the first change is applied to a flushed instance.
+     * Invoked after the first change is applied to a flushed instance.
      */
     public void afterDirtyFlushed(LifecycleEvent event);
 }