| Issue 41: | Direct call to ASM code in CglibInvokable can have errors in presence of multiple ASM instances | |
| 1 person starred this issue and may be notified of changes. | Back to list |
Non-JarJar-ed ASM jar has base packaging of org.objectweb.asm.*, while cglib uses JarJar-ed path (net.sf.cglib.asm.* or something like that). In my experience today, there can be a collision between the two. Proposal is to pass the Method in to the constructor for CglibInvokable (besides the MethodProxy), and use it like in JavassistInvokable for determining argument count and method name. Still use MethodProxy for faster function execution (maybe test to make sure it's still significantly faster enough to make difference, otherwise the two invokables could be collapsed into a single class)
Jan 4, 2012
Project Member
#1
kandpwel...@gmail.com
Status:
Started
Jan 5, 2012
It worked here ohere I was experiencing the problem. If you don't review this, I'll mark it fixed shortly.
Status:
CodeComplete
Jan 9, 2012
closing
Status:
Fixed
|