svn commit: r489823 - /db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/sequence/SequenceManagerHelper.java

[email protected]
Newsgroups gmane.comp.jakarta.ojb.devel
Message-ID <[email protected]>
Author: arminw
Date: Fri Dec 22 17:38:33 2006
New Revision: 489823

URL: http://svn.apache.org/viewvc?view=rev&rev=489823
Log:
minor improvement, avoid redundant method calls

Modified:
    db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/sequence/SequenceManagerHelper.java

Modified: db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/sequence/SequenceManagerHelper.java
URL: http://svn.apache.org/viewvc/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/sequence/SequenceManagerHelper.java?view=diff&rev=489823&r1=489822&r2=489823
==============================================================================
--- db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/sequence/SequenceManagerHelper.java (original)
+++ db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/sequence/SequenceManagerHelper.java Fri Dec 22 17:38:33 2006
@@ -144,8 +144,7 @@
                 check for table-per-subclass inheritance
                 */
                 cldTargetClass = findInheritanceRoot(cldTargetClass);
-                Class topLevel = model.getTopLevelClass(cldTargetClass.getClassOfObject());
-                cldTargetClass = model.getDescriptorFor(topLevel);
+                cldTargetClass = cldTargetClass.getTopLevelClassDescriptor();
                 if(cldTargetClass.isInterface() || cldTargetClass.isAbstract())
                 {
                     /*
@@ -230,7 +229,7 @@
             // throw new PersistenceBrokerException("Given FieldDescriptor was null");
         }
         // first lookup top-level class
-        Class topLevel = brokerForClass.getTopLevelClass(field.getClassDescriptor().getClassOfObject());
+        Class topLevel = field.getClassDescriptor().getTopLevelClass();
         return getMaxId(brokerForClass, topLevel, field);
     }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.