gn format: support absolute file system paths

tools/gn/command_format.cc

 // Copyright 2014 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 <sstream>
 
 #include "base/command_line.h"
 #include "base/strings/string_split.h"
 #include "tools/gn/commands.h"
+#include "tools/gn/filesystem_utils.h"
 #include "tools/gn/input_file.h"
 #include "tools/gn/parser.h"
 #include "tools/gn/scheduler.h"
 #include "tools/gn/setup.h"
 #include "tools/gn/source_file.h"
 #include "tools/gn/tokenizer.h"
 
 namespace commands {
 
 const char kSwitchDumpTree[] = "dump-tree";
+const char kSwitchInPlace[] = "in-place";
 
 const char kFormat[] = "format";
 const char kFormat_HelpShort[] =
-    "format: Format .gn file.";
+    "format: Format .gn file. (ALPHA, WILL DESTROY DATA!)";
 const char kFormat_Help[] =
-    "gn format: Format .gn file. (ALPHA, WILL CURRENTLY DESTROY DATA!)\n"
-    "\n"
-    "  gn format //some/BUILD.gn\n"
-    "  gn format some\\BUILD.gn\n"
+    "gn format [--dump-tree] [--in-place] BUILD.gn\n"
     "\n"
     "  Formats .gn file to a standard format. THIS IS NOT FULLY IMPLEMENTED\n"
     "  YET! IT WILL EAT YOUR BEAUTIFUL .GN FILES. AND YOUR LAUNDRY.\n"
     "  At a minimum, make sure everything is `git commit`d so you can\n"
-    "  `git checkout -f` to recover.\n";
+    "  `git checkout -f` to recover.\n"
+    "\n"
+    "Arguments\n"
+    "  --dump-tree\n"
+    "      For debugging only, dumps the parse tree.\n"
+    "\n"
+    "  --in-place\n"
+    "      Instead writing the formatted file to stdout, replace the input\n"
+    "      with the formatted output.\n"
+    "\n"
+    "Examples\n"
+    "  gn format //some/BUILD.gn\n"
+    "  gn format some\\BUILD.gn\n"
+    "  gn format /abspath/some/BUILD.gn\n";
 
 namespace {
 
 const int kIndentSize = 2;
 const int kMaximumWidth = 80;
 
 class Printer {
  public:
   Printer();
   ~Printer();
@@ skipping 439 matching lines @@
   else if (style == kSequenceStyleList)
     Print("]");
   else if (style == kSequenceStyleBracedBlock)
     Print("}");
 
   margin_ = old_margin;
 }
 
 }  // namespace
 
-bool FormatFileToString(const std::string& input_filename,
+bool FormatFileToString(Setup* setup,
+                        const SourceFile& file,
                         bool dump_tree,
                         std::string* output) {
-  Setup setup;
   Err err;
-  SourceFile input_file(input_filename);
   const ParseNode* parse_node =
-      setup.scheduler().input_file_manager()->SyncLoadFile(
-          LocationRange(), &setup.build_settings(), input_file, &err);
+      setup->scheduler().input_file_manager()->SyncLoadFile(
+          LocationRange(), &setup->build_settings(), file, &err);
   if (err.has_error()) {
     err.PrintToStdout();
     return false;
   }
   if (dump_tree) {
     std::ostringstream os;
     parse_node->Print(os, 0);
     printf("----------------------\n");
     printf("-- PARSE TREE --------\n");
     printf("----------------------\n");
@@ skipping 13 matching lines @@
   // print the formatted output to stdout.
   if (args.size() != 1) {
     Err(Location(), "Expecting exactly one argument, see `gn help format`.\n")
         .PrintToStdout();
     return 1;
   }
 
   bool dump_tree =
       base::CommandLine::ForCurrentProcess()->HasSwitch(kSwitchDumpTree);
 
-  std::string input_name = args[0];
-  if (input_name[0] != '/') {
-    std::replace(input_name.begin(), input_name.end(), '\\', '/');
-    input_name = "//" + input_name;
-  }
+  Setup setup;
+  SourceDir source_dir =
+      SourceDirForCurrentDirectory(setup.build_settings().root_path());
+  SourceFile file = source_dir.ResolveRelativeFile(args[0]);
+
   std::string output_string;
-  if (FormatFileToString(input_name, dump_tree, &output_string)) {
+  if (FormatFileToString(&setup, file, dump_tree, &output_string)) {
+    bool in_place =
+        base::CommandLine::ForCurrentProcess()->HasSwitch(kSwitchInPlace);
+    CHECK(!in_place) << "TODO(scottmg)";

[scottmg, 2014/10/02 21:54:29]

(is there an easy way to map a SourceFile back to disk file?)

[brettw, 2014/10/02 22:03:27]

setup.build_settings()->GetFullPath(source_file)

     printf("%s", output_string.c_str());
   }
 
   return 0;
 }
 
 }  // namespace commands