remove ActionMetadata#getInputCount

Removing ActionMetadata#getInputCount instead of relying on all
implementations of ActionMetadata keeping getInputCount and
getInputs in sync. getInputCount is only used by a few tests
and has no applications in the main code at the moment, so it
is trivial to remove before we may come to rely on it more or its
semantics get more complicated.

--
MOS_MIGRATED_REVID=89130009

1 files changed, 0 insertions, 16 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java b/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
index 324d0b4608..f686334815 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
@@ -61,7 +61,6 @@ public abstract class AbstractAction implements Action {
   private Iterable<Artifact> inputs;
   private final ImmutableSet<Artifact> outputs;
 
-  private int cachedInputCount = -1;
   private String cachedKey;
 
   /**
@@ -131,21 +130,6 @@ public abstract class AbstractAction implements Action {
   protected void setInputs(Iterable<Artifact> inputs) {
     Preconditions.checkState(discoversInputs(), this);
     this.inputs = CollectionUtils.makeImmutable(inputs);
-    cachedInputCount = -1;
-  }
-
-  /*
-   * Get count of inputs.
-   *
-   * <p>Computes the count on first invocation, returns cached value for further invocations.
-   */
-  @Override
-  @ThreadSafe
-  public synchronized int getInputCount() {
-    if (cachedInputCount == -1) {
-      cachedInputCount = Iterables.size(getInputs());
-    }
-    return cachedInputCount;
   }
 
   @Override