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/syntax/Tuple.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Tuple.java b/src/main/java/com/google/devtools/build/lib/syntax/Tuple.java
index b0bad85..0462835 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Tuple.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Tuple.java
@@ -20,7 +20,6 @@
 import com.google.devtools.build.lib.events.Location;
 import com.google.devtools.build.lib.skylarkinterface.SkylarkModule;
 import com.google.devtools.build.lib.skylarkinterface.SkylarkModuleCategory;
-import com.google.devtools.build.lib.skylarkinterface.SkylarkPrinter;
 import java.util.AbstractCollection;
 import java.util.AbstractList;
 import java.util.Arrays;
@@ -163,13 +162,13 @@
   }
 
   @Override
-  public void repr(SkylarkPrinter printer) {
-    // TODO(adonovan): when SkylarkPrinter moves into this package,
+  public void repr(Printer printer) {
+    // TODO(adonovan): when Printer moves into this package,
     // inline and simplify this, the sole call with isTuple=true.
     printer.printList(this, /*isTuple=*/ true);
   }
 
-  // TODO(adonovan): SkylarkValue has 3 String methods yet still we need this fourth. Why?
+  // TODO(adonovan): StarlarkValue has 3 String methods yet still we need this fourth. Why?
   @Override
   public String toString() {
     return Starlark.repr(this);