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

runtime/vm/object_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/assembler.h"
 #include "vm/bigint_operations.h"
 #include "vm/isolate.h"
 #include "vm/object.h"
 #include "vm/object_store.h"
@@ skipping 21 matching lines @@
   function = Function::New(
       function_name, RawFunction::kRegularFunction,
       false, false, false, false, cls, 0);
   functions.SetAt(0, function);
   function_name = Symbols::New("bar");
   function = Function::New(
       function_name, RawFunction::kRegularFunction,
       false, false, false, false, cls, 0);
 
   const int kNumFixedParameters = 2;
-  const int kNumOptionalParameters = 3;
-  const bool kAreOptionalPositional = true;
-  function.SetNumberOfParameters(kNumFixedParameters,
-                                 kNumOptionalParameters,
-                                 kAreOptionalPositional);
+  const int kNumOptionalPositionalParameters = 3;
+  const int kNumOptionalNamedParameters = 0;
+  function.set_num_fixed_parameters(kNumFixedParameters);
+  function.SetNumOptionalParameters(kNumOptionalPositionalParameters,
+                                    kNumOptionalNamedParameters);
   functions.SetAt(1, function);
 
   function_name = Symbols::New("baz");
   function = Function::New(
       function_name, RawFunction::kRegularFunction,
       false, false, false, false, cls, 0);
   functions.SetAt(2, function);
 
   function_name = Symbols::New("Foo");
   function = Function::New(
@@ skipping 35 matching lines @@
   function_name = String::New("foo");
   function = cls.LookupDynamicFunction(function_name);
   EXPECT(!function.IsNull());
   EXPECT_EQ(0, function.num_fixed_parameters());
   EXPECT(!function.HasOptionalParameters());
 
   function_name = String::New("bar");
   function = cls.LookupDynamicFunction(function_name);
   EXPECT(!function.IsNull());
   EXPECT_EQ(kNumFixedParameters, function.num_fixed_parameters());
-  EXPECT_EQ(kNumOptionalParameters,
-            function.num_optional_positional_parameters() +
-            function.num_optional_named_parameters());
+  EXPECT_EQ(kNumOptionalPositionalParameters, function.NumOptionalParameters());
 
   const Array& interfaces = Array::Handle(Array::New(2));
   Class& interface = Class::Handle();
   String& interface_name = String::Handle();
   interface_name = Symbols::New("Harley");
   interface = Class::New(interface_name, script, Scanner::kDummyTokenIndex);
   interfaces.SetAt(0, Type::Handle(Type::NewNonParameterizedType(interface)));
   interface_name = Symbols::New("Norton");
   interface = Class::New(interface_name, script, Scanner::kDummyTokenIndex);
   interfaces.SetAt(1, Type::Handle(Type::NewNonParameterizedType(interface)));
@@ skipping 2993 matching lines @@
   isolate->heap()->CollectAllGarbage();
   EXPECT(weak1.key() == Object::null());
   EXPECT(weak1.value() == Object::null());
   EXPECT(weak2.key() == Object::null());
   EXPECT(weak2.value() == Object::null());
 }
 
 #endif  // defined(TARGET_ARCH_IA32) || defined(TARGET_ARCH_X64).
 
 }  // namespace dart