commit f06e370e176ecffdc52e03e644a0db2ef38cf484
author Klaus Aehlig <aehlig@google.com> Mon Sep 18 19:17:57 2017 +0200
committer László Csomor <laszlocsomor@google.com> Tue Sep 19 09:32:13 2017 +0200
tree 673c30db29d23653c6a3973c06d491c5e7bc1a97
parent 06d1988534c17a7fb69afe5fc1f9725efe246b37

Unify buildevnts indicating build completion

...by factoring out the common code parts.

Change-Id: I2ead39ea218dd535af9684a81a7bbe29b558aa27
PiperOrigin-RevId: 169106854

src/main/java/com/google/devtools/build/lib/analysis/NoBuildRequestFinishedEvent.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/NoBuildRequestFinishedEvent.java b/src/main/java/com/google/devtools/build/lib/analysis/NoBuildRequestFinishedEvent.java
index 27ee93a..f1294f2 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/NoBuildRequestFinishedEvent.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/NoBuildRequestFinishedEvent.java
@@ -14,47 +14,12 @@
 
 package com.google.devtools.build.lib.analysis;
 
-import com.google.common.collect.ImmutableList;
-import com.google.devtools.build.lib.buildeventstream.BuildEvent;
-import com.google.devtools.build.lib.buildeventstream.BuildEventConverters;
-import com.google.devtools.build.lib.buildeventstream.BuildEventId;
-import com.google.devtools.build.lib.buildeventstream.BuildEventStreamProtos;
-import com.google.devtools.build.lib.buildeventstream.GenericBuildEvent;
+import com.google.devtools.build.lib.buildeventstream.BuildCompletingEvent;
 import com.google.devtools.build.lib.util.ExitCode;
-import java.util.Collection;
 
 /** {@link BuildEvent} indicating that a request that does not involve building as finished. */
-public final class NoBuildRequestFinishedEvent implements BuildEvent {
-  private final ExitCode exitCode;
-  private final long finishTimeMillis;
-
+public final class NoBuildRequestFinishedEvent extends BuildCompletingEvent {
   public NoBuildRequestFinishedEvent(ExitCode exitCode, long finishTimeMillis) {
-    this.exitCode = exitCode;
-    this.finishTimeMillis = finishTimeMillis;
-  }
-
-  @Override
-  public Collection<BuildEventId> getChildrenEvents() {
-    return ImmutableList.<BuildEventId>of();
-  }
-
-  @Override
-  public BuildEventId getEventId() {
-    return BuildEventId.buildFinished();
-  }
-
-  @Override
-  public BuildEventStreamProtos.BuildEvent asStreamProto(BuildEventConverters converters) {
-    return GenericBuildEvent.protoChaining(this)
-        .setFinished(
-            BuildEventStreamProtos.BuildFinished.newBuilder()
-                .setExitCode(
-                    BuildEventStreamProtos.BuildFinished.ExitCode.newBuilder()
-                        .setName(exitCode.name())
-                        .setCode(exitCode.getNumericExitCode())
-                        .build())
-                .setFinishTimeMillis(finishTimeMillis)
-                .build())
-        .build();
+    super(exitCode, finishTimeMillis);
   }
 }