Limit the maximum number of formal parameters (32K fixed and 32K optional)

runtime/vm/code_generator_test.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "platform/assert.h"
 #include "vm/globals.h"
 #if defined(TARGET_ARCH_IA32) || defined(TARGET_ARCH_X64)
 
 #include "vm/ast.h"
 #include "vm/assembler.h"
@@ skipping 188 matching lines @@
 //   int dec(int a, [int b = 1]) native: "TestSmiSub";
 // The native entry TestSmiSub implements dec natively.
 CODEGEN_TEST_GENERATE(NativeDecCodegen, test) {
   // A NativeBodyNode, preceded by an EnterNode and followed by a ReturnNode,
   // implements the body of a native Dart function. Let's take this native
   // function as an example: int dec(int a, int b = 1) native;
   // Since this function has an optional parameter, its prologue will copy
   // incoming parameters to locals.
   SequenceNode* node_seq = test->node_sequence();
   const int num_fixed_params = 1;
-  const int num_opt_params = 1;
-  const int num_params = num_fixed_params + num_opt_params;
+  const int num_opt_pos_params = 1;
+  const int num_opt_named_params = 0;
+  const int num_params =
+      num_fixed_params + num_opt_pos_params + num_opt_named_params;
   LocalScope* local_scope = node_seq->scope();
   local_scope->AddVariable(NewTestLocalVariable("a"));
   local_scope->AddVariable(NewTestLocalVariable("b"));
   ASSERT(local_scope->num_variables() == num_params);
-  const Array& default_values = Array::ZoneHandle(Array::New(num_opt_params));
+  const Array& default_values =
+      Array::ZoneHandle(Array::New(num_opt_pos_params));
   default_values.SetAt(0, Smi::ZoneHandle(Smi::New(1)));  // b = 1.
   test->set_default_parameter_values(default_values);
   const Function& function = test->function();
-  function.SetNumberOfParameters(num_fixed_params, num_opt_params, true);
+  function.set_num_fixed_parameters(num_fixed_params);
+  function.SetNumOptionalParameters(num_opt_pos_params, num_opt_named_params);
   const bool has_opt_params = true;
   const String& native_name =
       String::ZoneHandle(Symbols::New("TestSmiSub"));
   NativeFunction native_function =
       reinterpret_cast<NativeFunction>(TestSmiSub);
   node_seq->Add(new ReturnNode(kPos,
                                new NativeBodyNode(kPos,
                                                   native_name,
                                                   native_function,
                                                   num_params,
@@ skipping 134 matching lines @@
 CODEGEN_TEST_RUN(InstanceCallCodegen, Smi::New(42))
 
 
 // Tested Dart code:
 //   int sum(int a, int b,
 //           [int c = 10, int d = 21, int e = -32]) native: "TestSmiSum";
 // The native entry TestSmiSum implements sum natively.
 CODEGEN_TEST_GENERATE(NativeSumCodegen, test) {
   SequenceNode* node_seq = test->node_sequence();
   const int num_fixed_params = 2;
-  const int num_opt_params = 3;
-  const int num_params = num_fixed_params + num_opt_params;
+  const int num_opt_pos_params = 3;
+  const int num_opt_named_params = 0;
+  const int num_params =
+      num_fixed_params + num_opt_pos_params + num_opt_named_params;
   LocalScope* local_scope = node_seq->scope();
   local_scope->AddVariable(NewTestLocalVariable("a"));
   local_scope->AddVariable(NewTestLocalVariable("b"));
   local_scope->AddVariable(NewTestLocalVariable("c"));
   local_scope->AddVariable(NewTestLocalVariable("d"));
   local_scope->AddVariable(NewTestLocalVariable("e"));
   ASSERT(local_scope->num_variables() == num_params);
-  const Array& default_values = Array::ZoneHandle(Array::New(num_opt_params));
+  const Array& default_values =
+      Array::ZoneHandle(Array::New(num_opt_pos_params));
   default_values.SetAt(0, Smi::ZoneHandle(Smi::New(10)));
   default_values.SetAt(1, Smi::ZoneHandle(Smi::New(21)));
   default_values.SetAt(2, Smi::ZoneHandle(Smi::New(-32)));
   test->set_default_parameter_values(default_values);
   const Function& function = test->function();
-  function.SetNumberOfParameters(num_fixed_params, num_opt_params, true);
+  function.set_num_fixed_parameters(num_fixed_params);
+  function.SetNumOptionalParameters(num_opt_pos_params, num_opt_named_params);
   function.set_parameter_types(Array::Handle(Array::New(num_params)));
   function.set_parameter_names(Array::Handle(Array::New(num_params)));
   const Type& param_type = Type::Handle(Type::DynamicType());
   for (int i = 0; i < num_params - 1; i++) {
     function.SetParameterTypeAt(i, param_type);
   }
   const bool has_opt_params = true;
   const String& native_name =
       String::ZoneHandle(Symbols::New("TestSmiSum"));
   NativeFunction native_function =
@@ skipping 109 matching lines @@
   app_lib ^= libs.At(num_libs - 1);
   ASSERT(!app_lib.IsNull());
   const Class& cls = Class::Handle(
       app_lib.LookupClass(String::Handle(Symbols::New("A"))));
   EXPECT_EQ(cls.raw(), result.clazz());
 }
 
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_IA32 || defined(TARGET_ARCH_X64)