From 70c31930941f0d9f22889b01e11ee8442dc5a827 Mon Sep 17 00:00:00 2001 From: Googler Date: Tue, 20 Jun 2017 10:17:37 -0400 Subject: Document JavaSkylarkApiProvider static factory methods and remove one constructor. PiperOrigin-RevId: 159557168 --- .../build/lib/rules/java/JavaSkylarkApiProvider.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java index 7d799325da..de7dc82b41 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaSkylarkApiProvider.java @@ -49,19 +49,21 @@ public final class JavaSkylarkApiProvider extends SkylarkApiProvider { @Nullable private final TransitiveInfoProviderMap transitiveInfoProviderMap; + /** + * Creates a Skylark API provider that reads information from its associated target's providers. + */ public static JavaSkylarkApiProvider fromRuleContext() { - return new JavaSkylarkApiProvider(); + return new JavaSkylarkApiProvider(null); } + /** + * Creates a Skylark API provider that reads information from an explicit provider map. + */ public static JavaSkylarkApiProvider fromProviderMap( TransitiveInfoProviderMap transitiveInfoProviderMap) { return new JavaSkylarkApiProvider(transitiveInfoProviderMap); } - private JavaSkylarkApiProvider() { - this.transitiveInfoProviderMap = null; - } - private JavaSkylarkApiProvider(TransitiveInfoProviderMap transitiveInfoProviderMap) { this.transitiveInfoProviderMap = transitiveInfoProviderMap; } -- cgit v1.2.3