commit 34f705832500efa7669942d26beb30d879dc064e
author Googler <noreply@google.com> Mon Nov 25 12:27:34 2019 -0800
committer Copybara-Service <copybara-worker@google.com> Mon Nov 25 12:29:47 2019 -0800
tree 0aa4afdf8f391caaebadd862a3849c36f726f52e
parent 565c199f51cf8b9898342c47c07666c1a8159375

bazel: rename SkylarkValue

This change moves skylarkinterface.SkylarkValue to syntax.StarlarkValue,
and also skylarkinterface.SkylarkPrinter to syntax.Printer, where its
functionality is merged into the existing class of that name.

This change is entirely mechanical, except for syntax.Printer (naturally),
and MethodLibrary and SkylarkEvaluationTest, which needed a tweak
due the removal of the Printer.debugPrint static method.

All classes related to the representation of values are now in the evaluator package;
skylarkinterface now contains only the annotations.

This is a breaking change for copybara.

BEGIN_PUBLIC
bazel: rename SkylarkValue
END_PUBLIC
PiperOrigin-RevId: 282410278

src/main/java/com/google/devtools/build/lib/packages/Attribute.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/Attribute.java b/src/main/java/com/google/devtools/build/lib/packages/Attribute.java
index ce7f1d7..9bd5b11 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/Attribute.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/Attribute.java
@@ -41,11 +41,11 @@
 import com.google.devtools.build.lib.packages.Type.LabelClass;
 import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec.VisibleForSerialization;
-import com.google.devtools.build.lib.skylarkinterface.SkylarkValue;
 import com.google.devtools.build.lib.syntax.ClassObject;
 import com.google.devtools.build.lib.syntax.EvalException;
 import com.google.devtools.build.lib.syntax.EvalUtils;
 import com.google.devtools.build.lib.syntax.Starlark;
+import com.google.devtools.build.lib.syntax.StarlarkValue;
 import com.google.devtools.build.lib.util.FileType;
 import com.google.devtools.build.lib.util.FileTypeSet;
 import com.google.devtools.build.lib.util.StringUtil;
@@ -1301,7 +1301,7 @@
    *
    * <p>Implementations of this interface must be immutable.
    */
-  public abstract static class ComputedDefault implements SkylarkValue {
+  public abstract static class ComputedDefault implements StarlarkValue {
     private final ImmutableList<String> dependencies;
 
     /**
@@ -1611,7 +1611,7 @@
    *     Label}, or a {@link List} of {@link Label} objects.
    */
   @Immutable
-  public abstract static class LateBoundDefault<FragmentT, ValueT> implements SkylarkValue {
+  public abstract static class LateBoundDefault<FragmentT, ValueT> implements StarlarkValue {
     /**
      * Functional interface for computing the value of a late-bound attribute.
      *