svn commit: r614725 - /commons/sandbox/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java

[sgoeschl]

Author: sgoeschl
Date: Wed Jan 23 15:06:59 2008
New Revision: 614725

URL: http://svn.apache.org/viewvc?rev=614725&view=rev
Log:
Removed "// TODO check exitValue and throw if not OK" in execute() since the 
exit value is already checked in executeInternal()

Modified:
    
commons/sandbox/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java

Modified: 
commons/sandbox/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java?rev=614725&r1=614724&r2=614725&view=diff
==============================================================================
--- 
commons/sandbox/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java
 (original)
+++ 
commons/sandbox/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java
 Wed Jan 23 15:06:59 2008
@@ -179,11 +179,8 @@
              */
             public void run() {
                 int exitValue = Executor.INVALID_EXITVALUE;
-                try {
-                    
+                try {                    
                     exitValue = executeInternal(command, environment, 
workingDirectory, streamHandler);
-
-                    // TODO check exitValue and throw if not OK
                     handler.onProcessComplete(exitValue);
                 } catch (ExecuteException e) {
                     handler.onProcessFailed(e);