Minor code review change.

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 10 matching lines @@
       name = name.substring(5);
       isInitializer = true;
     }
 
     type = method.resolveType(definition.type, false);
 
     if (definition.value != null) {
       // To match VM, detect cases where value was not actually specified in
       // code and don't signal errors.
       // TODO(jimhug): Clean up after issue #352 is resolved.
-      if (definition.value.span.start == definition.span.start) {
-        return;
-      }
+      if (!hasDefaultValue) return;
 
       if (method.name == ':call') {
         // TODO(jimhug): Need simpler way to detect "true" function types vs.
         //   regular methods being used as function types for closures.
         // TODO(sigmund): Disallow non-null default values for native calls?
         if (method.definition.body == null && !method.isNative) {
           world.error('default value not allowed on function type',
               definition.span);
         }
       } else if (method.isAbstract) {
@@ skipping 1544 matching lines @@
   }
 
   void forEach(void f(Member member)) {
     factories.forEach((_, Map constructors) {
       constructors.forEach((_, Member member) {
         f(member);
       });
     });
   }
 }