Return-Path: Delivered-To: apmail-xml-rpc-dev-archive@xml.apache.org Received: (qmail 99844 invoked by uid 500); 7 Aug 2002 02:41:26 -0000 Mailing-List: contact rpc-dev-help@xml.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: rpc-dev@xml.apache.org Delivered-To: mailing list rpc-dev@xml.apache.org Received: (qmail 99834 invoked by uid 500); 7 Aug 2002 02:41:26 -0000 Delivered-To: apmail-xml-rpc-cvs@apache.org Received: (qmail 99829 invoked from network); 7 Aug 2002 02:41:26 -0000 Received: from icarus.apache.org (63.251.56.143) by daedalus.apache.org with SMTP; 7 Aug 2002 02:41:26 -0000 Received: (qmail 65567 invoked by uid 1164); 7 Aug 2002 02:41:25 -0000 Date: 7 Aug 2002 02:41:25 -0000 Message-ID: <20020807024125.65566.qmail@icarus.apache.org> From: dlr@apache.org To: xml-rpc-cvs@apache.org Subject: cvs commit: xml-rpc/src/java/org/apache/xmlrpc XmlRpcServer.java X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Status: O X-Status: X-Keywords: dlr 2002/08/06 19:41:25 Modified: src/java/org/apache/xmlrpc XmlRpcServer.java Log: Invoker.execute(): Added check for unsupported void return type on handler methods. The need for better error reporting for this snafu was pointed out on the user list by Dejan Bosanace . Revision Changes Path 1.25 +7 -1 xml-rpc/src/java/org/apache/xmlrpc/XmlRpcServer.java Index: XmlRpcServer.java =================================================================== RCS file: /home/cvs/xml-rpc/src/java/org/apache/xmlrpc/XmlRpcServer.java,v retrieving revision 1.24 retrieving revision 1.25 diff -u -u -r1.24 -r1.25 --- XmlRpcServer.java 20 Mar 2002 15:11:03 -0000 1.24 +++ XmlRpcServer.java 7 Aug 2002 02:41:25 -0000 1.25 @@ -547,6 +547,12 @@ // It is some other exception throw new Exception(t.toString()); } + if (returnValue == null && method.getReturnType() == Void.TYPE) + { + // Not supported by the spec. + throw new IllegalArgumentException + ("void return types for handler methods not supported"); + } return returnValue; } }