// Copyright 2015 The Bazel Authors. All rights reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//    http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package com.google.devtools.build.lib.query2.output;

import com.google.devtools.build.lib.packages.Target;
import com.google.devtools.build.lib.query2.engine.DigraphQueryEvalResult;
import com.google.devtools.build.lib.query2.engine.OutputFormatterCallback;
import com.google.devtools.build.lib.query2.engine.QueryEvalResult;
import com.google.devtools.build.lib.query2.output.OutputFormatter.StreamedFormatter;
import com.google.devtools.build.lib.query2.output.QueryOptions.OrderOutput;
import java.io.IOException;
import java.io.OutputStream;
import java.util.Set;

/** Static utility methods for outputting a query. */
public class QueryOutputUtils {
  // Utility class cannot be instantiated.
  private QueryOutputUtils() {}

  public static boolean shouldStreamResults(QueryOptions queryOptions, OutputFormatter formatter) {
    return queryOptions.orderOutput == OrderOutput.NO
        && formatter instanceof StreamedFormatter;
  }

  public static void output(QueryOptions queryOptions, QueryEvalResult result,
      Set<Target> targetsResult, OutputFormatter formatter, OutputStream outputStream,
      AspectResolver aspectResolver)
      throws IOException, InterruptedException {
    /*
     * This is not really streaming, but we are using the streaming interface for writing into the
     * output everything in one batch. This happens when the QueryEnvironment does not
     * support streaming but we don't care about ordered results.
     */
    boolean orderedResults = !shouldStreamResults(queryOptions, formatter);
    if (orderedResults) {
      formatter.output(queryOptions,
          ((DigraphQueryEvalResult<Target>) result).getGraph().extractSubgraph(targetsResult),
          outputStream, aspectResolver);
    } else {
      StreamedFormatter streamedFormatter = (StreamedFormatter) formatter;
      streamedFormatter.setOptions(queryOptions, aspectResolver);
      OutputFormatterCallback.processAllTargets(
          streamedFormatter.createPostFactoStreamCallback(outputStream, queryOptions),
          targetsResult);
    }
  }
}

@@ -2077,34 +2077,12 @@
     
-boolean orderedResults = !
+if (
 shou
@@ -2125,219 +2125,9 @@
 ter)
-;%0A    if (orderedResults) %7B%0A      formatter.output(queryOptions,%0A          ((DigraphQueryEvalResult%3CTarget%3E) result).getGraph().extractSubgraph(targetsResult),%0A          outputStream, aspectResolver);%0A    %7D else
+)
  %7B%0A 
@@ -2428,16 +2428,354 @@
 esult);%0A
+    %7D else %7B%0A      @SuppressWarnings(%22unchecked%22)%0A      DigraphQueryEvalResult%3CTarget%3E digraphQueryEvalResult =%0A          (DigraphQueryEvalResult%3CTarget%3E) result;%0A      formatter.output(%0A          queryOptions,%0A          digraphQueryEvalResult.getGraph().extractSubgraph(targetsResult),%0A          outputStream,%0A          aspectResolver);%0A
     %7D%0A  

Make code much easier to read without misleading variable name and negation.