commit a6a9910ad8d5a22bfabfb20a92575090668f7a32
author ulfjack <ulfjack@google.com> Tue Jun 13 17:15:45 2017 +0200
committer Yun Peng <pcloudy@google.com> Wed Jun 14 13:13:21 2017 +0200
tree e54f7a4ec13faab5aec3b13756b2057dbd162b3f
parent 9323f3b8afb02df8ac359fb5f6805ba94361591f

Simplify BlazeModule.beforeCommand

Don't pass the Command annotation explicitly, but add it to CommandEnvironment
instead; most modules don't need it in the first place, so it was a lot of
boilerplate for not much. Also change it so that the command is passed to the
constructor.

Add some documentation to the beforeCommand method.

PiperOrigin-RevId: 158847128

src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java b/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java
index 0061e3f..ee2ca95 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java
@@ -44,7 +44,7 @@
   private boolean discardActions;
 
   @Override
-  public void beforeCommand(Command command, CommandEnvironment env) {
+  public void beforeCommand(CommandEnvironment env) {
     this.reporter = env.getReporter();
     this.eventBus = env.getEventBus();
     eventBus.register(this);