You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beehive.apache.org by ek...@apache.org on 2006/04/02 17:58:44 UTC

svn commit: r390840 - in /beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb: EJBControlImpl.java SessionEJBControlImpl.java

Author: ekoneil
Date: Sun Apr  2 08:58:42 2006
New Revision: 390840

URL: http://svn.apache.org/viewcvs?rev=390840&view=rev
Log:
Small changes to the EJBControl to suppress a warning about a varargs method.

BB: self
Test: EJBControl build pass


Modified:
    beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/EJBControlImpl.java
    beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/SessionEJBControlImpl.java

Modified: beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/EJBControlImpl.java
URL: http://svn.apache.org/viewcvs/beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/EJBControlImpl.java?rev=390840&r1=390839&r2=390840&view=diff
==============================================================================
--- beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/EJBControlImpl.java (original)
+++ beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/EJBControlImpl.java Sun Apr  2 08:58:42 2006
@@ -62,8 +62,7 @@
     public void onCreate() {
         // first time initialization
 
-        // assume v2 attributes
-        EJBHome ejbHome = (EJBHome) context.getControlPropertySet(EJBHome.class);
+        EJBHome ejbHome = context.getControlPropertySet(EJBHome.class);
         if (ejbHome == null)
             throw new ControlException("No @EJBHome property is defined");
 
@@ -84,8 +83,8 @@
             // todo: doesn't work this way in Geronimo!
             _jndiName = JNDI_APPSCOPED_PREFIX + EJBInfo.getEJBRefName(context.getControlInterface());
 
-	    // todo: works this way in geronimo, determine which is correct!
-//            _jndiName = EJBInfo.getEJBRefName(context.getControlInterface());
+            // todo: works this way in geronimo, determine which is correct!
+            // _jndiName = EJBInfo.getEJBRefName(context.getControlInterface());
         }
 
         // Obtain the JCX interface and identify the home/remote

Modified: beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/SessionEJBControlImpl.java
URL: http://svn.apache.org/viewcvs/beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/SessionEJBControlImpl.java?rev=390840&r1=390839&r2=390840&view=diff
==============================================================================
--- beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/SessionEJBControlImpl.java (original)
+++ beehive/trunk/system-controls/src/ejb/org/apache/beehive/controls/system/ejb/SessionEJBControlImpl.java Sun Apr  2 08:58:42 2006
@@ -64,7 +64,7 @@
         try {
             /* todo: for efficiency, this could be done once per home interface and cached */
             Method createMethod = _homeInterface.getMethod("create", new Class []{});
-            Object beanInstance = createMethod.invoke(_homeInstance, null);
+            Object beanInstance = createMethod.invoke(_homeInstance, (Object[])null);
             _autoCreated = true;
             return beanInstance;
         }