Remove the now-unused BlazeModule.beforeCommand variant. -- MOS_MIGRATED_REVID=103276489
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java index 4c6a57f..d4cf161 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
@@ -262,7 +262,7 @@ AbruptExitException exitCausingException = null; for (BlazeModule module : runtime.getBlazeModules()) { try { - module.beforeCommand(runtime, commandAnnotation, env); + module.beforeCommand(commandAnnotation, env); } catch (AbruptExitException e) { // Don't let one module's complaints prevent the other modules from doing necessary // setup. We promised to call beforeCommand exactly once per-module before each command
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeModule.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeModule.java index 97d9b21..a3675da 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeModule.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeModule.java
@@ -225,15 +225,6 @@ } /** - * Called before each command. - */ - @SuppressWarnings("unused") - public void beforeCommand(BlazeRuntime blazeRuntime, Command command, CommandEnvironment env) - throws AbruptExitException { - beforeCommand(command, env); - } - - /** * Returns the output service to be used. It is an error if more than one module returns an * output service. *