User: schaefera Date: 01/11/01 22:07:48 Modified: src/main/org/jboss/verifier/strategy AbstractVerifier.java Log: Fixed the bug 441291 therefore that a message selector can have leading and trailing spaces as well as line breaks and still does not break (leading and trailing spaces are remoed, line breaks as converted to a space and an empty string is converted to null meaning not restrictions). Scheduler does not register at the JNDI server as well as supports named attributes instead of only constructors arguments. Abstract Verifier checks not for RemoteException compliant with the RMI spec. Revision Changes Path 1.23 +19 -6 jboss/src/main/org/jboss/verifier/strategy/AbstractVerifier.java Index: AbstractVerifier.java =================================================================== RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/verifier/strategy/AbstractVerifier.java,v retrieving revision 1.22 retrieving revision 1.23 diff -u -r1.22 -r1.23 --- AbstractVerifier.java 2001/10/29 13:24:32 1.22 +++ AbstractVerifier.java 2001/11/02 06:07:48 1.23 @@ -19,7 +19,7 @@ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA * * This package and its source code is available at www.jboss.org - * $Id: AbstractVerifier.java,v 1.22 2001/10/29 13:24:32 negaton Exp $ + * $Id: AbstractVerifier.java,v 1.23 2001/11/02 06:07:48 schaefera Exp $ */ // standard imports @@ -61,8 +61,16 @@ * @author <a href="mailto:[EMAIL PROTECTED]">Juha Lindfors</a> * @author Aaron Mulder ([EMAIL PROTECTED]) * @author Vinay Menon ([EMAIL PROTECTED]) + * @author <a href="mailto:[EMAIL PROTECTED]">Andreas Schaefer</a> * - * @version $Revision: 1.22 $ + * <p><b>Revisions:</b></p> + * <p><b>20011101: Andy</b> + * <ul> + * <li>Changed the throwRemoteException() method to check accordingly to the RMI spec.</li> + * </ul> + * </p> + * + * @version $Revision: 1.23 $ * @since JDK 1.3 */ public abstract class AbstractVerifier implements VerificationStrategy { @@ -178,11 +186,16 @@ 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; + for (int i = 0; i < exception.length; ++i) { +// Not true see bug report #434739 +// if (java.rmi.RemoteException.class.isAssignableFrom(exception[i])) +// According to the RMI spec. a remote interface must throw an RemoteException +// or any of its super classes therefore the check must be done vice versa + if( exception[ i ].isAssignableFrom( java.rmi.RemoteException.class ) ) { + return true; + } + } return false; }
_______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development