diff --git a/src/main/org/apache/tools/ant/AntClassLoader.java b/src/main/org/apache/tools/ant/AntClassLoader.java index 61cacceb9..103b118e7 100644 --- a/src/main/org/apache/tools/ant/AntClassLoader.java +++ b/src/main/org/apache/tools/ant/AntClassLoader.java @@ -1407,12 +1407,7 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener { */ public synchronized void cleanup() { for (final Enumeration e = jarFiles.elements(); e.hasMoreElements();) { - final JarFile jarFile = e.nextElement(); - try { - jarFile.close(); - } catch (final IOException ioe) { - // ignore - } + FileUtils.close(e.nextElement()); } jarFiles = new Hashtable(); if (project != null) { diff --git a/src/main/org/apache/tools/ant/taskdefs/LogStreamHandler.java b/src/main/org/apache/tools/ant/taskdefs/LogStreamHandler.java index ed16cf36f..bc69c0668 100644 --- a/src/main/org/apache/tools/ant/taskdefs/LogStreamHandler.java +++ b/src/main/org/apache/tools/ant/taskdefs/LogStreamHandler.java @@ -23,6 +23,7 @@ import java.io.IOException; import org.apache.tools.ant.BuildException; import org.apache.tools.ant.ProjectComponent; import org.apache.tools.ant.Task; +import org.apache.tools.ant.util.FileUtils; /** * Logs standard output and error of a subprocess to the log system of ant. @@ -59,12 +60,7 @@ public class LogStreamHandler extends PumpStreamHandler { */ public void stop() { super.stop(); - try { - getErr().close(); - getOut().close(); - } catch (IOException e) { - // plain impossible - throw new BuildException(e); - } + FileUtils.close(getErr()); + FileUtils.close(getOut()); } } diff --git a/src/main/org/apache/tools/ant/taskdefs/SQLExec.java b/src/main/org/apache/tools/ant/taskdefs/SQLExec.java index 6debd6d35..c4076bc20 100644 --- a/src/main/org/apache/tools/ant/taskdefs/SQLExec.java +++ b/src/main/org/apache/tools/ant/taskdefs/SQLExec.java @@ -692,19 +692,11 @@ public class SQLExec extends JDBCTask { } } finally { try { - if (getStatement() != null) { - getStatement().close(); - } - } catch (SQLException ex) { - // ignore - } - try { - if (getConnection() != null) { - getConnection().close(); - } + FileUtils.close(getStatement()); } catch (SQLException ex) { // ignore } + FileUtils.close(getConnection()); } log(goodSql + " of " + totalSql + " SQL statements executed successfully"); diff --git a/src/main/org/apache/tools/ant/taskdefs/condition/Socket.java b/src/main/org/apache/tools/ant/taskdefs/condition/Socket.java index d6a69ec81..43a40e022 100644 --- a/src/main/org/apache/tools/ant/taskdefs/condition/Socket.java +++ b/src/main/org/apache/tools/ant/taskdefs/condition/Socket.java @@ -23,6 +23,7 @@ import java.io.IOException; import org.apache.tools.ant.BuildException; import org.apache.tools.ant.Project; import org.apache.tools.ant.ProjectComponent; +import org.apache.tools.ant.util.FileUtils; /** * Condition to wait for a TCP/IP socket to have a listener. Its attributes are: @@ -73,13 +74,7 @@ public class Socket extends ProjectComponent implements Condition { } catch (IOException e) { return false; } finally { - if (s != null) { - try { - s.close(); - } catch (IOException ioe) { - // Intentionally left blank - } - } + FileUtils.close(s); } return true; } diff --git a/src/main/org/apache/tools/ant/taskdefs/cvslib/RedirectingStreamHandler.java b/src/main/org/apache/tools/ant/taskdefs/cvslib/RedirectingStreamHandler.java index 713de0c4b..e9b755d6a 100644 --- a/src/main/org/apache/tools/ant/taskdefs/cvslib/RedirectingStreamHandler.java +++ b/src/main/org/apache/tools/ant/taskdefs/cvslib/RedirectingStreamHandler.java @@ -22,6 +22,7 @@ import java.io.IOException; import org.apache.tools.ant.BuildException; import org.apache.tools.ant.taskdefs.PumpStreamHandler; +import org.apache.tools.ant.util.FileUtils; /** * A dummy stream handler that just passes stuff to the parser. @@ -49,13 +50,8 @@ class RedirectingStreamHandler public void stop() { super.stop(); - try { - getErr().close(); - getOut().close(); - } catch (final IOException e) { - // plain impossible - throw new BuildException(e); - } + FileUtils.close(getErr()); + FileUtils.close(getOut()); } } diff --git a/src/main/org/apache/tools/ant/util/depend/AbstractAnalyzer.java b/src/main/org/apache/tools/ant/util/depend/AbstractAnalyzer.java index 5c95d75b8..eb5f97ad3 100644 --- a/src/main/org/apache/tools/ant/util/depend/AbstractAnalyzer.java +++ b/src/main/org/apache/tools/ant/util/depend/AbstractAnalyzer.java @@ -267,16 +267,10 @@ public abstract class AbstractAnalyzer implements DependencyAnalyzer { } } else { // must be a zip of some sort - ZipFile zipFile = null; - try { - zipFile = new ZipFile(element); + try (ZipFile zipFile = new ZipFile(element)) { if (zipFile.getEntry(resourceLocation) != null) { return element; } - } finally { - if (zipFile != null) { - zipFile.close(); - } } } }