In /org/jboss/verifier/strategy/AbstractVerifier.java
throwsRemoteException
is defined as follows:

    /*
     * checks if the method includes java.rmi.RemoteException or its
     * subclass in its throws clause.
     */
    public boolean throwsRemoteException(Method method) {

        Class[] exception = method.getExceptionTypes();

        for (int i = 0; i < exception.length; ++i)
            if
(java.rmi.RemoteException.class.isAssignableFrom(exception[i]))
                return true;

        return false;
    }

Shouldn't that read something like:

    /*
     * checks if the method includes java.rmi.RemoteException or its
     * superclass in its throws clause.
     */
    public boolean throwsRemoteException(Method method) {

        Class[] exception = method.getExceptionTypes();

        for (int i = 0; i < exception.length; ++i)
            if (exception[i].class.isAssignableFrom
(java.rmi.RemoteException.class))
                return true;

        return false;
    }


We want to check whether we can find a declared exception to which we
can assign the RemoteException that is thrown by the container ( not
whether we could assign the declared to exception to RemoteException ).

        Thanks,

        Lars

_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com


_______________________________________________
JBoss-user mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-user

Reply via email to