You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by du...@apache.org on 2006/01/17 22:40:52 UTC

svn commit: r369926 - /webservices/axis/trunk/java/src/org/apache/axis/configuration/FileProvider.java

Author: dug
Date: Tue Jan 17 13:40:49 2006
New Revision: 369926

URL: http://svn.apache.org/viewcvs?rev=369926&view=rev
Log:
When you can't find the service just return null not an exception.
When the code is just looking to see if a service exists it shouldn't
have to deal with the overhead of catching and creating an exception,
returning null is the right way to do it.

Modified:
    webservices/axis/trunk/java/src/org/apache/axis/configuration/FileProvider.java

Modified: webservices/axis/trunk/java/src/org/apache/axis/configuration/FileProvider.java
URL: http://svn.apache.org/viewcvs/webservices/axis/trunk/java/src/org/apache/axis/configuration/FileProvider.java?rev=369926&r1=369925&r2=369926&view=diff
==============================================================================
--- webservices/axis/trunk/java/src/org/apache/axis/configuration/FileProvider.java (original)
+++ webservices/axis/trunk/java/src/org/apache/axis/configuration/FileProvider.java Tue Jan 17 13:40:49 2006
@@ -229,10 +229,12 @@
      */
     public SOAPService getService(QName qname) throws ConfigurationException {
         SOAPService service = deployment.getService(qname);
+        /* Why????? Dug
         if (service == null) {
             throw new ConfigurationException(Messages.getMessage("noService10",
                                                            qname.toString()));
         }
+        */
         return service;
     }