User: squirest
  Date: 02/02/25 18:02:59

  Modified:    src/main/org/jboss/mx/interceptor MBeanInterceptor.java
                        MBeanInvocation.java
  Log:
  moved and removed stuff.  new items will go into capability until a better home can 
be found
  
  Revision  Changes    Path
  1.2       +3 -1      jmx/src/main/org/jboss/mx/interceptor/MBeanInterceptor.java
  
  Index: MBeanInterceptor.java
  ===================================================================
  RCS file: 
/cvsroot/jboss/jmx/src/main/org/jboss/mx/interceptor/MBeanInterceptor.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- MBeanInterceptor.java     29 Jan 2002 03:33:22 -0000      1.1
  +++ MBeanInterceptor.java     26 Feb 2002 02:02:59 -0000      1.2
  @@ -6,16 +6,18 @@
    */
   package org.jboss.mx.interceptor;
   
  +import org.jboss.mx.capability.ResourceInvoker;
  +
   
   /**
    * Base class for MBean interceptors
    *
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Juha Lindfors</a>.
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Trevor Squires</a>.
  - * @version $Revision: 1.1 $
  + * @version $Revision: 1.2 $
    *
    */
  -public class MBeanInterceptor
  +public class MBeanInterceptor implements ResourceInvoker
   {
   
      // Attributes ----------------------------------------------------
  
  
  
  1.2       +28 -13    jmx/src/main/org/jboss/mx/interceptor/MBeanInvocation.java
  
  Index: MBeanInvocation.java
  ===================================================================
  RCS file: /cvsroot/jboss/jmx/src/main/org/jboss/mx/interceptor/MBeanInvocation.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- MBeanInvocation.java      29 Jan 2002 03:33:22 -0000      1.1
  +++ MBeanInvocation.java      26 Feb 2002 02:02:59 -0000      1.2
  @@ -4,33 +4,36 @@
    * Distributable under LGPL license.
    * See terms of license at gnu.org.
    */
  +
   package org.jboss.mx.interceptor;
   
  +import javax.management.ObjectName;
  +
   
   /**
  - * Abstraction of the invocation that travels through the interceptor
  - * stack.
  + * Object representing an MBean invocation
    *
  - * QUERY THS - I can see this needing a Context object but also,
  - * should this be mutable with a placeholder for the return value?
  - * IMHO that seems more like an invocation as used by interceptors...
  + * Note to self: try to sneak in stateless interceptors so that nobody
  + * notices.
    *
  - * @see org.jboss.mx.interceptor.Interceptor
  - *
  - * @author  <a href="mailto:[EMAIL PROTECTED]";>Juha Lindfors</a>.
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Trevor Squires</a>.
  - * @version $Revision: 1.1 $
  - *
    */
   public class MBeanInvocation
   {
      // Constants -----------------------------------------------------
  +   // DYNAMIC MBEAN
      public final static int OPCODE_INVOKE = 1;
      public final static int OPCODE_GETATTRIBUTE = 2;
      public final static int OPCODE_GETATTRIBUTES = 3;
      public final static int OPCODE_SETATTRIBUTE = 4;
      public final static int OPCODE_SETATTRIBUTES = 5;
      public final static int OPCODE_GETMBEANINFO = 6;
  +   // NOTIFICATION BROADCASTER
  +   public final static int OPCODE_ADDNOTIFICATIONLISTENER = 7;
  +   public final static int OPCODE_REMOVENOTIFICATIONLISTENER = 8;
  +   public final static int OPCODE_GETNOTIFICATIONINFO = 9;
  +   // NOTIFICATION LISTENER
  +   public final static int OPCODE_HANDLENOTIFICATION = 10;
   
      public final static String[] REVERSE_OPCODES = {
         "OPCODE_UNKNOWN",
  @@ -39,13 +42,27 @@
         "OPCODE_GETATTRIBUTES",
         "OPCODE_SETATTRIBUTE",
         "OPCODE_SETATTRIBUTES",
  -      "OPCODE_GETMBEANINFO"
  +      "OPCODE_GETMBEANINFO",
  +      "OPCODE_ADDNOTIFICATIONLISTENER",
  +      "OPCODE_REMOVENOTIFICATIONLISTENER",
  +      "OPCODE_GETNOTIFICATIONINFO",
  +      "OPCODE_HANDLENOTIFICATION",
      };
   
      // Attributes ----------------------------------------------------
      private int opcode;
      private Object[] args;
   
  +   // FIXME - there's extra stuff that I think should go in here
  +   // perhaps as some sort of MBeanEnvironment object.
  +   // I believe that it's separate from any sort of "context" we allow because
  +   // IMHO the context should be independant of the Agent impl
  +   //
  +   // First, I think we should supply the ObjectName for the interceptors
  +   private ObjectName objectName;
  +   // Next, I think we should supply the MBean's classloader.
  +   private ClassLoader mbeanLoader;
  +
      // Constructors --------------------------------------------------
      public MBeanInvocation(int opcode, Object[] args)
      {
  @@ -63,6 +80,4 @@
      {
         return opcode;
      }
  -
   }
  -
  
  
  

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to