commit f6a7e5a39ee5669c0fdfce93032cab45d6c89358
author dslomov <dslomov@google.com> Wed Jul 05 07:23:31 2017 -0400
committer John Cater <jcater@google.com> Wed Jul 05 10:58:56 2017 -0400
tree 9905a5e85f19a94eb4cd0082d6a146d9e06a0c3b
parent 2843eadf4c1d9175c60d32ffd2cba2100256c1fd

Use the same data structure for native and Skylark providers.

The memory cost of adding Skylark provider is now the same as native.
Skylark providers (declared and legacy) benefit from the same
shape-sharing optimization as native providers.

RELNOTES: None.
PiperOrigin-RevId: 160944263

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

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProvider.java b/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProvider.java
index de538b9..9ee8933 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProvider.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/TransitiveInfoProvider.java
@@ -60,4 +60,19 @@
  * @see com.google.devtools.build.lib.rules.RuleConfiguredTargetFactory
  */
 public interface TransitiveInfoProvider {
+
+  /**
+   * Implement this to mark that a native provider should be exported with
+   * certain name to Skylark.
+   * Broken: only works for rules, not for aspects.
+   * DO NOT USE FOR NEW CODE!
+   *
+   * Use native declared providers
+   * ({@link com.google.devtools.build.lib.packages.NativeClassObjectConstructor}) to
+   * expose providers to both native and Skylark code.
+   */
+  @Deprecated
+  interface WithLegacySkylarkName {
+    String getSkylarkName();
+  }
 }