You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by el...@apache.org on 2020/06/30 16:49:14 UTC
[maven-dependency-plugin] branch close created (now 1cf7034)
This is an automated email from the ASF dual-hosted git repository.
elharo pushed a change to branch close
in repository https://gitbox.apache.org/repos/asf/maven-dependency-plugin.git.
at 1cf7034 use try with resources
This branch includes the following new commits:
new 1cf7034 use try with resources
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
[maven-dependency-plugin] 01/01: use try with resources
Posted by el...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
elharo pushed a commit to branch close
in repository https://gitbox.apache.org/repos/asf/maven-dependency-plugin.git
commit 1cf70340cc7761e2ccb34cc31cee60427070e0ce
Author: Elliotte Rusty Harold <el...@ibiblio.org>
AuthorDate: Tue Jun 30 12:48:52 2020 -0400
use try with resources
---
.../dependency/analyze/AnalyzeDuplicateMojo.java | 18 +++++---------
.../fromDependencies/BuildClasspathMojo.java | 28 ++++------------------
.../plugins/dependency/utils/DependencyUtil.java | 25 ++++++-------------
3 files changed, 17 insertions(+), 54 deletions(-)
diff --git a/src/main/java/org/apache/maven/plugins/dependency/analyze/AnalyzeDuplicateMojo.java b/src/main/java/org/apache/maven/plugins/dependency/analyze/AnalyzeDuplicateMojo.java
index ee06f57..79cb9ca 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/analyze/AnalyzeDuplicateMojo.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/analyze/AnalyzeDuplicateMojo.java
@@ -1,5 +1,7 @@
package org.apache.maven.plugins.dependency.analyze;
+import java.io.IOException;
+
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
@@ -37,8 +39,8 @@ import org.apache.maven.plugin.MojoFailureException;
import org.apache.maven.plugins.annotations.Mojo;
import org.apache.maven.plugins.annotations.Parameter;
import org.apache.maven.project.MavenProject;
-import org.codehaus.plexus.util.IOUtil;
import org.codehaus.plexus.util.ReaderFactory;
+import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
/**
* Analyzes the <code><dependencies/></code> and <code><dependencyManagement/></code> tags in the
@@ -85,21 +87,13 @@ public class AnalyzeDuplicateMojo
MavenXpp3Reader pomReader = new MavenXpp3Reader();
Model model = null;
- Reader reader = null;
- try
+ try ( Reader reader = ReaderFactory.newXmlReader( project.getFile() ) )
{
- reader = ReaderFactory.newXmlReader( project.getFile() );
model = pomReader.read( reader );
- reader.close();
- reader = null;
- }
- catch ( Exception e )
- {
- throw new MojoExecutionException( "IOException: " + e.getMessage(), e );
}
- finally
+ catch ( IOException | XmlPullParserException e )
{
- IOUtil.close( reader );
+ throw new MojoExecutionException( "Exception: " + e.getMessage(), e );
}
Set<String> duplicateDependencies = Collections.emptySet();
diff --git a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/BuildClasspathMojo.java b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/BuildClasspathMojo.java
index ea14b73..d055ad8 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/BuildClasspathMojo.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/BuildClasspathMojo.java
@@ -45,7 +45,6 @@ import org.apache.maven.plugins.dependency.utils.DependencyUtil;
import org.apache.maven.project.MavenProjectHelper;
import org.apache.maven.shared.artifact.filter.collection.ArtifactsFilter;
import org.apache.maven.shared.transfer.repository.RepositoryManager;
-import org.codehaus.plexus.util.IOUtil;
import org.codehaus.plexus.util.StringUtils;
/**
@@ -332,13 +331,9 @@ public class BuildClasspathMojo
// make sure the parent path exists.
out.getParentFile().mkdirs();
- Writer w = null;
- try
+ try ( Writer w = new BufferedWriter( new FileWriter( out ) ) )
{
- w = new BufferedWriter( new FileWriter( out ) );
w.write( cpString );
- w.close();
- w = null;
getLog().info( "Wrote classpath file '" + out + "'." );
}
catch ( IOException ex )
@@ -346,18 +341,14 @@ public class BuildClasspathMojo
throw new MojoExecutionException( "Error while writing to classpath file '" + out + "': " + ex.toString(),
ex );
}
- finally
- {
- IOUtil.close( w );
- }
}
/**
* Reads into a string the file specified by the mojo param 'outputFile'. Assumes, the instance variable
* 'outputFile' is not null.
*
- * @return the string contained in the classpathFile, if exists, or null otherwise.
- * @throws IOException in case of an error.
+ * @return the string contained in the classpathFile, if it exists, or null otherwise
+ * @throws IOException in case of an error
*/
protected String readClasspathFile()
throws IOException
@@ -373,26 +364,15 @@ public class BuildClasspathMojo
return null;
}
StringBuilder sb = new StringBuilder();
- BufferedReader r = null;
-
- try
+ try ( BufferedReader r = new BufferedReader( new FileReader( outputFile ) ) )
{
- r = new BufferedReader( new FileReader( outputFile ) );
-
for ( String line = r.readLine(); line != null; line = r.readLine() )
{
sb.append( line );
}
- r.close();
- r = null;
-
return sb.toString();
}
- finally
- {
- IOUtil.close( r );
- }
}
/**
diff --git a/src/main/java/org/apache/maven/plugins/dependency/utils/DependencyUtil.java b/src/main/java/org/apache/maven/plugins/dependency/utils/DependencyUtil.java
index 5af5a42..eacedbf 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/utils/DependencyUtil.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/utils/DependencyUtil.java
@@ -29,7 +29,6 @@ import java.util.Objects;
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.ArtifactUtils;
import org.apache.maven.plugin.logging.Log;
-import org.codehaus.plexus.util.IOUtil;
import org.codehaus.plexus.util.StringUtils;
/**
@@ -227,22 +226,11 @@ public final class DependencyUtil
public static synchronized void write( String string, File file, boolean append, Log log )
throws IOException
{
- file.getParentFile().mkdirs();
+ file.getParentFile().mkdirs();
- FileWriter writer = null;
-
- try
+ try ( FileWriter writer = new FileWriter( file, append ) )
{
- writer = new FileWriter( file, append );
-
writer.write( string );
-
- writer.close();
- writer = null;
- }
- finally
- {
- IOUtil.close( writer );
}
}
@@ -250,7 +238,7 @@ public final class DependencyUtil
* Writes the specified string to the log at info level.
*
* @param string the string to write
- * @param log where to log information.
+ * @param log where to log information
* @throws IOException if an I/O error occurs
*/
public static synchronized void log( String string, Log log )
@@ -279,9 +267,10 @@ public final class DependencyUtil
}
/**
- * clean up configuration string before it can be tokenized
- * @param str The str which should be cleaned.
- * @return cleaned up string.
+ * Clean up configuration string before it can be tokenized.
+ *
+ * @param str the string which should be cleaned
+ * @return cleaned up string
*/
public static String cleanToBeTokenizedString( String str )
{