Code review changes

frog/member.dart

 // 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.
 
 /** A formal parameter to a [Method]. */
 class Parameter {
   FormalNode definition;
   Member method;
 
   String name;
@@ skipping 799 matching lines @@
       // TODO(sigmund): can we avoid generating the entire type, but only what
       // we need?
       declaringType.markUsed();
       var type = declaringType.isTop ? '' : '${declaringType.jsname}.';
       return new Value(functionType, '$type$jsname', node.span);
     }
     _providePropertySyntax = true;
     return new Value(functionType, '${target.code}.get\$$jsname()', node.span);
   }
 
+  /**
+   * Checks if the named arguments are in their natural or 'home' positions,
+   * i.e. they may be passed directly without inserting, deleting or moving the
+   * arguments to correspond with the parameters.
+   */
   bool namesInHomePositions(Arguments args) {
     if (!args.hasNames) return true;
 
     for (int i = args.bareCount; i < args.values.length; i++) {
-      if (i >= parameters.length)
+      if (i >= parameters.length) {
         return false;
-      if (args.getName(i) != parameters[i].name)
+      }
+      if (args.getName(i) != parameters[i].name) {
         return false;
+      }
     }
     return true;
   }
 
   bool namesInOrder(Arguments args) {
     if (!args.hasNames) return true;
 
     int lastParameter = null;
     for (int i = args.bareCount; i < parameters.length; i++) {
       var p = args.getIndexOfName(parameters[i].name);
@@ skipping 952 matching lines @@
   }
 
   void forEach(void f(Member member)) {
     factories.forEach((_, Map constructors) {
       constructors.forEach((_, Member member) {
         f(member);
       });
     });
   }
 }