commit 7e146dd68023cdf8fe17c26a5fe858345c798f8a
author Stiopa Koltsov <stepan.koltsov@gmail.com> Tue Jan 04 12:21:42 2022 -0800
committer Copybara-Service <copybara-worker@google.com> Tue Jan 04 12:23:20 2022 -0800
tree f69a45389ccaa465afffa14492f0e5536cd531d4
parent 7d8d55ac84b771a3efa3aec1c126cf0f61b374a4

Starlark: assert Program.loads.size() == Program.loadLocations.size()

Internal state check, mostly to make code more readable.

Closes #12923.

PiperOrigin-RevId: 419653657

src/main/java/net/starlark/java/syntax/Program.java

diff --git a/src/main/java/net/starlark/java/syntax/Program.java b/src/main/java/net/starlark/java/syntax/Program.java
index ba981db..1bf6547 100644
--- a/src/main/java/net/starlark/java/syntax/Program.java
+++ b/src/main/java/net/starlark/java/syntax/Program.java
@@ -13,6 +13,7 @@
 // limitations under the License.
 package net.starlark.java.syntax;
 
+import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
 
 /**
@@ -28,6 +29,9 @@
 
   private Program(
       Resolver.Function body, ImmutableList<String> loads, ImmutableList<Location> loadLocations) {
+    Preconditions.checkArgument(
+        loads.size() == loadLocations.size(), "each load must have a corresponding location");
+
     // TODO(adonovan): compile here.
     this.body = body;
     this.loads = loads;