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