gn format: penalty-based line breaking WIP XXX

tools/gn/parser.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 "tools/gn/parser.h"
 
 #include "base/logging.h"
 #include "tools/gn/functions.h"
 #include "tools/gn/operators.h"
 #include "tools/gn/token.h"
@@ skipping 271 matching lines @@
 
 scoped_ptr<ParseNode> Parser::IdentifierOrCall(scoped_ptr<ParseNode> left,
                                                Token token) {
   scoped_ptr<ListNode> list(new ListNode);
   list->set_begin_token(token);
   list->set_end(make_scoped_ptr(new EndNode(token)));
   scoped_ptr<BlockNode> block;
   bool has_arg = false;
   if (LookAhead(Token::LEFT_PAREN)) {
     Token start_token = Consume();
+    list->set_begin_token(start_token);
     // Parsing a function call.
     has_arg = true;
-    if (Match(Token::RIGHT_PAREN)) {
+    if (LookAhead(Token::RIGHT_PAREN)) {
       // Nothing, just an empty call.
+      list->set_end(make_scoped_ptr(new EndNode(Consume())));
     } else {
       list = ParseList(start_token, Token::RIGHT_PAREN, false);
       if (has_error())
         return scoped_ptr<ParseNode>();
-      Consume(Token::RIGHT_PAREN, "Expected ')' after call");
+      list->set_end(make_scoped_ptr(
+          new EndNode(Consume(Token::RIGHT_PAREN, "Expected ')' after call"))));
     }
     // Optionally with a scope.
     if (LookAhead(Token::LEFT_BRACE)) {
       block = ParseBlock();
       if (has_error())
         return scoped_ptr<ParseNode>();
     }
   }
 
   if (!left && !has_arg) {
@@ skipping 299 matching lines @@
         break;
       }
     }
 
     // Suffix comments were assigned in reverse, so if there were multiple on
     // the same node, they need to be reversed.
     if ((*i)->comments() && !(*i)->comments()->suffix().empty())
       const_cast<ParseNode*>(*i)->comments_mutable()->ReverseSuffix();
   }
 }