You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2020/11/21 17:22:48 UTC

[commons-exec] branch master updated: Remove useless returns.

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-exec.git


The following commit(s) were added to refs/heads/master by this push:
     new d36f987  Remove useless returns.
d36f987 is described below

commit d36f9876740c487e7ef699c405376495a336a510
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sat Nov 21 12:22:44 2020 -0500

    Remove useless returns.
---
 src/test/java/org/apache/commons/exec/DefaultExecutorTest.java | 2 --
 src/test/java/org/apache/commons/exec/issues/Exec36Test.java   | 1 -
 2 files changed, 3 deletions(-)

diff --git a/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java b/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
index ce68900..2c1b74e 100644
--- a/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
+++ b/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
@@ -566,11 +566,9 @@ public class DefaultExecutorTest {
                     .println("The code samples to do that in windows look like a joke ... :-( .., no way I'm doing that");
             System.err.println("The test 'testExecuteWithRedirectedStreams' does not support the following OS : "
                     + System.getProperty("os.name"));
-            return;
         } else {
             System.err.println("The test 'testExecuteWithRedirectedStreams' does not support the following OS : "
                     + System.getProperty("os.name"));
-            return;
         }
     }
 
diff --git a/src/test/java/org/apache/commons/exec/issues/Exec36Test.java b/src/test/java/org/apache/commons/exec/issues/Exec36Test.java
index f0e59bd..796b92a 100644
--- a/src/test/java/org/apache/commons/exec/issues/Exec36Test.java
+++ b/src/test/java/org/apache/commons/exec/issues/Exec36Test.java
@@ -101,7 +101,6 @@ public class Exec36Test {
         }
         else {
             System.err.println("The test 'testExec36_1' does not support the following OS : " + System.getProperty("os.name"));
-            return;
         }
     }