Add the ability to pass long lists of inputs to 'gn refs'.

tools/gn/command_refs.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <map>
 #include <set>
 
 #include "base/command_line.h"
+#include "base/files/file_util.h"
+#include "base/strings/string_split.h"
 #include "tools/gn/commands.h"
 #include "tools/gn/deps_iterator.h"
 #include "tools/gn/filesystem_utils.h"
 #include "tools/gn/input_file.h"
 #include "tools/gn/item.h"
 #include "tools/gn/setup.h"
 #include "tools/gn/standard_out.h"
 #include "tools/gn/target.h"
 
 namespace commands {
@@ skipping 240 matching lines @@
   FilterAndPrintTargetSet(false, results);
   return results.size();
 }
 
 }  // namespace
 
 const char kRefs[] = "refs";
 const char kRefs_HelpShort[] =
     "refs: Find stuff referencing a target or file.";
 const char kRefs_Help[] =
-    "gn refs <out_dir> (<label_pattern>|<label>|<file>)* [--all]\n"
+    "gn refs <out_dir> (<label_pattern>|<label>|<file>|@<response_file>)* "
+    "[--all]\n"
     "        [--all-toolchains] [--as=...] [--testonly=...] [--type=...]\n"
     "\n"
     "  Finds reverse dependencies (which targets reference something). The\n"
     "  input is a list containing:\n"
     "\n"
     "   - Target label: The result will be which targets depend on it.\n"
     "\n"
     "   - Config label: The result will be which targets list the given\n"
     "     config in its \"configs\" or \"public_configs\" list.\n"
     "\n"
     "   - Label pattern: The result will be which targets depend on any\n"
     "     target matching the given pattern. Patterns will not match\n"
     "     configs. These are not general regular expressions, see\n"
     "     \"gn help label_pattern\" for details.\n"
     "\n"
     "   - File name: The result will be which targets list the given file in\n"
     "     its \"inputs\", \"sources\", \"public\", or \"data\". Any input\n"
     "     that does not contain wildcards and does not match a target or a\n"
     "     config will be treated as a file.\n"
     "\n"
+    "   - Response file: If the input starts with an \"@\", it will be\n"
+    "     interpreted as a path to a file containing a list of labels or\n"
+    "     file names, one per line. This allows us to handle long lists\n"
+    "     of inputs without worrying about command line limits.\n"
+    "\n"
     "Options\n"
     "\n"
     "  --all\n"
     "      When used without --tree, will recurse and display all unique\n"
     "      dependencies of the given targets. For example, if the input is\n"
     "      a target, this will output all targets that depend directly or\n"
     "      indirectly on the input. If the input is a file, this will output\n"
     "      all targets that depend directly or indirectly on that file.\n"
     "\n"
     "      When used with --tree, turns off eliding to show a complete tree.\n"
@@ skipping 81 matching lines @@
   bool tree = cmdline->HasSwitch("tree");
   bool all = cmdline->HasSwitch("all");
   bool all_toolchains = cmdline->HasSwitch("all-toolchains");
 
   Setup* setup = new Setup;
   setup->set_check_for_bad_items(false);
   if (!setup->DoSetup(args[0], false) || !setup->Run())
     return 1;
 
   // The inputs are everything but the first arg (which is the build dir).
-  std::vector<std::string> inputs(args.begin() + 1, args.end());
+  std::vector<std::string> inputs;
+  for (auto it = args.begin() + 1; it != args.end(); it++) {

[brettw, 2015/04/17 21:23:25]

I think this case is much clearer as an integer iterator, personally:
  for (size_t i = 1; i < args.size(); i++)
and it's shorter.

[Dirk Pranke, 2015/04/17 21:35:16]

Done.

+    auto arg = *it;

[brettw, 2015/04/17 21:23:25]

This is secretly hiding a string copy. You could make it a reference, but I'd
just delete it (you actually didn't even use it consistently below anyway)

[Dirk Pranke, 2015/04/17 21:35:16]

True. Done.

+
+    if (arg[0] == '@') {
+      // The argument is as a path to a response file.
+      std::string contents;
+      std::vector<std::string> lines;
+      bool ret = base::ReadFileToString(base::FilePath(arg.substr(1)),

[brettw, 2015/04/17 21:23:25]

This will fail on Windows. You'll want UTF8ToFilePath (in filesystem utils)

[Dirk Pranke, 2015/04/17 21:35:15]

Done.

+                                        &contents);
+      if (!ret) {
+        Err(Location(), "Response file " + arg.substr(1) + " not found.")
+            .PrintToStdout();
+        return 1;
+      }
+      base::SplitString(contents, '\n', &lines);
+      for (const auto& line : lines) {
+        if (!line.empty())
+          inputs.push_back(line);
+      }
+    } else {
+      // The argument is a label or a path.
+      inputs.push_back(*it);
+    }
+  }
 
   // Get the matches for the command-line input.
   UniqueVector<const Target*> target_matches;
   UniqueVector<const Config*> config_matches;
   UniqueVector<const Toolchain*> toolchain_matches;
   UniqueVector<SourceFile> file_matches;
   if (!ResolveFromCommandLineInput(setup, inputs, all_toolchains,
                                    &target_matches, &config_matches,
                                    &toolchain_matches, &file_matches))
     return 1;
@@ skipping 42 matching lines @@
 
   // If you ask for the references of a valid target, but that target has
   // nothing referencing it, we'll get here without having printed anything.
   if (!quiet && cnt == 0)
     OutputString("Nothing references this.\n", DECORATION_YELLOW);
 
   return 0;
 }
 
 }  // namespace commands