From 654717f6f2e9e626b60debc657d3a8723f057b97 Mon Sep 17 00:00:00 2001
From: Dmitry Lomov <dslomov@google.com>
Date: Thu, 2 Mar 2017 14:39:52 +0000
Subject: Refactor implementation of native and Skylark declared providers.

1) Instead of having a single class for both, split them into
   {Skylark,Native}ClassObjectConstructors
2) Allow NativeClassObjectConstructors to customize their instantiation
   logic.
3) Prepare ClassObjectConstructor.Key to be serializable.

--
PiperOrigin-RevId: 148997553
MOS_MIGRATED_REVID=148997553
---
 .../devtools/build/lib/rules/objc/AppleSkylarkCommon.java    | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'src/main/java/com/google/devtools/build/lib/rules/objc/AppleSkylarkCommon.java')

diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/AppleSkylarkCommon.java b/src/main/java/com/google/devtools/build/lib/rules/objc/AppleSkylarkCommon.java
index 03317d1ef7..fd9b10bbf0 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/objc/AppleSkylarkCommon.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/objc/AppleSkylarkCommon.java
@@ -16,8 +16,8 @@ package com.google.devtools.build.lib.rules.objc;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.devtools.build.lib.actions.Artifact;
+import com.google.devtools.build.lib.packages.ClassObjectConstructor;
 import com.google.devtools.build.lib.packages.SkylarkClassObject;
-import com.google.devtools.build.lib.packages.SkylarkClassObjectConstructor;
 import com.google.devtools.build.lib.rules.apple.AppleToolchain;
 import com.google.devtools.build.lib.rules.apple.Platform;
 import com.google.devtools.build.lib.rules.apple.Platform.PlatformType;
@@ -116,7 +116,7 @@ public class AppleSkylarkCommon {
             + "the XcodeVersionProperties provider, use this as the key with which to retrieve it.",
     structField = true
   )
-  public SkylarkClassObjectConstructor getXcodeVersionPropertiesConstructor() {
+  public ClassObjectConstructor getXcodeVersionPropertiesConstructor() {
     return XcodeVersionProperties.SKYLARK_CONSTRUCTOR;
   }
 
@@ -127,7 +127,7 @@ public class AppleSkylarkCommon {
             + "the AppleDylibBinary provider, use this as the key with which to retrieve it.",
     structField = true
   )
-  public SkylarkClassObjectConstructor getAppleDylibBinaryConstructor() {
+  public ClassObjectConstructor getAppleDylibBinaryConstructor() {
     return AppleDylibBinaryProvider.SKYLARK_CONSTRUCTOR;
   }
 
@@ -138,7 +138,7 @@ public class AppleSkylarkCommon {
             + "the AppleExecutableBinary provider, use this as the key with which to retrieve it.",
     structField = true
   )
-  public SkylarkClassObjectConstructor getAppleExecutableBinaryConstructor() {
+  public ClassObjectConstructor getAppleExecutableBinaryConstructor() {
     return AppleExecutableBinaryProvider.SKYLARK_CONSTRUCTOR;
   }
 
@@ -150,7 +150,7 @@ public class AppleSkylarkCommon {
             + "with which to retrieve it.",
     structField = true
   )
-  public SkylarkClassObjectConstructor getAppleLoadableBundleBinaryConstructor() {
+  public ClassObjectConstructor getAppleLoadableBundleBinaryConstructor() {
     return AppleLoadableBundleBinaryProvider.SKYLARK_CONSTRUCTOR;
   }
 
@@ -161,7 +161,7 @@ public class AppleSkylarkCommon {
             + "attributes exposed by ios_device.",
     structField = true
   )
-  public SkylarkClassObjectConstructor getIosDeviceProviderConstructor() {
+  public ClassObjectConstructor getIosDeviceProviderConstructor() {
     return IosDeviceProvider.SKYLARK_CONSTRUCTOR;
   }
 
-- 
cgit v1.2.3