[jboss-cvs] JBossAS SVN: r77044 - projects/aop/trunk/aop/src/main/org/jboss/aop/instrument.

[jboss-cvs-commits at lists.jboss.org]

Author: flavia.rainone at jboss.com
Date: 2008-08-13 20:20:29 -0400 (Wed, 13 Aug 2008)
New Revision: 77044

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/MethodExecutionTransformer.java
Log:
[JBAOP-617] The problem with the gc on prepareAll hotswap test is solved.
The values of a CtClass array were not being set in the method MethodExecutionTransformer.unwrap,
causing NPE inside Javassist later on.

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/MethodExecutionTransformer.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/MethodExecutionTransformer.java	2008-08-13 23:46:45 UTC (rev 77043)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/MethodExecutionTransformer.java	2008-08-14 00:20:29 UTC (rev 77044)
@@ -212,7 +212,7 @@
          CtClass[] javassistParameterTypes = new CtClass[parameterTypes.length];
          for (int i = 0; i < parameterTypes.length; i++)
          {
-            classPool.getLocally(parameterTypes[i].getName());
+            javassistParameterTypes[i] = classPool.getLocally(parameterTypes[i].getName());
          }
          CtMethod javassistWMethod = clazz.getDeclaredMethod(method.getName(), javassistParameterTypes);
          if (wrapper.isNotPrepared(javassistWMethod, WrapperTransformer.SINGLE_TRANSFORMATION_INDEX))