[jboss-cvs] JBossAS SVN: r106574 - in projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3: stateless and 1 other directory.

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

Author: jaikiran
Date: 2010-07-12 03:49:14 -0400 (Mon, 12 Jul 2010)
New Revision: 106574

Modified:
   projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/mdb/MessagingContainer.java
   projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/stateless/StatelessContainer.java
Log:
EJBTHREE-2125 No need to reset method accessibility since the methods are only used by AOP

Modified: projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/mdb/MessagingContainer.java
===================================================================
--- projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/mdb/MessagingContainer.java	2010-07-12 06:39:56 UTC (rev 106573)
+++ projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/mdb/MessagingContainer.java	2010-07-12 07:49:14 UTC (rev 106574)
@@ -660,8 +660,6 @@
       // get hold of the unadvised method, so that we can mark it accessible
       // for the duration of this call (Remember, timeout methods can be with private, protected, package access modifier)
       Method unadvisedMethod = info.getUnadvisedMethod();
-      // store the old value of method accesibility, so that it can be restored later 
-      boolean unadvisedMethodAccessibilityOldVal = unadvisedMethod.isAccessible();
       // mark as accessible before invoking
       unadvisedMethod.setAccessible(true);
       try
@@ -680,10 +678,6 @@
          if (throwable instanceof Exception) throw (Exception) throwable;
          throw new RuntimeException(throwable);
       }
-      finally
-      {
-         // reset the timeout method accessibility
-         unadvisedMethod.setAccessible(unadvisedMethodAccessibilityOldVal);
-      }
+      
    }
 }
\ No newline at end of file

Modified: projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/stateless/StatelessContainer.java
===================================================================
--- projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/stateless/StatelessContainer.java	2010-07-12 06:39:56 UTC (rev 106573)
+++ projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/stateless/StatelessContainer.java	2010-07-12 07:49:14 UTC (rev 106574)
@@ -669,8 +669,6 @@
          // get hold of the unadvised method, so that we can mark it accessible
          // for the duration of this call (Remember, timeout methods can be with private, protected, package access modifier)
          Method unadvisedMethod = info.getUnadvisedMethod();
-         // store the old value of method accesibility, so that it can be restored later 
-         boolean unadvisedMethodAccessibilityOldVal = unadvisedMethod.isAccessible();
          // mark as accessible before invoking
          unadvisedMethod.setAccessible(true);
 
@@ -693,9 +691,6 @@
          }
          finally
          {
-            // reset the timeout method accessibility
-            unadvisedMethod.setAccessible(unadvisedMethodAccessibilityOldVal);
-
             AllowedOperationsAssociation.popInMethodFlag();
 
          }