tools/gn: disallow non-canonical integer literals

tools/gn/parse_tree.cc

Index: tools/gn/parse_tree.cc
diff --git a/tools/gn/parse_tree.cc b/tools/gn/parse_tree.cc
index f5897065548f01b102328c3edaa41a7f31a27af6..88aa0a90bab9bb243f05437782106dba2db2431d 100644
--- a/tools/gn/parse_tree.cc
+++ b/tools/gn/parse_tree.cc
@@ -634,8 +634,16 @@ Value LiteralNode::Execute(Scope* scope, Err* err) const {
     case Token::FALSE_TOKEN:
       return Value(this, false);
     case Token::INTEGER: {
+      auto s = value_.value();

[tfarina, 2015/03/27 00:17:17]

to not use it at all. I don't like to have to go to the header (parse_tree.h) to
see what value_ is, then to go to header of value_, to figure out what value()
is.

[mdempsky, 2015/03/27 00:18:54]

Done.

+      if ((s.starts_with("0") && s.size() > 1) || s.starts_with("-0")) {
+        if (s == "-0")
+          *err = MakeErrorDescribing("Negative zero doesn't make sense");
+        else
+          *err = MakeErrorDescribing("Leading zeros not allowed");
+        return Value();
+      }
       int64 result_int;
-      if (!base::StringToInt64(value_.value(), &result_int)) {
+      if (!base::StringToInt64(s, &result_int)) {
         *err = MakeErrorDescribing("This does not look like an integer");
         return Value();
       }