Make sure setter does not conflict with method

runtime/vm/parser.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 "vm/parser.h"
 
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
 #include "vm/compiler.h"
 #include "vm/compiler_stats.h"
@@ skipping 564 matching lines @@
             bool is_interface,
             intptr_t token_pos)
       : clazz_(cls),
         class_name_(cls_name),
         is_interface_(is_interface),
         token_pos_(token_pos),
         functions_(GrowableObjectArray::Handle(GrowableObjectArray::New())),
         fields_(GrowableObjectArray::Handle(GrowableObjectArray::New())) {
   }
 
+  // Parameter 'name' is the unmangled name, i.e. without the setter
+  // name mangling.
   bool FunctionNameExists(const String& name, RawFunction::Kind kind) const {
     // First check if a function or field of same name exists.
-    if (FunctionExists(name)) {
+    if ((kind != RawFunction::kSetterFunction) && FunctionExists(name)) {
       return true;
     }
     // Now check whether there is a field and whether its implicit getter
     // or setter collides with the name.
     Field* field = LookupField(name);
     if (field != NULL) {
       if (kind == RawFunction::kSetterFunction) {
         // It's ok to have an implicit getter, it does not collide with
         // this setter function.
         if (!field->is_final()) {
@@ skipping 9351 matching lines @@
 void Parser::SkipQualIdent() {
   ASSERT(IsIdentifier());
   ConsumeToken();
   if (CurrentToken() == Token::kPERIOD) {
     ConsumeToken();  // Consume the kPERIOD token.
     ExpectIdentifier("identifier expected after '.'");
   }
 }
 
 }  // namespace dart