commit a11e2d068a4ae10f851612efa9dd5eacc7f54329
author adonovan <adonovan@google.com> Fri Dec 06 07:11:35 2019 -0800
committer Copybara-Service <copybara-worker@google.com> Fri Dec 06 07:12:35 2019 -0800
tree 23f9a146956ba6323b1cb289ea639b5b48284eb2
parent fd0b46b437fe2b698b1db87d29dffe55f6bfeead

bazel packages: rename InfoInterface to Info

Info is now an interface. The logic of the former abstract class of that name
has been pushed down into it subclasses, most of which do not need to materialize
Location and Provider as fields. This saved two words per Info instance.

PiperOrigin-RevId: 284178767

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

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProviderMapBuilder.java b/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProviderMapBuilder.java
index 3d87e47..b7f6669 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProviderMapBuilder.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProviderMapBuilder.java
@@ -15,7 +15,7 @@
 package com.google.devtools.build.lib.analysis;
 
 import com.google.common.base.Preconditions;
-import com.google.devtools.build.lib.packages.InfoInterface;
+import com.google.devtools.build.lib.packages.Info;
 import com.google.devtools.build.lib.packages.Provider;
 import java.util.Arrays;
 import java.util.LinkedHashMap;
@@ -49,8 +49,7 @@
     Preconditions.checkNotNull(providerClass);
     Preconditions.checkNotNull(provider);
     Preconditions.checkState(
-        !(provider instanceof InfoInterface), "Expose %s as native declared provider",
-        providerClass);
+        !(provider instanceof Info), "Expose %s as native declared provider", providerClass);
 
     // TODO(arielb): throw an exception if the providerClass is already present?
     // This is enforced by aspects but RuleConfiguredTarget presents violations
@@ -59,7 +58,7 @@
     return this;
   }
 
-  public TransitiveInfoProviderMapBuilder put(InfoInterface classObject) {
+  public TransitiveInfoProviderMapBuilder put(Info classObject) {
     Preconditions.checkNotNull(classObject);
     Preconditions.checkState(!(classObject instanceof TransitiveInfoProvider),
         "Declared provider %s should not implement TransitiveInfoProvider",
@@ -105,8 +104,8 @@
   }
 
   @Nullable
-  public InfoInterface getProvider(Provider.Key key) {
-    return (InfoInterface) providers.get(key);
+  public Info getProvider(Provider.Key key) {
+    return (Info) providers.get(key);
   }
 
   public TransitiveInfoProviderMap build() {