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

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Tue Aug 21 08:25:21 EDT 2007


Author: kabir.khan at jboss.com
Date: 2007-08-21 08:25:20 -0400 (Tue, 21 Aug 2007)
New Revision: 64730

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/GeneratedAdvisorInstrumentor.java
Log:
Tidy up error handling

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/GeneratedAdvisorInstrumentor.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/GeneratedAdvisorInstrumentor.java	2007-08-21 11:20:31 UTC (rev 64729)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/GeneratedAdvisorInstrumentor.java	2007-08-21 12:25:20 UTC (rev 64730)
@@ -111,15 +111,17 @@
       try
       {
          this.clazz = clazz;
-         super.transform(clazz, advisor);
-
-         if (genadvisor != null)
+         if (super.transform(clazz, advisor))
          {
-            addInstanceAdvisorWrappers(clazz);
-            TransformerCommon.compileOrLoadClass(clazz, genadvisor);
-            TransformerCommon.compileOrLoadClass(clazz, genInstanceAdvisor);
+            if (genadvisor != null)
+            {
+               addInstanceAdvisorWrappers(clazz);
+               TransformerCommon.compileOrLoadClass(clazz, genadvisor);
+               TransformerCommon.compileOrLoadClass(clazz, genInstanceAdvisor);
+            }
+            return true;
          }
-         return true;
+         return false;
       }
       catch (Throwable e)
       {




More information about the jboss-cvs-commits mailing list