diff options
author | dslomov <dslomov@google.com> | 2017-07-31 21:07:51 +0200 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2017-08-01 12:21:52 +0200 |
commit | de965ac47e5dedf9dd43004d7523dbfa5ab64de1 (patch) | |
tree | 1bca577a04df1c739807a8d3490296717a3b7d7b /src/main/java/com/google/devtools/build/lib/rules/apple | |
parent | 24183314aaa7887d80d6c1b7b9ce613889bea8e5 (diff) |
Better names for declared providers-related classes.
Follows
https://docs.google.com/document/d/1aAIVWvHPERDz2cv_PCFGwr8dvh5FcAkENFoRsNS4clk/.
RELNOTES: None.
PiperOrigin-RevId: 163728291
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/apple')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/apple/ApplePlatform.java | 22 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/apple/XcodeVersionProperties.java | 11 |
2 files changed, 14 insertions, 19 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/apple/ApplePlatform.java b/src/main/java/com/google/devtools/build/lib/rules/apple/ApplePlatform.java index cd22873f97..6be2024464 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/apple/ApplePlatform.java +++ b/src/main/java/com/google/devtools/build/lib/rules/apple/ApplePlatform.java @@ -16,9 +16,9 @@ package com.google.devtools.build.lib.rules.apple; import com.google.common.collect.ImmutableSet; import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; -import com.google.devtools.build.lib.packages.ClassObjectConstructor; -import com.google.devtools.build.lib.packages.NativeClassObjectConstructor; -import com.google.devtools.build.lib.packages.SkylarkClassObject; +import com.google.devtools.build.lib.packages.Info; +import com.google.devtools.build.lib.packages.NativeProvider; +import com.google.devtools.build.lib.packages.Provider; import com.google.devtools.build.lib.skylarkinterface.SkylarkCallable; import com.google.devtools.build.lib.skylarkinterface.SkylarkModule; import com.google.devtools.build.lib.skylarkinterface.SkylarkModuleCategory; @@ -212,15 +212,13 @@ public enum ApplePlatform implements SkylarkValue { } /** Returns a Skylark struct that contains the instances of this enum. */ - public static SkylarkClassObject getSkylarkStruct() { - ClassObjectConstructor constructor = - new NativeClassObjectConstructor<SkylarkClassObject>( - SkylarkClassObject.class, "platforms") {}; + public static Info getSkylarkStruct() { + Provider constructor = new NativeProvider<Info>(Info.class, "platforms") {}; HashMap<String, Object> fields = new HashMap<>(); for (ApplePlatform type : values()) { fields.put(type.skylarkKey, type); } - return new SkylarkClassObject(constructor, fields); + return new Info(constructor, fields); } @Override @@ -288,15 +286,13 @@ public enum ApplePlatform implements SkylarkValue { } /** Returns a Skylark struct that contains the instances of this enum. */ - public static SkylarkClassObject getSkylarkStruct() { - ClassObjectConstructor constructor = - new NativeClassObjectConstructor<SkylarkClassObject>( - SkylarkClassObject.class, "platform_types") {}; + public static Info getSkylarkStruct() { + Provider constructor = new NativeProvider<Info>(Info.class, "platform_types") {}; HashMap<String, Object> fields = new HashMap<>(); for (PlatformType type : values()) { fields.put(type.skylarkKey, type); } - return new SkylarkClassObject(constructor, fields); + return new Info(constructor, fields); } @Override diff --git a/src/main/java/com/google/devtools/build/lib/rules/apple/XcodeVersionProperties.java b/src/main/java/com/google/devtools/build/lib/rules/apple/XcodeVersionProperties.java index da22fa547e..a75fae2ad2 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/apple/XcodeVersionProperties.java +++ b/src/main/java/com/google/devtools/build/lib/rules/apple/XcodeVersionProperties.java @@ -19,22 +19,21 @@ import com.google.common.base.Optional; import com.google.common.base.Strings; import com.google.common.collect.ImmutableMap; import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; -import com.google.devtools.build.lib.packages.NativeClassObjectConstructor; -import com.google.devtools.build.lib.packages.SkylarkClassObject; +import com.google.devtools.build.lib.packages.Info; +import com.google.devtools.build.lib.packages.NativeProvider; import java.util.Map; import javax.annotation.Nullable; /** A tuple containing information about a version of xcode and its properties. */ @Immutable -public class XcodeVersionProperties extends SkylarkClassObject { +public class XcodeVersionProperties extends Info { /** Skylark name for the XcodeVersionProperties provider. */ public static final String SKYLARK_NAME = "XcodeProperties"; /** Skylark constructor and identifier for XcodeVersionProperties provider. */ - public static final NativeClassObjectConstructor<XcodeVersionProperties> SKYLARK_CONSTRUCTOR = - new NativeClassObjectConstructor<XcodeVersionProperties>( - XcodeVersionProperties.class, SKYLARK_NAME) {}; + public static final NativeProvider<XcodeVersionProperties> SKYLARK_CONSTRUCTOR = + new NativeProvider<XcodeVersionProperties>(XcodeVersionProperties.class, SKYLARK_NAME) {}; @VisibleForTesting public static final String DEFAULT_IOS_SDK_VERSION = "8.4"; @VisibleForTesting public static final String DEFAULT_WATCHOS_SDK_VERSION = "2.0"; |