Fix call to evalInternal in RegexFilterExpression and VisibleFunction so it directly uses the invert parameter.
PiperOrigin-RevId: 210594733
diff --git a/src/main/java/com/google/devtools/build/lib/query2/engine/RegexFilterExpression.java b/src/main/java/com/google/devtools/build/lib/query2/engine/RegexFilterExpression.java
index ab899aa..e66fa63 100644
--- a/src/main/java/com/google/devtools/build/lib/query2/engine/RegexFilterExpression.java
+++ b/src/main/java/com/google/devtools/build/lib/query2/engine/RegexFilterExpression.java
@@ -41,16 +41,6 @@
QueryExpression expression,
final List<Argument> args,
Callback<T> callback) {
- return evalInternal(env, context, expression, args, callback, /*invert=*/ false);
- }
-
- private <T> QueryTaskFuture<Void> evalInternal(
- final QueryEnvironment<T> env,
- QueryExpressionContext<T> context,
- QueryExpression expression,
- final List<Argument> args,
- Callback<T> callback,
- boolean invert) {
String rawPattern = getPattern(args);
final Pattern compiledPattern;
try {
diff --git a/src/main/java/com/google/devtools/build/lib/query2/engine/VisibleFunction.java b/src/main/java/com/google/devtools/build/lib/query2/engine/VisibleFunction.java
index d7a5a57..055381d 100644
--- a/src/main/java/com/google/devtools/build/lib/query2/engine/VisibleFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/query2/engine/VisibleFunction.java
@@ -80,15 +80,6 @@
QueryExpression expression,
final List<Argument> args,
final Callback<T> callback) {
- return evalInternal(env, context, args, callback, /*invert=*/ false);
- }
-
- private <T> QueryTaskFuture<Void> evalInternal(
- QueryEnvironment<T> env,
- QueryExpressionContext<T> context,
- List<Argument> args,
- Callback<T> callback,
- boolean invert) {
final QueryTaskFuture<ThreadSafeMutableSet<T>> toSetFuture =
QueryUtil.evalAll(env, context, args.get(0).getExpression());
Function<ThreadSafeMutableSet<T>, QueryTaskFuture<Void>> computeVisibleNodesAsyncFunction =