From 04b5ab2e303a1a09fa2b942c264b16df6d6d472a Mon Sep 17 00:00:00 2001
From: brandjon <brandjon@google.com>
Date: Thu, 11 Jan 2018 09:09:56 -0800
Subject: Refactor SkylarkInfo constructors

- Info now has one protected constructor. (Would've preferred the builder pattern, but inheritance makes it much more verbose.)
- Direct SkylarkInfo subclass access is replaced by factory methods and an isCompact() accessor.
- Added/simplified tests

RELNOTES: None
PiperOrigin-RevId: 181616757
---
 .../java/com/google/devtools/build/lib/analysis/ActionsProvider.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/main/java/com/google/devtools/build/lib/analysis')

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java b/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java
index 429f419b2c..105dbd819e 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java
@@ -46,6 +46,6 @@ public final class ActionsProvider {
       }
     }
     ImmutableMap<String, Object> fields = ImmutableMap.<String, Object>of("by_file", map);
-    return SkylarkInfo.fromMap(SKYLARK_CONSTRUCTOR, fields, Location.BUILTIN);
+    return SkylarkInfo.createSchemaless(SKYLARK_CONSTRUCTOR, fields, Location.BUILTIN);
   }
 }
-- 
cgit v1.2.3