slawekjaranowski commented on pull request #486:
URL: https://github.com/apache/maven-surefire/pull/486#issuecomment-1066097475


   Never mind if we use one or two exceptions - this change add code, like:
   
   ``` java
   throw firstForkException == null
                   ? new MojoExecutionException( msg )
                   : new MojoExecutionException( msg, firstForkException );
   ```
   
   IMHO it is not necessary the same result is:
   
   ```java
   throw new MojoExecutionException( msg, firstForkException );
   ```
   
   we needn't check if cause is null or not
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to