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;
}