[jboss-cvs] jboss-seam/src/main/org/jboss/seam/contexts ...

Gavin King gavin.king at jboss.com
Tue Jul 25 16:15:17 EDT 2006


  User: gavin   
  Date: 06/07/25 16:15:17

  Modified:    src/main/org/jboss/seam/contexts  Contexts.java
  Log:
  cleanup all those static methods
  
  Revision  Changes    Path
  1.39      +2 -18     jboss-seam/src/main/org/jboss/seam/contexts/Contexts.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Contexts.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/contexts/Contexts.java,v
  retrieving revision 1.38
  retrieving revision 1.39
  diff -u -b -r1.38 -r1.39
  --- Contexts.java	16 Apr 2006 22:24:07 -0000	1.38
  +++ Contexts.java	25 Jul 2006 20:15:17 -0000	1.39
  @@ -18,7 +18,7 @@
    * 
    * @author Gavin King
    * @author <a href="mailto:theute at jboss.org">Thomas Heute</a>
  - * @version $Revision: 1.38 $
  + * @version $Revision: 1.39 $
    */
   public class Contexts {
   
  @@ -186,7 +186,7 @@
               {
                  try
                  {
  -                  callDestroyMethod( component, context.get(name) );
  +                  component.callDestroyMethod( context.get(name) );
                  }
                  catch (Exception e)
                  {
  @@ -201,20 +201,4 @@
         }
      }
   
  -   private static void callDestroyMethod(Component component, Object instance)
  -   {
  -      if ( component.hasDestroyMethod() )
  -      {
  -         String methodName = component.getDestroyMethod().getName();
  -         try {
  -            Method method = instance.getClass().getMethod(methodName);
  -            Reflections.invokeAndWrap( method, instance );
  -         }
  -         catch (NoSuchMethodException e)
  -         {
  -            log.warn("could not find destroy method", e);
  -         }
  -      }
  -   }
  -   
   }
  \ No newline at end of file
  
  
  



More information about the jboss-cvs-commits mailing list