Delete Environment.hasVariable
It's virtually unused, as callers generally want to know what is the value.
https://github.com/bazelbuild/bazel/issues/5637
RELNOTES: None.
PiperOrigin-RevId: 210401091
diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Environment.java b/src/main/java/com/google/devtools/build/lib/syntax/Environment.java
index e32b545..888642e 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Environment.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Environment.java
@@ -1062,10 +1062,6 @@
return this;
}
- public boolean hasVariable(String varname) {
- return lookup(varname) != null;
- }
-
/**
* Initializes a binding in this Environment. It is an error if the variable is already bound.
* This is not for end-users, and will throw an AssertionError in case of conflict.
@@ -1074,7 +1070,7 @@
* @return this Environment, in fluid style
*/
public Environment setup(String varname, Object value) {
- if (hasVariable(varname)) {
+ if (lookup(varname) != null) {
throw new AssertionError(String.format("variable '%s' already bound", varname));
}
return setupOverride(varname, value);