Cleanups.

runtime/vm/code_generator.cc

 // Copyright (c) 2011, 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 "vm/code_generator.h"
 
 #include "vm/code_index_table.h"
 #include "vm/code_patcher.h"
 #include "vm/compiler.h"
 #include "vm/dart_api_impl.h"
@@ skipping 898 matching lines @@
   cache = Array::Grow(cache, cache.Length() + (8 * kNumEntries));
   class_.set_functions_cache(cache);
   EnterFunctionAt(ix,
                   cache,
                   function,
                   num_arguments,
                   num_named_arguments);
 }
 
 
-// TODO(regis): The actual names of named arguments must match as well.
+// Only the number of named arguments is checked, but not the actual names.
 RawCode* FunctionsCache::LookupCode(const String& function_name,
                                     int num_arguments,
                                     int num_named_arguments) {
   const Array& cache = Array::Handle(class_.functions_cache());
   String& test_name = String::Handle();
   for (intptr_t i = 0; i < cache.Length(); i += kNumEntries) {
     test_name ^= cache.At(i + FunctionsCache::kFunctionName);
     if (test_name.IsNull()) {
       // Found NULL, no more entries to check, abort lookup.
       return Code::null();
@@ skipping 13 matching lines @@
       }
     }
   }
   // The cache is null terminated, therefore the loop above should never
   // terminate by itself.
   UNREACHABLE();
   return Code::null();
 }
 
 }  // namespace dart