You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by jk...@apache.org on 2005/05/23 21:14:36 UTC
cvs commit: ant/src/main/org/apache/tools/ant ProjectHelper.java
jkf 2005/05/23 12:14:36
Modified: src/main/org/apache/tools/ant ProjectHelper.java
Log:
Catching Throwable while calling new on a no statement constructor (all fields / base classes only have no statement constructors as well) seems overdone. Removing catch clause.
Revision Changes Path
1.117 +1 -9 ant/src/main/org/apache/tools/ant/ProjectHelper.java
Index: ProjectHelper.java
===================================================================
RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/ProjectHelper.java,v
retrieving revision 1.116
retrieving revision 1.117
diff -u -r1.116 -r1.117
--- ProjectHelper.java 30 Mar 2005 08:35:43 -0000 1.116
+++ ProjectHelper.java 23 May 2005 19:14:36 -0000 1.117
@@ -219,15 +219,7 @@
if (helper != null) {
return helper;
} else {
- try {
- // Default
- // return new ProjectHelperImpl();
- return new ProjectHelper2();
- } catch (Throwable e) {
- String message = "Unable to load default ProjectHelper due to "
- + e.getClass().getName() + ": " + e.getMessage();
- throw new BuildException(message, e);
- }
+ return new ProjectHelper2();
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org