You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by kw...@apache.org on 2007/02/05 17:18:24 UTC

svn commit: r503757 - in /incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel: AbstractBrokerFactory.java BrokerImpl.java

Author: kwsutter
Date: Mon Feb  5 08:18:23 2007
New Revision: 503757

URL: http://svn.apache.org/viewvc?view=rev&rev=503757
Log:
Use single line commenting style (//) instead of the block commenting style (/*..*/).

Modified:
    incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java
    incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java?view=diff&rev=503757&r1=503756&r2=503757
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java Mon Feb  5 08:18:23 2007
@@ -287,8 +287,7 @@
             Broker broker;
             for (Iterator itr = _brokers.iterator(); itr.hasNext();) {
                 broker = (Broker) itr.next();
-                /* Check for null here because _brokers is a weak reference
-                collection */
+                // Check for null because _brokers contains weak references
                 if ((broker != null) && (!broker.isClosed()))
                     broker.close();
             }

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java?view=diff&rev=503757&r1=503756&r2=503757
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java Mon Feb  5 08:18:23 2007
@@ -3094,9 +3094,8 @@
     public void detachAll(OpCallbacks call, boolean flush) {
         beginOperation(true);
         try {
-            /* If a flush is desired (based on input parm), then check if the
-             * "dirty" flag is set before calling flush().
-             */
+            // If a flush is desired (based on input parm), then check if the
+            //   "dirty" flag is set before calling flush().
             if ((flush) && ((_flags & FLAG_FLUSH_REQUIRED) != 0))
                 flush();
             detachAllInternal(call);
@@ -3486,7 +3485,7 @@
             default:
                 // use store manager for native sequence
                 if (fmd == null) {
-                    // this will return a sequence even for app id classes, 
+                    // this will return a sequence even for app id classes,
                     // which is what we want for backwards-compatibility
                     return _store.getDataStoreIdSequence(meta);
                 }