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);
}