[jboss-jira] [JBoss JIRA] Commented: (JASSIST-102) Bug report for CtNewMethod.copy

Bug report for CtNewMethod.copy ------------------------------- Key: JASSIST-102 URL: https://jira.jboss.org/jira/browse/JASSIST-102 Project: Javassist Issue Type: Bug Affects Versions: 3.11.0.GA Reporter: Shigeru Chiba Assignee: Shigeru Chiba Priority: Minor ** the following message was directly sent to chiba on December 26, 2009. Dear Shigeru, when copying a method from class A to class B, the class name of A is replaced by B all over the descriptor of the method. This is not documented, and is not required in all cases; I understand that the opcodes for "this" return B instead of A, and this may justify using a different return type (but doesn't justify using different argument types). I use this method to build manually my proxies and to replicate the original signature, while later replacing the method's body. I suggest documenting this behaviour, and adding an overload with a boolean parameter to request or suppress this renaming. Here's the stack trace (built manually) that shows where this happens (I first noticed this behaviour experimentally): MethodInfo.read(MethodInfo src, String methodname, Map classnames) (line 478) MethodInfo(ConstPool cp, String methodname, MethodInfo src, Map classnameMap) (line 104) void CtBehaviour.copy(CtBehavior src, boolean isCons, ClassMap map) (line 66) CtMethod(CtMethod src, CtClass declaring, ClassMap map) (line 113) CtMethod CtNewMethod.copy(CtMethod src, CtClass declaring, ClassMap map) (line 159) Kind regards -- Paolo Giarrusso