[Jython-bugs] [ jython-Bugs-610576 ] Impl of abstract method not found

[SourceForge.net <no...@so...>]

Bugs item #610576, was opened at 2002-09-17 09:30
Message generated for change (Comment added) made by cgroves
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112867&aid=610576&group_id=12867

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core
Group: targeted for 2.2beta1
>Status: Closed
>Resolution: Fixed
Priority: 9
Submitted By: Finn Bock (bckfnn)
Assigned to: Frank Wierzbicki (fwierzbicki)
Summary: Impl of abstract method not found

Initial Comment:
The ProxyMaker first adds methods for interfaces, even
when an implementation of the method exists in java
superclass. This cause error:
AttributeError: abstract method "addListDataListener"
not implemented 

----------------------------------------------------------------------

>Comment By: Charles Groves (cgroves)
Date: 2006-09-01 16:52

Message:
Logged In: YES 
user_id=1174327

Finn's fix seemed to take care of it.  Commited in r2923.

----------------------------------------------------------------------

Comment By: Finn Bock (bckfnn)
Date: 2002-09-17 09:38

Message:
Logged In: YES 
user_id=4201

I think something like this is enough but I can't test now.:

Index: ProxyMaker.java
===================================================================
RCS file:
/cvsroot/jython/jython/org/python/compiler/ProxyMaker.java,v
retrieving revision 2.16
diff -u -r2.16 ProxyMaker.java
--- ProxyMaker.java     27 Nov 2001 19:07:21 -0000      2.16
+++ ProxyMaker.java     17 Sep 2002 14:36:53 -0000
@@ -767,6 +767,7 @@
         classfile.addInterface("org/python/core/PyProxy");

         Hashtable seenmethods = new Hashtable();
+        addMethods(superclass, seenmethods);
         for (int i=0; i<interfaces.length; i++) {
             if (interfaces[i].isAssignableFrom(superclass)) {
                 Py.writeWarning("compiler",
@@ -777,12 +778,9 @@
             classfile.addInterface(mapClass(interfaces[i]));
             addMethods(interfaces[i], seenmethods);
         }
-        addMethods(superclass, seenmethods);
         doConstants();
         addClassDictInit();
     }

----------------------------------------------------------------------

Comment By: Finn Bock (bckfnn)
Date: 2002-09-17 09:35

Message:
Logged In: YES 
user_id=4201

Added as test366.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112867&aid=610576&group_id=12867