Fix FormalParameterErrorLocations member names

src/parser.cc

 // Copyright 2012 the V8 project 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 "src/v8.h"
 
 #include "src/api.h"
 #include "src/ast.h"
 #include "src/bailout-reason.h"
 #include "src/base/platform/platform.h"
@@ skipping 3702 matching lines @@
   const AstRawString* raw_name = proxy->raw_name();
   Scanner::Location param_location(proxy->position(),
                                    proxy->position() + raw_name->length());
 
   if (proxy->is_this()) {
     ReportMessageAt(param_location, "this_formal_parameter");
     *ok = false;
     return;
   }
 
-  if (!error_locs->eval_or_arguments_.IsValid() && IsEvalOrArguments(raw_name))
-    error_locs->eval_or_arguments_ = param_location;
-  if (!error_locs->reserved_.IsValid() && IsFutureStrictReserved(raw_name))
-    error_locs->reserved_ = param_location;
-  if (!error_locs->undefined_.IsValid() && IsUndefined(raw_name))
-    error_locs->undefined_ = param_location;
+  if (!error_locs->eval_or_arguments.IsValid() && IsEvalOrArguments(raw_name))
+    error_locs->eval_or_arguments = param_location;
+  if (!error_locs->reserved.IsValid() && IsFutureStrictReserved(raw_name))
+    error_locs->reserved = param_location;
+  if (!error_locs->undefined.IsValid() && IsUndefined(raw_name))
+    error_locs->undefined = param_location;
 
   // TODO(wingo): Fix quadratic check.  (Scope::IsDeclaredParameter has the same
   // issue.)
   for (int i = 0; i < result->length(); i++) {
     // Eagerly report the error here; duplicate formal parameter names are never
     // allowed in arrow functions.
     if (raw_name == result->at(i)) {
       ReportMessageAt(param_location,
                       "duplicate_arrow_function_formal_parameter");
       *ok = false;
@@ skipping 209 matching lines @@
         ParseFormalParameterList(&error_locs, &has_rest, CHECK_OK);
     Expect(Token::RPAREN, CHECK_OK);
     int formals_end_position = scanner()->location().end_pos;
 
     CheckArityRestrictions(params->length(), arity_restriction, start_position,
                            formals_end_position, CHECK_OK);
 
     scope->set_start_position(start_position);
 
     num_parameters = DeclareFormalParameters(params, scope_, has_rest);
-    if (error_locs.duplicate_.IsValid()) {
+    if (error_locs.duplicate.IsValid()) {
       duplicate_parameters = FunctionLiteral::kHasDuplicateParameters;
     }
 
 
     Expect(Token::LBRACE, CHECK_OK);
 
     // If we have a named function expression, we add a local variable
     // declaration to the body of the function with the name of the
     // function and let it refer to the function itself (closure).
     // NOTE: We create a proxy and resolve it here so that in the
@@ skipping 1804 matching lines @@
 
 Expression* Parser::SpreadCallNew(Expression* function,
                                   ZoneList<v8::internal::Expression*>* args,
                                   int pos) {
   args->InsertAt(0, function, zone());
 
   return factory()->NewCallRuntime(
       ast_value_factory()->reflect_construct_string(), NULL, args, pos);
 }
 } }  // namespace v8::internal