This is an automated email from the ASF dual-hosted git repository. rfscholte pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/maven-release.git
The following commit(s) were added to refs/heads/master by this push: new 57d6b59 Remove unused private fields Remove constructors with unused parameters 57d6b59 is described below commit 57d6b591cabfc1b1b417db28f723fbb9a94dc989 Author: Piotrek Żygieło <pzygi...@users.noreply.github.com> AuthorDate: Fri May 15 09:05:50 2020 +0200 Remove unused private fields Remove constructors with unused parameters Signed-off-by: rfscholte <rfscho...@apache.org> --- .../maven/shared/release/exec/ForkedMavenExecutor.java | 5 ++--- .../maven/shared/release/exec/InvokerMavenExecutor.java | 2 +- .../shared/release/exec/MavenExecutorException.java | 16 +--------------- 3 files changed, 4 insertions(+), 19 deletions(-) diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java index e4a5539..af0051f 100644 --- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java +++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java @@ -141,13 +141,12 @@ public class ForkedMavenExecutor if ( result != 0 ) { - throw new MavenExecutorException( "Maven execution failed, exit code: \'" + result + "\'", result, - stdOut.toString(), stdErr.toString() ); + throw new MavenExecutorException( "Maven execution failed, exit code: \'" + result + "\'", result ); } } catch ( CommandLineException e ) { - throw new MavenExecutorException( "Can't run goal " + goals, stdOut.toString(), stdErr.toString(), e ); + throw new MavenExecutorException( "Can't run goal " + goals, e ); } finally { diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java index df61c39..eb9e044 100644 --- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java +++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java @@ -392,7 +392,7 @@ public class InvokerMavenExecutor { throw new MavenExecutorException( "Maven execution failed, exit code: \'" + invocationResult.getExitCode() + "\'", - invocationResult.getExitCode(), "", "" ); + invocationResult.getExitCode() ); } } catch ( MavenInvocationException e ) diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java index 3bd996c..74300c7 100644 --- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java +++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java @@ -27,27 +27,13 @@ package org.apache.maven.shared.release.exec; public class MavenExecutorException extends Exception { - private String stdErr; - - private String stdOut; - private int exitCode; - public MavenExecutorException( String message, int exitCode, String stdOut, String stdErr ) + public MavenExecutorException( String message, int exitCode ) { super( message ); this.exitCode = exitCode; - this.stdOut = stdOut; - this.stdErr = stdErr; - } - - public MavenExecutorException( String message, String stdOut, String stdErr, Exception e ) - { - super( message, e ); - - this.stdOut = stdOut; - this.stdErr = stdErr; } public MavenExecutorException( String message, Throwable cause )