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

svn commit: r439548 - /incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/InheritanceComparator.java

Author: skim
Date: Fri Sep  1 23:17:06 2006
New Revision: 439548

URL: http://svn.apache.org/viewvc?rev=439548&view=rev
Log:
bug in ordering

Modified:
    incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/InheritanceComparator.java

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/InheritanceComparator.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/InheritanceComparator.java?rev=439548&r1=439547&r2=439548&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/InheritanceComparator.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/InheritanceComparator.java Fri Sep  1 23:17:06 2006
@@ -22,7 +22,7 @@
  *
  * @author Abe White
  */
-class InheritanceComparator
+public class InheritanceComparator
     implements Comparator {
 
     private Class _base = Object.class;
@@ -64,9 +64,9 @@
         int i2 = levels(c2);
         if (i1 == i2) {
             // sort simple interfaces as well as simple order test will fail.
-            if (c1.isAssignableFrom(o2.getClass()))
+            if (c1.isAssignableFrom(c2))
                 return -1;
-            if (c2.isAssignableFrom(o1.getClass()))
+            if (c2.isAssignableFrom(c1))
                 return 1;
             return c1.getName().compareTo(c2.getName());
         }