Remove an unused throws clause, and make one more specific

--
PiperOrigin-RevId: 147716435
MOS_MIGRATED_REVID=147716435

2 files changed, 2 insertions, 2 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/shell/Command.java b/src/main/java/com/google/devtools/build/lib/shell/Command.java
index d0571154af..2e13636283 100644
--- a/src/main/java/com/google/devtools/build/lib/shell/Command.java
+++ b/src/main/java/com/google/devtools/build/lib/shell/Command.java
@@ -698,7 +698,7 @@ public final class Command {
       final Consumers.OutErrConsumers outErrConsumers,
       final boolean killSubprocessOnInterrupt,
       final boolean closeOutputStreams)
-      throws CommandException {
+      throws ExecFailedException {
 
     logCommand();
 
diff --git a/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java b/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java
index 52184909df..dac751669d 100644
--- a/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java
@@ -120,7 +120,7 @@ public class JavaSubprocessFactory implements Subprocess.Factory {
    * Returns a {@link ProcessBuilder.Redirect} appropriate for the parameters. If a file redirected
    * to exists, deletes the file before redirecting to it.
    */
-  private Redirect getRedirect(StreamAction action, File file) throws IOException {
+  private Redirect getRedirect(StreamAction action, File file) {
     switch (action) {
       case DISCARD:
         return Redirect.to(new File("/dev/null"));