[Jython-checkins] SF.net SVN: jython:[5701] branches/newstyle-java-types/src/org/python/core/ PyJavaType.java

[<cg...@us...>]

Revision: 5701
          http://jython.svn.sourceforge.net/jython/?rev=5701&view=rev
Author:   cgroves
Date:     2008-12-05 11:38:02 +0000 (Fri, 05 Dec 2008)

Log Message:
-----------
Avoid non-public interface methods similarly to non-public class methods

Modified Paths:
--------------
    branches/newstyle-java-types/src/org/python/core/PyJavaType.java

Modified: branches/newstyle-java-types/src/org/python/core/PyJavaType.java
===================================================================
--- branches/newstyle-java-types/src/org/python/core/PyJavaType.java	2008-12-05 11:21:55 UTC (rev 5700)
+++ branches/newstyle-java-types/src/org/python/core/PyJavaType.java	2008-12-05 11:38:02 UTC (rev 5701)
@@ -327,6 +327,10 @@
     private void handleSuperMethodArgCollisions() {
         for (Class iface : underlying_class.getInterfaces()) {
             for (Method meth : iface.getMethods()) {
+                if (!Modifier.isPublic(meth.getDeclaringClass().getModifiers())) {
+                    // Ignore methods from non-public interfaces as they're similarly bugged
+                    continue;
+                }
                 String nmethname = normalize(meth.getName());
                 PyObject[] where = new PyObject[1];
                 PyObject obj = lookup_where(nmethname, where);


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.